Return non-zero exit status if reading the input mp3 file failed.
I prepared a patch which solves this issue. See attachment.
lame produces invalid result while exit status is zero
Multi-line messages are displayed as single line
Does a low value for lastXChosen affect any behaviour? It probably does not affects stock Scintilla. It should just result in vertical key movement hugging the left side just as it would with an accurate value. I am experimenting with "wobble-free" vertical caret movement (it highly depends on lastXChosen) and noticed that lastXChosen is not correct at the very beginning. I thought is is better to make it correct from the beginning rather than check "is this correct?" every time. This looks wrong...
Scintilla: Editor::lastXChosen has incorrect initilal value
Case of member functions fixed. Selections renamed to SelectionKeeper. If you do not like this name please be more specific and let me know exact name. Dropping n and using directly selections.size() causes warning "signed/unsigned comparison". Range loop used to avoid warnings. Constructor made explicit. Conversions renamed to PosFromLoc and LocFromPos. Comment cut.
Case of member functions fixed. Selections renamed to SelectionKeeper. If you do not like this name please be more specific and let me know exact name. Dropping n and using directly selections.size() causes warning "signed/unsigned comparison". Range loop used to avoid warnings. Constructor made explicit. Conversions renamed to PosFromLoc and LocFromPos. Comment cut.