Re: [PATCH spice-gtk 0/2] Allow to change the preferred compression on the fly

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

 



Hi,

On Tue, Nov 10, 2015 at 05:44:05PM +0100, Pavel Grunt wrote:
> Hi,
>
> this series adds new public function for changing the preferred image compression.
> The new function is used in spicy under Option -> Preferred Image compression
>
> Pavel Grunt (2):
>   channel-display: Add spice_display_change_preferred_compression
>   spicy: Add support for changing compression
>
>  doc/reference/spice-gtk-sections.txt |  1 +
>  src/channel-display.c                | 41 +++++++++++++++++++----
>  src/channel-display.h                |  1 +
>  src/map-file                         |  1 +
>  src/spice-glib-sym-file              |  1 +
>  src/spicy.c                          | 63 ++++++++++++++++++++++++++++++++++++
>  6 files changed, 101 insertions(+), 7 deletions(-)

Both patches looks good and I personally liked them. We already have the
message so having an public API seems good idea, at least for testing
tool like spicy.

Locally, while playing video, processing with quic is much higher then
lz.. I thought it would be oppositve, funny :)

This could be used by client's to better usage of its resources. It
would be good to have some debug messages about what compressions is
being used because as this is 'preferred' it does not mean that it would
switch the compression to it, right?

So, ack by me.
  Victor Toso
_______________________________________________
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]