Fixed crash when trying to scroll in an already closed SelectionDialog

src/selectiondialog.cc

@@ -90,7 +90,7 @@ SelectionDialogBase::SelectionDialogBase(Gtk::TextView& text_view, Glib::RefPtr<
 }
 
 void SelectionDialogBase::cursor_changed() {
-  if(!shown)
+  if(!is_visible())
     return;
   auto it=list_view_text.get_selection()->get_selected();
   std::string row;
@@ -112,7 +112,6 @@ void SelectionDialogBase::add_row(const std::string& row) {
 }
 
 void SelectionDialogBase::show() {
-  shown=true;
   window.show_all();
   text_view.grab_focus();
   
@@ -124,6 +123,7 @@ void SelectionDialogBase::show() {
     else if(list_view_text.get_model()->children().begin()!=list_view_text.get_selection()->get_selected()) {
       while(Gtk::Main::events_pending())
         Gtk::Main::iteration(false);
+      if(is_visible())
         list_view_text.scroll_to_row(list_view_text.get_model()->get_path(list_view_text.get_selection()->get_selected()), 0.5);
     }
   }
@@ -138,9 +138,8 @@ void SelectionDialogBase::set_cursor_at_last_row() {
 }
 
 void SelectionDialogBase::hide() {
-  if(!shown)
+  if(!is_visible())
     return;
-  shown=false;
   window.hide();
   if(on_hide)
     on_hide();

src/selectiondialog.h

@@ -37,13 +37,13 @@ public:
   void set_cursor_at_last_row();
   void show();
   void hide();
+  bool is_visible() {return window.is_visible();}
   
   std::function<void()> on_hide;
   std::function<void(const std::string& selected, bool hide_window)> on_select;
   std::function<void(const std::string &selected)> on_changed;
   Glib::RefPtr<Gtk::TextBuffer::Mark> start_mark;
   
-  bool shown=false;
 protected:
   void cursor_changed();
   

src/source.cc

@@ -1244,11 +1244,11 @@ std::string Source::View::get_token(Gtk::TextIter iter) {
 }
 
 bool Source::View::on_key_press_event(GdkEventKey* key) {
-  if(selection_dialog && selection_dialog->shown) {
+  if(selection_dialog && selection_dialog->is_visible()) {
     if(selection_dialog->on_key_press(key))
       return true;
   }
-  if(autocomplete_dialog && autocomplete_dialog->shown) {
+  if(autocomplete_dialog && autocomplete_dialog->is_visible()) {
     if(autocomplete_dialog->on_key_press(key))
       return true;
   }

src/source_clang.cc

@@ -568,7 +568,7 @@ void Source::ClangViewParse::show_type_tooltips(const Gdk::Rectangle &rectangle)
 Source::ClangViewAutocomplete::ClangViewAutocomplete(const boost::filesystem::path &file_path, Glib::RefPtr<Gsv::Language> language):
     Source::ClangViewParse(file_path, language), autocomplete_state(AutocompleteState::IDLE) {
   get_buffer()->signal_changed().connect([this](){
-    if(autocomplete_dialog && autocomplete_dialog->shown)
+    if(autocomplete_dialog && autocomplete_dialog->is_visible())
       delayed_reparse_connection.disconnect();
     else {
       if(!has_focus())
@@ -596,7 +596,7 @@ Source::ClangViewAutocomplete::ClangViewAutocomplete(const boost::filesystem::pa
   });
   
   signal_key_release_event().connect([this](GdkEventKey* key){
-    if(autocomplete_dialog && autocomplete_dialog->shown) {
+    if(autocomplete_dialog && autocomplete_dialog->is_visible()) {
       if(autocomplete_dialog->on_key_release(key))
         return true;
     }

src/source_spellcheck.cc

@@ -27,7 +27,7 @@ Source::SpellCheckView::SpellCheckView() : Gsv::View() {
   spellcheck_error_tag->property_underline()=Pango::Underline::UNDERLINE_ERROR;
   
   signal_key_press_event().connect([this](GdkEventKey *event) {
-    if(spellcheck_suggestions_dialog && spellcheck_suggestions_dialog->shown) {
+    if(spellcheck_suggestions_dialog && spellcheck_suggestions_dialog->is_visible()) {
       if(spellcheck_suggestions_dialog->on_key_press(event))
         return true;
     }