Opened 11 years ago

Closed 11 years ago

#1974 closed defect (fixed)

speex regression

Reported by: ami_stuff Owned by:
Priority: important Component: avcodec
Version: git-master Keywords: regression speex
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

the attached file is 48kHz, but current ffmpeg reports it as 32kHz

03.10.2012 git-df82454 - ok
05.10.2012 git-d9dfe9a - bad

http://samples.mplayerhq.hu/A-codecs/speex/talk109-q5.spx

C:\>ffmpeg -i talk109-q5.spx -f null -
ffmpeg version N-47116-g109268c Copyright (c) 2000-2012 the FFmpeg developers
  built on Nov 27 2012 02:39:53 with gcc 4.5.0 (GCC) 20100414 (Fedora MinGW 4.5.
0-1.fc14)
  configuration: --prefix=/var/www/users/research/ffmpeg/snapshots/build --arch=
x86 --target-os=mingw32 --cross-prefix=i686-pc-mingw32- --cc='ccache i686-pc-min
gw32-gcc' --enable-w32threads --enable-memalign-hack --enable-runtime-cpudetect
--enable-cross-compile --enable-static --disable-shared --extra-libs='-lws2_32 -
lwinmm -lpthread' --extra-cflags='--static -I/var/www/users/research/ffmpeg/snap
shots/build/include' --extra-ldflags='-static -L/var/www/users/research/ffmpeg/s
napshots/build/lib' --enable-bzlib --enable-zlib --enable-gpl --enable-version3
--enable-nonfree --enable-libx264 --enable-libspeex --enable-libtheora --enable-
libvorbis --enable-libfaac --enable-libxvid --enable-libopencore-amrnb --enable-
libopencore-amrwb --enable-libmp3lame --enable-libvpx --disable-decoder=libvpx
  libavutil      52.  9.102 / 52.  9.102
  libavcodec     54. 77.100 / 54. 77.100
  libavformat    54. 37.100 / 54. 37.100
  libavdevice    54.  3.100 / 54.  3.100
  libavfilter     3. 23.102 /  3. 23.102
  libswscale      2.  1.102 /  2.  1.102
  libswresample   0. 17.101 /  0. 17.101
  libpostproc    52.  2.100 / 52.  2.100
Input #0, ogg, from 'talk109-q5.spx':
  Duration: 00:27:08.15, start: 0.010604, bitrate: 28 kb/s
    Stream #0:0: Audio: speex, 32000 Hz, mono, s16
    Metadata:
      TITLE           : OLS 2004: Object-based reverse mapping
      AUTHOR          : Dave McCracken
Output #0, null, to 'pipe:':
  Metadata:
    encoder         : Lavf54.37.100
    Stream #0:0: Audio: pcm_s16le, 32000 Hz, mono, s16, 512 kb/s
    Metadata:
      TITLE           : OLS 2004: Object-based reverse mapping
      AUTHOR          : Dave McCracken
Stream mapping:
  Stream #0:0 -> #0:0 (libspeex -> pcm_s16le)
Press [q] to stop, [?] for help
size=       0kB time=00:27:08.16 bitrate=   0.0kbits/s
video:0kB audio:152640kB subtitle:0 global headers:0kB muxing overhead -100.0000
00%

Change History (4)

comment:1 by Elon Musk, 11 years ago

Sample rate set from ogg demuxer differs from one set in libspeexdec. So look for changes in libspeex decoder.

comment:2 by reimar, 11 years ago

This was broken by 3b061c5e10f78caaf3b2a45cf7a92e50d4d20bfb which changed the code to ignore the sample rate specified in the speex header for no good reason at all and without mentioning it in the commit message.

comment:3 by Carl Eugen Hoyos, 11 years ago

Component: undeterminedavcodec
Keywords: regression speex added
Priority: normalimportant
Status: newopen
Version: unspecifiedgit-master

You definitely expect far too much;-)

comment:4 by Carl Eugen Hoyos, 11 years ago

Resolution: fixed
Status: openclosed

Fixed by Reimar.

Note: See TracTickets for help on using tickets.