On Thu, 11 Oct 2012, Mans Rullgard wrote:

parse_packed_headers() returns either zero or a negative error code
so its return type must be signed.

Signed-off-by: Mans Rullgard <m...@mansr.com>
---
libavformat/rtpdec_xiph.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/rtpdec_xiph.c b/libavformat/rtpdec_xiph.c
index dc739ee..38f12bb 100644
--- a/libavformat/rtpdec_xiph.c
+++ b/libavformat/rtpdec_xiph.c
@@ -243,7 +243,7 @@ static int get_base128(const uint8_t ** buf, const uint8_t 
* buf_end)
/**
 * Based off parse_packed_headers in Vorbis RTP
 */
-static unsigned int
+static int
parse_packed_headers(const uint8_t * packed_headers,
                     const uint8_t * packed_headers_end,
                     AVCodecContext * codec, PayloadContext * xiph_data)
--
1.7.12

LGTM

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to