[FFmpeg-devel] [PATCH] AVR demuxer

[Peter Ross]

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

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121105/ce8f92af/attachment.asc>