Make sure QRegister::view is properly initialized and cleaned up

 * it must be initialized after the UI (Interface::main), so I added
   a View::initialize() function
 * the old initialize() method was renamed to setup()
 * use a global instance of QRegister::view so it is guaranteed to
   be destroyed only after any QRegisters that could still need it
 * Document API adapted to work with ViewCurrent references

1 files changed, 15 insertions, 2 deletions

diff --git a/src/interface-gtk.h b/src/interface-gtk.h
index 5a083ab..6c8e055 100644
--- a/src/interface-gtk.h
+++ b/src/interface-gtk.h
@@ -34,8 +34,21 @@ typedef class ViewGtk : public View<ViewGtk> {
 	ScintillaObject *sci;
 
 public:
-	ViewGtk();
-	~ViewGtk();
+	ViewGtk() : sci(NULL) {}
+
+	/* implementation of View::initialize() */
+	void initialize_impl(void);
+
+	inline ~ViewGtk()
+	{
+		/*
+		 * This does NOT destroy the Scintilla object
+		 * and GTK widget, if it is the current view
+		 * (and therefore added to the vbox).
+		 */
+		if (sci)
+			g_object_unref(G_OBJECT(sci));
+	}
 
 	inline GtkWidget *
 	get_widget(void)