[FFmpeg-devel] RTP mark bit not passed to parse_packet

[Luca Abeni]

Ronald S. Bultje wrote:
> Hi Luca,
> 
> On Thu, Jan 15, 2009 at 9:16 AM, Luca Abeni <lucabe72 at email.it> wrote:
>> You could use "2" directly, since this is a value used
>> only in ff_rdt_parse_packet(), and rdt_parse_packet(), and has nothing
>> to do with an AVPacket...
> 
> Don't get me wrong, 2 is fine with me and I understand your objections
> to using AVPacket.flags values. But just to be clear, we need it
> defined somewhere, i.e. not just "2", we need a public place
> (rtp_internal.h or so)
Agreed. We can have a define in rtp.h or so (I would say that since it
really is an RDT thing it should go in rdt.c, but I see that RDT is already
using rtp.h and friends...)


BTW, rtp_internal.h is going to die: I have some patches for merging rtp.h
and rtp_internal.h and splitting the resulting file in rtp.h, rtpdec.h and
rtpenc.h. This fixes some dependencies and allows to cleanup the code a
little bit...

I am waiting for all the RTP changes to be stabilised, and then I'll submit
my patches.


				Luca