[Spice-devel] [spice-server 3/4] reds-stream: Use true/false instead of TRUE/FALSE

[Christophe Fergeau]

On Thu, Mar 09, 2017 at 09:15:53AM -0500, Frediano Ziglio wrote:
> > 
> > ---
> >  server/reds-stream.c | 30 +++++++++++++++---------------
> >  1 file changed, 15 insertions(+), 15 deletions(-)
> > 
> > diff --git a/server/reds-stream.c b/server/reds-stream.c
> > index 096ed43..50a8584 100644
> > --- a/server/reds-stream.c
> > +++ b/server/reds-stream.c
> > @@ -202,12 +202,12 @@ bool reds_stream_write_all(RedsStream *stream, const
> > void *in_buf, size_t n)
> >              if (now == -1 && (errno == EINTR || errno == EAGAIN)) {
> >                  continue;
> >              }
> > -            return FALSE;
> > +            return false;
> >          }
> >          n -= now;
> >          buf += now;
> >      }
> > -    return TRUE;
> > +    return true;
> >  }
> >  
> >  #if HAVE_SASL
> > @@ -240,19 +240,19 @@ int reds_stream_get_family(const RedsStream *s)
> >  
> >  bool reds_stream_is_plain_unix(const RedsStream *s)
> >  {
> > -    spice_return_val_if_fail(s != NULL, FALSE);
> > +    spice_return_val_if_fail(s != NULL, false);
> >  
> >      if (reds_stream_get_family(s) != AF_UNIX)
> > -        return FALSE;
> > +        return false;
> 
> Should we fix the missing brackets too?

I've fixed the locations which were touched by this patch, and pushed
the first 3 patches. Thanks for the rebase/small adjustements to the
s/int/bool/ patch.

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/20170309/c088e3ec/attachment-0001.sig>