NevermoreEngine icon indicating copy to clipboard operation
NevermoreEngine copied to clipboard

Optimise + document Table library

Open OttoHatt opened this issue 2 years ago • 1 comments

Updated some of the table util functions to use the newer table. methods. Hopefully they'll be faster!(?). Also changed some doc phrasing.

I've done my best to test, but I know changes to a module like this can incur a lot of hard-to-find and obscure bugs (likely to do with mix/matching dictionary-style and array-style tables). Consider testing thoroughly with older nevermore projects, or simply not merging????

OttoHatt avatar Jun 18 '22 23:06 OttoHatt

It may be a good idea to rename the .readonly methods; they provide the nice functionality in erroring on a nil index, but I think it's easy to assume it does the same as table.freeze (which errors on write but not on nil indexing).

OttoHatt avatar Jun 18 '22 23:06 OttoHatt

Closed in favor of #321

Quenty avatar Jan 21 '23 00:01 Quenty