#1648 Retrograde and Retrograde Invert commands broken

[Mark R. Rubin]

Commits [0c975c367f] and [ac2658f839], respectively, broke RetrogradeCommand and RetrogradeInvertCommand (the Notation and Matrix editors' "Adjust -> Convert -> Retrograde" and "Retrograde Invert" menu items). Both commands now delete the current selection instead of performing the desired modifications.

The fixes are easy: Restore the segment.insert(toInsert[j]); calls in each .cpp file. The local jtr variable that was commented out in the commits is unused and unneeded. I can't submit a merge request because my codebase has diverged too far from master.

A future enhancement (option) to keep the notes diatonically in-key while inverting would be nice, but the current chromatic implementations are useful as-is and should be restored to their previous working condition.

[Philip Leishman]

Thanks for finding that !
I seem to have been a little overzealous with removing the unused variable.
See merge request.

[Ted Felix]

Merged as [84faef]. Please test latest git.