vide icon indicating copy to clipboard operation
vide copied to clipboard

Use generalized iteration instead of `next, tbl`

Open jiwonz opened this issue 7 months ago • 2 comments

Vide's source code is currently using next, tbl instead of generalized iteration.

Internally, Luau seems to have almost the same function with generalized iteration and next, tbl, but generalized iteration is probably slightly faster than next, tbl if I remember correctly. And it looks more concise and simple.

jiwonz avatar May 03 '25 07:05 jiwonz

This is something you can PR in yourself, opening an issue for this doesn't really make much sense, there's not enough room for discussion, a PR will either be accepted or denied

TaylorsRus avatar May 03 '25 12:05 TaylorsRus

This is something you can PR in yourself, opening an issue for this doesn't really make much sense, there's not enough room for discussion, a PR will either be accepted or denied

Alright, don't be mad I was really just wondering and wanted to make sure if it's really ok to change them. Also even i was going to PR this, I would open this issue because I usually open before doing PR (Well, didn't get much time to PR this in myself)

jiwonz avatar May 03 '25 18:05 jiwonz