Duplicate/nested for loop

[ronyeh]

vexflow/src/stringnumber.ts

Lines 52 to 58 in faf894c

	for (i = 0; i < nums.length; ++i) {
	num = nums[i];
	note = num.getNote();
	for (i = 0; i < nums.length; ++i) {
	num = nums[i];
	note = num.getNote();

It's late and I'm seeing double. :-D

While working on my PR I ran across these lines which stood out to me and just look wrong.

I believe the outer for loop can just be removed, correct? Since they both iterate on index i, by the time the inner for loop finishes, the outer one will also finish.

[sschmidTU]

yeah, looks like one of the loops can be removed, they do the same thing.
It definitely doesn't make sense to have nested loops on the same index variable (i).

[ronyeh]

Thanks for the sanity check @sschmidTU . I added a PR with the fix: #1014