[FFmpeg-devel] [PATCH] RTP depacketizer for AMR

[Luca Abeni]

Hi Martin,

Martin Storsj? wrote:
[...]
>> But let's see if and how the AMR decoder can cope with the data produced by
>> this demuxer if the payload handler does not set the timestamps.
> 
> If rtpdec.c sets pts to AV_NOPTS_VALUE, the correct pts is set by some 
> other code, so that method does seem to work, we only need some kind of 
> fix for rtpdec.c, so that it actually sets this value in these cases.

Actually looking at the code, it seems to me that the fix is simple:
remove the call to finalize_packet() at line 417, and the
"timestamp = 0" assignment at line 414.

Yes, I know the current code says something different, but I still believe
it is wrong :)


				Luca