Instead of doing this with a weakref, do it by connecting to the destroy

2002-07-16  Peter Williams  <peterw@ximian.com>

	* e-msg-composer.c (create_composer): Instead of doing this with a
	weakref, do it by connecting to the destroy signal. The weakref was
	not getting notified in certain conditions that I could never really
	pin down, and this caused problems at exit.

	* e-msg-composer-hdrs.c (create_headers): Reorder to create reply_to
	before from, because create_from_optionmenu causes from_changed to
	be called, which expects reply_to.entry to exist.

svn path=/trunk/; revision=17480

1 files changed, 10 insertions, 6 deletions

diff --git a/composer/e-msg-composer-hdrs.c b/composer/e-msg-composer-hdrs.c
index a39fd806c6..d98b452cff 100644
--- a/composer/e-msg-composer-hdrs.c
+++ b/composer/e-msg-composer-hdrs.c
@@ -365,13 +365,11 @@ create_headers (EMsgComposerHdrs *hdrs)
 	EMsgComposerHdrsPrivate *priv = hdrs->priv;
 	
 	/*
-	 * From:
-	 */
-	priv->from.label = gtk_label_new (_("From:"));
-	priv->from.entry = create_from_optionmenu (hdrs);
-	
-	/*
 	 * Reply-To:
+	 *
+	 * Create this before we call create_from_optionmenu,
+	 * because that causes from_changed to be called, which
+	 * expects the reply_to fields to be initialized.
 	 */
 	priv->reply_to.label = gtk_label_new (_("Reply-To:"));
 	priv->reply_to.entry = e_entry_new ();
@@ -382,6 +380,12 @@ create_headers (EMsgComposerHdrs *hdrs)
 			NULL);
 	
 	/*
+	 * From:
+	 */
+	priv->from.label = gtk_label_new (_("From:"));
+	priv->from.entry = create_from_optionmenu (hdrs);
+	
+	/*
 	 * Subject:
 	 */
 	priv->subject.label = gtk_label_new (_("Subject:"));