On Wed, 2010-06-23 at 16:32 +0300, Yonit Halperin wrote:
> fixes a crash in qxl_soft_reset
> ---

I'm commiting this as in. I agree in sprit with gerds comment about less
duplication, but I don't think doing it at this time for this particular
function is a good idea. Instead we need to take a step back and look at
really refactoring out the channel code, into a separate file. Yonits
red_channel.c might be a good start. With the SpiceMarshaller stuff
taking some of the work, making the display channel a bit less special
maybe this is easier now even.

-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
       [email protected]            [email protected] 
He's a gun-slinging flyboy inventor haunted by memories of 'Nam. She's a 
plucky punk nun with someone else's memories. They fight crime! 

_______________________________________________
Spice-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to