[FFmpeg-devel] [PATCH] QCLP wrong sampling rate

Michael Niedermayer michaelni
Wed Sep 24 19:28:59 CEST 2008


On Wed, Sep 24, 2008 at 10:10:02AM -0700, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > On Sat, Aug 16, 2008 at 09:58:08PM +0900, KO Myung-Hun wrote:
> >>
> >> Baptiste Coudurier wrote:
> >>> Hi,
> >>>
> >>> KO Myung-Hun wrote:
> >>>   
> >>>> Reynaldo H. Verdejo Pinochet wrote:
> >>>>     
> >>>>> Hi KO Myung-Hun
> >>>>>
> >>>>> KO Myung-Hun wrote:
> >>>>>  
> >>>>>       
> >>>>>> Hi/2.
> >>>>>>
> >>>>>> Sampling rate of QCLP audio is always set to 8000Hz. But it's wrong.
> >>>>>>
> >>>>>> Test clips : http://sketchup.google.com/gsu6vtvideos.html
> >>>>>>     
> >>>>>>         
> >>>>> I had problems with these kind of streams in the past
> >>>>>
> >>>>> If this QCLP tag identifies TIA/EIA/ISS-733 conforming
> >>>>> data then this is just wrong. See, these are sampled at
> >>>>> 8kHz -- period. this is no optional recomendation but
> >>>>> a requirement (You can take a look at section 2.4.1
> >>>>> of the all infamous C.S0020-0with3Gcover.pdf if you
> >>>>> feel like it) Now, if this patch works out decoding
> >>>>> for you I'd suggest adding a warning to state out-spec
> >>>>> data, at the very least.
> >>>>>   
> >>>>>       
> >>>> I've added a warning to the patch as your suggestion.
> >>>>     
> >>> Does this file decodes correctly with ffmpeg ?
> >>> With soc decoder or non redistributable one ?
> >>>
> >>>   
> >> Ah, MPlayer with QuickTime support.
> > 
> > baptiste?
> > IMHO the mov demuxer should not override the stored sampling rate for
> > qcelp & amr
> 
> Well, I'm not disagreeing, the code was this way a long time ago, should
> be fixed.

thanks

Though iam realizing now that the overriding may be needed for .mp4, though
thats just a feeling based on the reliability of such fields for other
codecs like aac in .mp4 ...


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080924/e8a47c91/attachment.pgp>



More information about the ffmpeg-devel mailing list