[FFmpeg-devel] [PATCH] RTSP-MS 12/15: ASF payload support

Luca Abeni lucabe72
Wed Mar 18 00:18:16 CET 2009


On Tue, 2009-03-17 at 08:49 -0400, Ronald S. Bultje wrote:
[...]
> > you are missing FF_INPUT_BUFFER_PADDING_SIZE besides
> > why not just set extradata(_size) to 0 in the source? or does that fail somehow?
> 
> Good point, and that works indeed. In this revision, I also removed
> the "rtx" payload which I initially added because I now know it isn't
> needed for basic playback (and wouldn't use the same packet layout
> anyway).
> 
> New patch attached. Luca (Abeni :-) ), could you review the rtpdec.c parts?

The rtpdec part is just registering 2 payload handlers, right?
Why not having the ff_ms_rtp_asf_pfv_handler() and
ff_ms_rtp_asf_pfa_handler() prototypes in a header file (rtp_asf.h)?

Anyway, looks ok if it works.


				Luca





More information about the ffmpeg-devel mailing list