[FFmpeg-devel] [PATCH] AVR demuxer

Paul B Mahol onemda at gmail.com
Mon Nov 5 12:11:37 CET 2012


On 11/5/12, Peter Ross <pross at xvid.org> wrote:
> On Mon, Nov 05, 2012 at 12:31:29AM +0000, Paul B Mahol wrote:
>> On 11/4/12, Peter Ross <pross at xvid.org> wrote:
>> > On Sun, Nov 04, 2012 at 02:28:22PM +0000, Paul B Mahol wrote:
>> >> On 11/4/12, Peter Ross <pross at xvid.org> wrote:
>> >> > On Sun, Nov 04, 2012 at 04:04:31AM +0000, Paul B Mahol wrote:
>> >> >> Signed-off-by: Paul B Mahol <onemda at gmail.com>
>> >> >> ---
>> >> >>  doc/general.texi         |  2 +
>> >> >>  libavformat/Makefile     |  1 +
>> >> >>  libavformat/allformats.c |  1 +
>> >> >>  libavformat/avr.c        | 95
>> >> >> ++++++++++++++++++++++++++++++++++++++++++++++++
>> >> >>  4 files changed, 99 insertions(+)
>> >> >>  create mode 100644 libavformat/avr.c
>> >> >>
>> >>
>> >> [..]
>> >> > accuracy could be improved by test other fields for sanity
>> >> > e.g bits_per_coded_sample, and sample_rate
>> >>
>> >> and compare it to what min value?
>> >
>> > zero!
>> >
>> > just a suggestion. also bps will never be > 64.
>>
>> Most if not all demuxers do not do this and that is not their job,
>> it just bloat code adding myriad checks through lavf files instead in
>> one single place....
>
> Plenty do, but fair point.
>
> bink
> dnxhd
> dxa
> eacdata
> idcin
> westwood_aud
> xa
> mm
> seirravmd

You mean for probe. That is doable, but it may not work well, because
there is a lot bytes that can be almost anything.


More information about the ffmpeg-devel mailing list