From: nljlistbox2@gmail.com (N. Jackson)
To: Eli Zaretskii <eliz@gnu.org>
Cc: 25851@debbugs.gnu.org
Subject: bug#25851: 25.2; GTK warning when starting Emacs when desktop file has more than one frame
Date: Sun, 26 Feb 2017 17:41:22 -0500 [thread overview]
Message-ID: <87wpccmv71.fsf@moondust.localdomain> (raw)
In-Reply-To: <83fuj2smzr.fsf@gnu.org> (Eli Zaretskii's message of "Sat, 25 Feb 2017 10:17:28 +0200")
At 10:17 +0200 on Saturday 2017-02-25, Eli Zaretskii wrote:
>
> So we are back at square one: we need to understand why the warning
> isn't get silenced by this:
>
> /* Silence warning about visible children. */
> id = g_log_set_handler ("Gtk", G_LOG_LEVEL_WARNING | G_LOG_FLAG_FATAL
> | G_LOG_FLAG_RECURSION, my_log_handler, NULL);
>
> Can you look into the source of g_warning and see why the above
> doesn't avoid these warnings, and what should we do to avoid it?
Hello Eli,
I was unsuccessful in this assignment (so far at least). I suspect
the answer is that GTK is using structured logging, but that is
just a wild guess at this point.
[I don't know how to do source-level debugging of code that I
didn't build myself. Both glib and Gtk+ on my system were
built/installed by the distribution's package manager and I have
only the source code from upstream, not the source code for the
exact versions on my system.]
I did ascertain that the log handler `my_log_handler' that we
install in the code snippet above is never called.
For testing purposes, I also installed a catch-all handler, with
g_log_set_default_handler(), and that doesn't get called either.
I have no direct experience at all with either glib or Gtk+, but
the documentation at [1] [I don't know for what version this is
for, or if it's even relevant], seems to say that
g_log_set_handler() has no effect if structured logging is turned
on. It seems to say that structured logging is in effect if
G_LOG_USE_STRUCTURED is defined.
There is some indication that Gtk+3.22.8 might use the new
structured logging. For example:
$ grep -rn "G_LOG_USE_STRUCTURED" /data/projects/vc/gtk+-3.22.8/
/data/projects/vc/gtk+-3.22.8/gdk/mir/Makefile.in:526: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/mir/Makefile.am:9: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/Makefile.in:697: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/win32/Makefile.in:602: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/win32/Makefile.am:9: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/wayland/Makefile.in:537: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/wayland/Makefile.am:9: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/Makefile.am:38: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/x11/Makefile.in:535: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/x11/Makefile.am:9: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/quartz/Makefile.in:531: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/quartz/Makefile.am:8: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/broadway/Makefile.in:574: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gdk/broadway/Makefile.am:10: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gtk/Makefile.in:1325: -DG_LOG_USE_STRUCTURED=1 \
/data/projects/vc/gtk+-3.22.8/gtk/Makefile.am:7: -DG_LOG_USE_STRUCTURED=1 \
...
I don't have any more direct evidence at this stage. I might press
on with this enquiry, but I admit I am way out of my depth at this
stage.
N.
next prev parent reply other threads:[~2017-02-26 22:41 UTC|newest]
Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-23 16:08 bug#25851: 25.2; GTK warning when starting Emacs when desktop file has more than one frame N. Jackson
2017-02-23 16:24 ` Eli Zaretskii
2017-02-24 2:33 ` N. Jackson
2017-02-24 8:07 ` Eli Zaretskii
2017-02-24 13:41 ` N. Jackson
2017-02-24 13:53 ` N. Jackson
2017-02-24 14:10 ` Eli Zaretskii
2017-02-24 16:09 ` N. Jackson
2017-02-24 20:28 ` N. Jackson
2017-02-25 8:17 ` Eli Zaretskii
2017-02-26 22:41 ` N. Jackson [this message]
2017-02-27 0:31 ` N. Jackson
2017-02-27 16:18 ` Eli Zaretskii
2017-02-27 18:26 ` N. Jackson
2017-02-27 18:37 ` Eli Zaretskii
2017-02-28 9:46 ` martin rudalics
2017-03-01 20:05 ` N. Jackson
2017-03-23 8:00 ` martin rudalics
2017-03-23 14:11 ` N. Jackson
2017-03-24 9:01 ` martin rudalics
2017-03-24 20:28 ` N. Jackson
2017-03-25 6:26 ` Eli Zaretskii
2017-03-28 13:15 ` N. Jackson
2017-03-29 7:36 ` martin rudalics
2017-03-25 9:25 ` martin rudalics
2017-04-27 19:28 ` N. Jackson
2017-04-11 6:49 ` martin rudalics
2017-04-27 19:55 ` N. Jackson
2017-04-29 10:30 ` martin rudalics
2017-04-29 19:32 ` N. Jackson
2017-04-30 8:32 ` martin rudalics
2017-04-30 16:13 ` N. Jackson
2017-04-30 19:36 ` martin rudalics
2017-02-28 9:46 ` martin rudalics
2017-02-28 15:51 ` Eli Zaretskii
2017-02-28 18:42 ` martin rudalics
2017-02-28 18:50 ` Eli Zaretskii
2017-03-01 8:29 ` martin rudalics
2017-03-01 16:18 ` Eli Zaretskii
2017-03-01 19:36 ` martin rudalics
2017-03-01 19:47 ` Eli Zaretskii
2017-03-01 20:11 ` Eli Zaretskii
2017-03-02 11:00 ` martin rudalics
2017-03-02 15:09 ` Eli Zaretskii
2017-03-02 17:57 ` martin rudalics
2017-03-02 20:10 ` Eli Zaretskii
2017-03-03 8:13 ` martin rudalics
2017-03-03 8:25 ` Eli Zaretskii
2017-03-01 20:16 ` N. Jackson
2017-03-03 8:13 ` martin rudalics
2017-03-03 13:05 ` N. Jackson
2017-03-03 14:24 ` martin rudalics
2017-03-06 18:25 ` N. Jackson
2017-03-06 18:44 ` martin rudalics
2017-02-25 8:21 ` martin rudalics
2017-02-26 22:47 ` N. Jackson
2017-02-27 2:22 ` N. Jackson
2017-02-27 8:04 ` martin rudalics
2017-02-27 17:56 ` N. Jackson
2017-02-28 9:46 ` martin rudalics
2017-02-25 7:55 ` Eli Zaretskii
2017-02-26 22:09 ` N. Jackson
2017-03-23 7:59 ` martin rudalics
2017-03-23 13:47 ` Drew Adams
2017-03-23 14:34 ` N. Jackson
2017-03-24 9:01 ` martin rudalics
2017-03-24 20:37 ` N. Jackson
2017-03-25 9:25 ` martin rudalics
2017-03-23 15:24 ` Eli Zaretskii
2017-03-24 9:02 ` martin rudalics
2017-04-27 19:45 ` N. Jackson
2017-04-27 19:52 ` Noam Postavsky
2017-04-28 14:15 ` N. Jackson
2017-04-28 14:25 ` N. Jackson
2017-09-25 16:31 ` N. Jackson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87wpccmv71.fsf@moondust.localdomain \
--to=nljlistbox2@gmail.com \
--cc=25851@debbugs.gnu.org \
--cc=eliz@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).