Hi,

On Mon, Mar 06, 2017 at 11:26:15AM +0100, Victor Toso wrote:
> Hi,
> 
> On Mon, Mar 06, 2017 at 11:15:58AM +0100, Pavel Grunt wrote:
> > ACK
> >
> > Thanks,
> > Pavel
> > 
> > I think some of these setting properties are deprecated
> 
> Oh, true. The second one here is deprecated and ignored from gtk v3.10
> onwards. I'll send a followup patch.

Ah, i missread the docs. It isn't being ignored now but it will be
ignored in the future.

  "This setting can still be used for application overrides, but will be
   ignored in the future"

I'll keep the patch as it is then.

Thanks!
> 
> >
> > On Mon, 2017-03-06 at 11:09 +0100, Victor Toso wrote:
> > > From: Victor Toso <m...@victortoso.com>
> > > 
> > > As gboolean is 4 bytes long while bool is 1 byte, valgrind will
> > > complain
> > > about it.
> > > 
> > > Invalid write of size 4
> > >    at 0xAE9F4FB: value_lcopy_boolean (gvaluetypes.c:78)
> > >    by 0xAE7CFD3: g_object_get_valist (gobject.c:2236)
> > >    by 0xAE7D40B: g_object_get (gobject.c:2324)
> > >    by 0x40758F: keyboard_grab_cb (spicy.c:566)
> > >    by 0xAE753E4: g_closure_invoke (gclosure.c:804)
> > >    by 0xAE87431: signal_emit_unlocked_R (gsignal.c:3635)
> > >    by 0xAE9005E: g_signal_emit_valist (gsignal.c:3391)
> > >    by 0xAE9043E: g_signal_emit (gsignal.c:3447)
> > >    by 0x4E4A6FB: try_keyboard_grab (spice-widget.c:889)
> > >    by 0x4E4CAB1: focus_in_event (spice-widget.c:1834)
> > >    by 0x9325E70: _gtk_marshal_BOOLEAN__BOXEDv (gtkmarshalers.c:131)
> > >    by 0xAE75561: _g_closure_invoke_va (gclosure.c:867)
> > > 
> > > Signed-off-by: Victor Toso <victort...@redhat.com>
> > > ---
> > >  tools/spicy.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/tools/spicy.c b/tools/spicy.c
> > > index cd1677f..2772cfa 100644
> > > --- a/tools/spicy.c
> > > +++ b/tools/spicy.c
> > > @@ -75,8 +75,8 @@ struct _SpiceWindow {
> > >      gint             win_x;
> > >      gint             win_y;
> > >  #endif
> > > -    bool             enable_accels_save;
> > > -    bool             enable_mnemonics_save;
> > > +    gboolean         enable_accels_save;
> > > +    gboolean         enable_mnemonics_save;
> > >  };
> > >  
> > >  struct _SpiceWindowClass


Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to