On Thu, 13 Oct 2011 14:20:29 +0300, Martin Storsjö <[email protected]> wrote: > Eventually, the old way of passing options by adding > stuff to the URL can be dropped. > > This avoids having to tamper with the user-specified URL to > pass options on the transport mode. This also works better > with redirects, since the options don't need to be parsed out > from the URL. > --- > libavformat/rtsp.c | 28 ++++++++++++++++++++++++++-- > libavformat/rtsp.h | 12 +++++++++++- > libavformat/version.h | 5 ++++- > 3 files changed, 41 insertions(+), 4 deletions(-) > > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c > index e4c4c98..23c7b24 100644 > --- a/libavformat/rtsp.c > +++ b/libavformat/rtsp.c > @@ -59,9 +59,17 @@ > > #define OFFSET(x) offsetof(RTSPState, x) > #define DEC AV_OPT_FLAG_DECODING_PARAM > +#define ENC AV_OPT_FLAG_ENCODING_PARAM > const AVOption ff_rtsp_options[] = { > { "initial_pause", "Don't start playing the stream immediately", > OFFSET(initial_pause), AV_OPT_TYPE_INT, {0}, 0, 1, DEC }, > FF_RTP_FLAG_OPTS(RTSPState, rtp_muxer_flags), > + { "rtsp_transport", "RTSP transport protocols", > OFFSET(lower_transport_mask), AV_OPT_TYPE_FLAGS, {0}, 0, INT_MAX, DEC|ENC, > "rtsp_transport" }, \
flags should have min set to INT_MIN, so 'all' keyword works. same for rtsp_flags Otherwise looks fine to me, you can push with the above fixed. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
