This pull request removes all warnings from zz when compiling. It does not change any logic in the code.
It also makes sure errors cannot be introduced. For example, value was renamed to _value because it was unused. But some commented out code refers to it as "value", which is defined in the previous for loop. By putting this loop into its ownscope, we ensure that anybody re-introducing this code would have to make sure to rename _value to value back.
aep
commented
3 years ago
Hi,
This pull request removes all warnings from zz when compiling. It does not change any logic in the code.
It also makes sure errors cannot be introduced. For example,
valuewas renamed to_valuebecause it was unused. But some commented out code refers to it as "value", which is defined in the previous for loop. By putting this loop into its ownscope, we ensure that anybody re-introducing this code would have to make sure to rename_valuetovalueback.