Re: [Spice-devel] [spice-gtk v1 5/7] gtk-session: move atom's debug

2017-02-23 Thread Victor Toso
Hi, On Thu, Feb 23, 2017 at 02:19:31PM +0100, Pavel Grunt wrote: > On Thu, 2017-02-23 at 14:15 +0100, Pavel Grunt wrote: > > On Wed, 2017-02-22 at 14:11 +0100, Victor Toso wrote: > > > From: Victor Toso > > > > > > We already iterate in all n_atoms and get its name with > > > gdk_atom_name(), le

Re: [Spice-devel] [spice-gtk v1 5/7] gtk-session: move atom's debug

2017-02-23 Thread Pavel Grunt
On Thu, 2017-02-23 at 14:15 +0100, Pavel Grunt wrote: > On Wed, 2017-02-22 at 14:11 +0100, Victor Toso wrote: > > From: Victor Toso > > > > We already iterate in all n_atoms and get its name with > > gdk_atom_name(), let's move the debug there too; > > > > As spice_util_get_debug() uses g_once()

Re: [Spice-devel] [spice-gtk v1 5/7] gtk-session: move atom's debug

2017-02-23 Thread Pavel Grunt
On Wed, 2017-02-22 at 14:11 +0100, Victor Toso wrote: > From: Victor Toso > > We already iterate in all n_atoms and get its name with > gdk_atom_name(), let's move the debug there too; > > As spice_util_get_debug() uses g_once(), this should not impact > performance or might improve it with one

[Spice-devel] [spice-gtk v1 5/7] gtk-session: move atom's debug

2017-02-22 Thread Victor Toso
From: Victor Toso We already iterate in all n_atoms and get its name with gdk_atom_name(), let's move the debug there too; As spice_util_get_debug() uses g_once(), this should not impact performance or might improve it with one memory allocation less. While at it, move the generic SPICE_DEBUG o