[Spice-devel] [PATCH spice-server v3 4/5] gstreamer: Do not warn for tested formats

Frediano Ziglio fziglio at redhat.com
Wed Nov 9 10:19:13 UTC 2016


These formats were tested manually using gst-test utility
in server/tests.

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 server/gstreamer-encoder.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Changes since v2:
- extended commit message.

diff --git a/server/gstreamer-encoder.c b/server/gstreamer-encoder.c
index 54e1b2e..27872cf 100644
--- a/server/gstreamer-encoder.c
+++ b/server/gstreamer-encoder.c
@@ -758,7 +758,7 @@ static const SpiceFormatForGStreamer format_map[] =  {
     {SPICE_BITMAP_FMT_INVALID, "", 0, 0, 0, 0, 0, 0},
     {SPICE_BITMAP_FMT_RGBA, "BGRA", 32, 24, 4321, 0xff000000, 0xff0000, 0xff00},
     {SPICE_BITMAP_FMT_16BIT, "RGB15", 16, 15, 4321, 0x001f, 0x03E0, 0x7C00},
-    /* TODO: Test the other formats */
+    /* TODO: Test the other formats under GStreamer 0.10*/
     {SPICE_BITMAP_FMT_32BIT, "BGRx", 32, 24, 4321, 0xff000000, 0xff0000, 0xff00},
     {SPICE_BITMAP_FMT_24BIT, "BGR", 24, 24, 4321, 0xff0000, 0xff00, 0xff},
 };
@@ -770,9 +770,11 @@ static const SpiceFormatForGStreamer *map_format(SpiceBitmapFmt format)
     int i;
     for (i = 0; i < G_N_ELEMENTS(format_map); i++) {
         if (format_map[i].spice_format == format) {
+#ifdef HAVE_GSTREAMER_0_10
             if (i > 2) {
                 spice_warning("The %d format has not been tested yet", format);
             }
+#endif
             return &format_map[i];
         }
     }
-- 
2.7.4



More information about the Spice-devel mailing list