Try to fix NULLs in E2Lib.PlayerChips

[Astralcircle]

I'm not sure if this will fix the error, but it's worth a try

- addons/wire/lua/entities/gmod_wire_expression2/init.lua:311: attempt to index local 'tab' (a nil value)
1. getTotalTime - addons/wire/lua/entities/gmod_wire_expression2/init.lua:311
 2. checkCpuTime - addons/wire/lua/entities/gmod_wire_expression2/init.lua:342
  3. <unknown> - addons/wire/lua/entities/gmod_wire_expression2/init.lua:376
   4. <unknown> - addons/hook-library/lua/includes/modules/hook.lua:313

[thegrb93]

Might be worth replacing E2Lib.PlayerChips __index with a 'getOrCreate' function. Then regular indexing can return nil and you don't need to use rawget.

The problem is because of a null chip, not owner. https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/init.lua#L310
It seems like a chip became invalid but didn't get removed from the table?

[thegrb93]

I forget if chip owner can change or not, but that might be a possible cause if so.

[Astralcircle]

Really don't know how it appears. When it appears, it spams thousands of errors every tick and seems to only disappear when the owner leaves

[thegrb93]

You could check if 'tab' is nil and delete the chip from the table if so. I'd add back the 'chips' table and make it indexed by chip, so you can use pairs to iterate and delete them while iterating.

[thegrb93]

Ideally, the OnRemove would be functioning correctly though.

[Astralcircle]

Maybe something happens to the self.player value during removing? Maybe it should just iterate through the entire E2Lib.PlayerChips table when removing a chip to make it more reliable?