[FFmpeg-devel] [PATCH] rtpdec_svq3: Don't look for the unused RTP_FLAG_KEY flag
Sun Oct 3 16:00:51 CEST 2010
When looking over some code, I noticed that the svq3 depacketizer looks
for the RTP_FLAG_KEY flag in the flags variable, and sets AV_PKT_FLAG_KEY
accordingly. However, nothing ever sets the RTP_FLAG_KEY in flags (there's
no such bit in a RTP packet, only RTP_FLAG_MARKER is extracted and set),
RTP_FLAG_KEY is only used internally within some RDT code and somewhere in
the rtpdec_asf code currently.
Therefore, these lines of code in rtpdec_svq3 are nothing more than
historical cruft/misunderstandings. Decoding still works just fine if this
Josh, I'm CCing you since I'm not sure if you're still reading this list
regularly. I checked the wiki docs on this, and it currently says
"the RTP marker bit indicates whether the payload data is a keyframe"
which doesn't correspond to what the code currently does, and isn't true.
(Currently the RTP marker bit isn't used at all in the code, and I'm not
sure exactly what it maps to. In most formats, it indicates end of
multi-packet frame, but we have another bit within the payload for that.)
So, are you ok with removing this incorrect sentence?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1473 bytes
More information about the ffmpeg-devel