@@ -634,10 +634,16 @@ int Lines::UpdateSelection(unsigned int sel_id, unsigned int start, unsigned int } void Lines::RemoveSelection(unsigned int sel_id) { + // FS#393 0 Quickly pressing and releasing CTRL and ALT while making a + // selection can cause this function to be called in a way that violates this + // assertion. In that case, we don't want to blow up here. + if (selections.empty()) return; + wxASSERT(!selections.empty() && sel_id < selections.size()); selections.erase(selections.begin()+sel_id); - if (m_lastSel == (int)sel_id) m_lastSel = -1; + if (m_lastSel == (int)sel_id) + m_lastSel = -1; } void Lines::RemoveAllSelections(bool checkShadow, unsigned int pos) { src/Lines.cpp c60c9a617164aa45efb06687b8e110d910d5d79e adamv flangy@gmail.com http://github.com/etexteditor/e/commit/7fa8d8064d18a15d9da4b1289d33c7deeea1cb29 7fa8d8064d18a15d9da4b1289d33c7deeea1cb29 2009-11-02T16:09:37-08:00 2009-11-02T16:06:57-08:00 FS#393 - Don't modify the selection if none exists. 1bbcc38ceda9bf5fcbd009b5c2c7e6341449a875 adamv flangy@gmail.com