[Spice-devel] [Spice-commits] 3 commits - client/cmd_line_parser.cpp client/cmd_line_parser.h client/windows client/x11 server/mjpeg_encoder.c

[Christophe Fergeau]

On Tue, Aug 02, 2011 at 02:15:34AM -0700, Christophe Fergau wrote:
> New commits:
> commit 3582adb989cdb6e1e75bf9341ffcebf35e58b737
> Author: Christophe Fergeau <cfergeau at redhat.com>
> Date:   Fri Jul 29 18:52:15 2011 +0200
> 
>     mjpeg: add missing SPICE_BITMAP_FMT_RGBA
>     
>     I forgot to handle SPICE_BITMAP_FMT_RGBA when mapping from
>     spice image formats to libjpeg-turbo colorspaces.
> 
> diff --git a/server/mjpeg_encoder.c b/server/mjpeg_encoder.c
> index ea2e15f..4b1023a 100644
> --- a/server/mjpeg_encoder.c
> +++ b/server/mjpeg_encoder.c
> @@ -207,6 +207,7 @@ int mjpeg_encoder_start_frame(MJpegEncoder *encoder, SpiceBitmapFmt format,
>      encoder->cinfo.input_components = 3;
>      switch (format) {
>      case SPICE_BITMAP_FMT_32BIT:
> +    case SPICE_BITMAP_FMT_RGBA:
>          encoder->bytes_per_pixel = 4;
>  #ifdef JCS_EXTENSIONS
>          encoder->cinfo.in_color_space   = JCS_EXT_BGRX;

Hrm, once again I didn't mean to push this without review, let me know if
there are issues with it, I can revert.

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