[Spice-devel] [PATCH spice-server 3/4] reds-stream: Introduce reds_stream_set_no_delay() helper

[Christophe Fergeau]

On Wed, Feb 15, 2017 at 09:14:10AM -0500, Frediano Ziglio wrote:
> > 
> > On Wed, Feb 15, 2017 at 06:39:13AM -0500, Frediano Ziglio wrote:
> > > 
> > > Patch is good however the use of bool is inconsistent.
> > > Although I _much_ prefer bool/true/false we decided for
> > > gboolean/TRUE/FALSE.
> > 
> > reds-stream.[ch] is already exclusively using bool instead of gboolean,
> > so I'd prefer to stick with this. I have a patch (series) in preparation
> > changing gboolean to bool (with justification), so I would not change
> > reds-stream.[ch] just yet.
> > 
> > Christophe
> > 
> 
> Can you be at least consistent in server/reds-stream.c ?
> 
> bool -> true/false
> 

Existing code already uses bool together with TRUE/FALSE, so I would
keep the discussion about TRUE vs true for another time as well ;)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170215/0e1d8dc2/attachment.sig>