You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
cycles is only used to print the id before a table, but if the additional references aren't printed then the id isn't useful. Definitely doesn't seem worth the cost just to indicate that a value is referenced deeper inside the table. I had cases where inspect(t, { depth = 1 }) took 1000 ms but removing countCycles reduced it to 1 ms.
The text was updated successfully, but these errors were encountered:
inspect can be slow with
{ depth = 1 }
on large tables because countCycles still fully processes the whole table.inspect.lua/inspect.lua
Lines 304 to 314 in 9c8a68d
cycles is only used to print the id before a table, but if the additional references aren't printed then the id isn't useful. Definitely doesn't seem worth the cost just to indicate that a value is referenced deeper inside the table. I had cases where
inspect(t, { depth = 1 })
took 1000 ms but removing countCycles reduced it to 1 ms.The text was updated successfully, but these errors were encountered: