> > On Mon, Sep 26, 2016 at 09:12:41AM +0100, Frediano Ziglio wrote: > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > > --- > > server/red-channel-client-private.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/server/red-channel-client-private.h > > b/server/red-channel-client-private.h > > index 46d4fca..804b39a 100644 > > --- a/server/red-channel-client-private.h > > +++ b/server/red-channel-client-private.h > > @@ -38,6 +38,7 @@ struct RedChannelClientPrivate > > } ack_data; > > > > struct { > > + /* this can be either main.marshaller or urgent.marshaller */ > > If this can also be NULL, I'd mention it explicitly. > > Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > > Christophe > Mumble... no, cannot be NULL, actually marshaller are initialized during construction and freed on destruction. Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel