From b7a4c319fda22aa91ce29692d728ec6103b514f6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Mon, 23 Feb 2015 21:35:48 +0200 Subject: rtpdec: Allow setting the need_parsing field in RTPDynamicProtocolHandler MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This allows getting rid of quite a bit of boilerplate in depacketizers. The default value (initializing need_parsing to 0, aka AVSTREAM_PARSE_NONE) is the same as it is initialized to by default in AVStream. Signed-off-by: Martin Storsjö --- libavformat/rtpdec.h | 1 + libavformat/rtsp.c | 9 ++++++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/libavformat/rtpdec.h b/libavformat/rtpdec.h index 9f1a475ba0..e015942c90 100644 --- a/libavformat/rtpdec.h +++ b/libavformat/rtpdec.h @@ -116,6 +116,7 @@ struct RTPDynamicProtocolHandler { const char *enc_name; enum AVMediaType codec_type; enum AVCodecID codec_id; + enum AVStreamParseType need_parsing; int static_payload_id; /* 0 means no payload id is set. 0 is a valid * payload ID (PCMU), too, but that format doesn't * require any custom depacketization code. */ diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index a3d6814bab..72e5b5b75c 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -176,13 +176,16 @@ static int get_sockaddr(const char *buf, struct sockaddr_storage *sock) #if CONFIG_RTPDEC static void init_rtp_handler(RTPDynamicProtocolHandler *handler, - RTSPStream *rtsp_st, AVCodecContext *codec) + RTSPStream *rtsp_st, AVStream *st) { + AVCodecContext *codec = st ? st->codec : NULL; if (!handler) return; if (codec) codec->codec_id = handler->codec_id; rtsp_st->dynamic_handler = handler; + if (st) + st->need_parsing = handler->need_parsing; if (handler->alloc) { rtsp_st->dynamic_protocol_context = handler->alloc(); if (!rtsp_st->dynamic_protocol_context) @@ -215,7 +218,7 @@ static int sdp_parse_rtpmap(AVFormatContext *s, if (codec->codec_id == AV_CODEC_ID_NONE) { RTPDynamicProtocolHandler *handler = ff_rtp_handler_find_by_name(buf, codec->codec_type); - init_rtp_handler(handler, rtsp_st, codec); + init_rtp_handler(handler, rtsp_st, st); /* If no dynamic handler was found, check with the list of standard * allocated types, if such a stream for some reason happens to * use a private payload type. This isn't handled in rtpdec.c, since @@ -465,7 +468,7 @@ static void sdp_parse_line(AVFormatContext *s, SDPParseState *s1, /* Even static payload types may need a custom depacketizer */ handler = ff_rtp_handler_find_by_id( rtsp_st->sdp_payload_type, st->codec->codec_type); - init_rtp_handler(handler, rtsp_st, st->codec); + init_rtp_handler(handler, rtsp_st, st); if (handler && handler->init) handler->init(s, st->index, rtsp_st->dynamic_protocol_context); -- cgit v1.2.3