Re: Testing network availability

[Peter Bloomfield <PeterBloomfield bellsouth net>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Albrecht:

On 05/26/2017 08:14:33 AM Fri, Albrecht Dreß wrote:
> Hi Peter:
…
> Please find attached ver. 2 of the patch, fixing the issues above.
>
> It basically removes the smtp server from SendQueueInfo, but adds it to SendMessageInfo (instead of just its 
> name), so (un) ref'ing it is handled in send_message_info_new() and send_message_info_destroy().

This is all working well!

…except for one occasional glitch:

(balsa:23011): Gtk-CRITICAL **: gtk_window_set_transient_for: assertion 'parent == NULL || GTK_IS_WINDOW 
(parent)' failed
Gtk-Message: GtkDialog mapped without a transient parent. This is discouraged.

The offending call is in ensure_send_progress_dialog, passing "parent" as the transient parent for the 
send-progress dialog, which is in turn called from lbs_process_queue_real, passing SendQueueInfo::parent. Gdb shows that 
send_info->parent is not balsa_app.main_window, which is supposed to be passed down from send_queued_mail_activated. 
It's also not been NULL, in the cases I've looked at.

The other two members of send_info do not seem to have been corrupted; outbox is still my outbox, and finder 
is still balsa_find_sentbox_by_url, So somehow just that third member is getting over-written, and I don't 
know how to detect that.

Thoughts, anyone?

Peter
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAlkphtkACgkQH1/UtbkqdPXWewCfeOX4CWQWjiQpM5zP8J6JpLKP
Js8AoIsvJlFuvvqU4tpETjylVHdPO1kB
=N7Jn
-----END PGP SIGNATURE-----