Skip to content

refactor: use for...of for better readability#3472

Closed
noritaka1166 wants to merge 1 commit intobrianc:masterfrom
noritaka1166:refactor-code
Closed

refactor: use for...of for better readability#3472
noritaka1166 wants to merge 1 commit intobrianc:masterfrom
noritaka1166:refactor-code

Conversation

@noritaka1166
Copy link
Copy Markdown
Contributor

use for...of for better readability

@brianc
Copy link
Copy Markdown
Owner

brianc commented Jun 5, 2025

I appreciate the thought here but I'm going to stick with for i loops in the code where they exist. Last I checked they still have a performance improvement/advantage over a for of loop. While none of these code paths are seriously white-hot execution frequency I still don't mind using for i loops and so I'm just going to pass on this one. Thank you though!

@brianc brianc closed this Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants