From: Rob Clark Date: Wed, 10 Mar 2010 16:26:27 +0000 (-0600) Subject: mpeg4videoparse: fix compile error X-Git-Tag: 1.19.3~507^2~17349 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=79faa0b9fa46b03da46e0739093b8a9c3bc372c2;p=platform%2Fupstream%2Fgstreamer.git mpeg4videoparse: fix compile error gst/mpeg4videoparse/mpeg4videoparse.c:689: warning: comparison is always true due to limited range of data type https://bugzilla.gnome.org/show_bug.cgi?id=612454 --- diff --git a/gst/mpeg4videoparse/mpeg4videoparse.c b/gst/mpeg4videoparse/mpeg4videoparse.c index aaca907..ceafd6f 100644 --- a/gst/mpeg4videoparse/mpeg4videoparse.c +++ b/gst/mpeg4videoparse/mpeg4videoparse.c @@ -686,8 +686,9 @@ gst_mpeg4vparse_sink_setcaps (GstPad * pad, GstCaps * caps) /* Usually the codec data will be a visual object sequence, containing a visual object, with a video object/video object layer. */ res = gst_mpeg4vparse_handle_vos (parse, data, GST_BUFFER_SIZE (buf)); - } else if (data[3] >= VIDEO_OBJECT_STARTCODE_MIN && - data[3] <= VIDEO_OBJECT_STARTCODE_MAX) { + } else if (data[3] <= VIDEO_OBJECT_STARTCODE_MAX) { + /* VIDEO_OBJECT_STARTCODE_MIN is zero, and data is unsigned, so we + don't need to check min (and in fact that causes a compile err */ /* Sometimes, instead, it'll just have the video object/video object layer data. We can parse that too, though it'll give us slightly less information. */