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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Attachment: signature.asc
Description: PGP signature

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

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]