It would be nice to add an option to check if the for loop's expression is already set to pairs or ipairs (although using next would be better, but still), and converting this is not necessary and may decrease performance.
If there is no interface to change the settings of the modifier, it might not be a bad idea to add it as a new modifier.
To check if a valid pairs or ipairs is set, you could check the block similar to the bit32 change to see if pairs or ipairs is set to something else.
It would be nice to add an option to check if the
forloop's expression is already set topairsoripairs(although using next would be better, but still), and converting this is not necessary and may decrease performance.If there is no interface to change the settings of the modifier, it might not be a bad idea to add it as a new modifier.
To check if a valid
pairsoripairsis set, you could check the block similar to the bit32 change to see if pairs or ipairs is set to something else.