Re: glz cleanup

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

 



Hi

----- Original Message -----
> Hi,
> 
>  Hopefully this is not something someone discussed in the 193 unread
> mails from spice-devel I have.
> 
>  I'm trying to look more closely at the glz code, and it seems to be
> split: decoder in glz, encoder in spice.
> 
>  I propose to move them to spice-common. any objections?
> 
>  The reason I propose and don't just send patches is that while moving
> the server code was trivial moving the gtk code brings some baggage with
> it: gio-coroutine and coroutine_{winfibers,gthread,ucontext} plus
> spice_util (the later I may split to leave some gtk only util in
> spice-gtk). Any objects to moving the coroutine stuff to spice-common?

That sounds like a good idea. I'd prefer to move only glz-decoder though. 
Regarding the coroutine code, I would add a decoder callback for the call:

    if (!g_coroutine_condition_wait(g_coroutine_self(), wait_for_image, &data))
->
    if (!w->wait(wait_for_image, &data))

I'd replace all the SPICE_DEBUG() with spice_debug() or g_debug() 
> Thanks,
> Alon
> p.s. g_debug related cleanups are also due (btw the whole split seems to
> be because when spice-gtk was written spice still did not depend on
> glib. it does now. spice-common doesn't but it is trivial to make it so,
> and useful).
> p.p.s. I don't think any of this has been done in the replay branch
> (which we should merge..) so it shouldn't conflict much (just some
> include changes - glz_bla.h -> common/glz_bla.h)
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]