[Spice-devel] [PATCH 03/17] Move sync_write* to reds_stream.h

Christophe Fergeau cfergeau at redhat.com
Mon Jan 13 03:00:54 PST 2014


    On Sun, Jan 12, 2014 at 01:04:10PM +0200, Alon Levy wrote:
    > On 01/07/2014 01:14 PM, Christophe Fergeau wrote:
    > > They are renamed to reds_stream_write*
    > 
    > You also changed the return type for reds_stream_write_all (was
    > sync_write) to bool, which is now being cast to int in some code paths.

Which code path do you mean? The code paths in this patch are either
ignoring reds_stream_write_* return values or using it as a boolean (with
! or &&).

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20140113/980b201d/attachment.pgp>


More information about the Spice-devel mailing list