Hey, Series grew a few more patches since the first iteration after the review comments, the most notable change would be the addition of "channel: Remove RedChannel::handle_parsed" which addresses the issue with red_channel_client_parse() return value which Jonathon pointed out. This changes the parser/handle_parsed/handle_message logic in order to avoid the issue. Apart from this, the additional patches are meant to address some of the review comments Changes since v1: - Added patch "channel: Remove RedChannel::handle_parsed" - Removed confusing FIXMEs - Renamed methods when removing OutgoingHandlerInterface/IncomingHandlerInterface - Removed unneeded check from red_channel_client_alloc_msg_buf/red_channel_client_release_msg_buf - Removed some vfunc typedefs as they become unneeded - Added patches "Remove red-channel-client-private.h" "red-channel-client: Rename {Outgoing,Incoming}Handler" "red-channel-client: Consistently name RedChannelClient 'rcc'" Christophe _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel