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

[Michael Niedermayer]

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

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- 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/3b648ae7/attachment.pgp>