No worries, thank you for the work and the fixes!
Ok, this looks good to me now. I must admit I haven't checked the values in the arrays, but I trust you :)
Also, before you merge into trunk, just two things:
1) Can you add an empty line after this line:
18 + [6, 43, 15, 2, 44, 16],
This way the comment is separated from the previous array as in the rest of the file.
2) Can you add a brief note on the commit message about how you calculated the values in the arrays?
Thanks!
« Back to merge proposal
No worries, thank you for the work and the fixes!
Ok, this looks good to me now. I must admit I haven't checked the values in the arrays, but I trust you :)
Also, before you merge into trunk, just two things:
1) Can you add an empty line after this line:
18 + [6, 43, 15, 2, 44, 16],
This way the comment is separated from the previous array as in the rest of the file.
2) Can you add a brief note on the commit message about how you calculated the values in the arrays?
Thanks!