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