[FFmpeg-devel] [PATCH 2/2] Add ID support for .oma/.aa3

[Aurelien Jacobs]

On Wed, Jun 16, 2010 at 02:25:45PM +0200, Maxim wrote:
> Maxim schrieb:
> > Maxim schrieb:
> >   
> >> Michael Karcher schrieb:
> >>   
> >>     
> >>> Am Sonntag, den 13.06.2010, 13:03 +0200 schrieb Reimar D?ffinger:
> >>>   
> >>>     
> >>>       
> >>>>>> Don't worry too much about that.
> >>>>>> Anyway, I at least see no more mistakes/issues...
> >>>>>>         
> >>>>>>           
> >>>>>>             
> >>>>> then theres no reason not to commit it and add yourself to MAINTAINERs
> >>>>>       
> >>>>>         
> >>>>>           
> >>>> I can, though I haven't looked much at it and the original authors
> >>>> (Maxim and Benjamin) are still around.
> >>>> I assume nobody wants to maintain it?
> >>>>     
> >>>>       
> >>>>         
> >>> Maxim seems to be willing to maintain it, see
> >>> http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2010-June/090262.html
> >>>   
> >>>     
> >>>       
> >> Yes, it's true. I'd gladly maintain that demuxer but haven't had enough
> >> time to peer into it.
> >> I have alot of .oma/.aa3 samples on my HD. Let me test the code tonight.
> >> If there is no issues we'll commit it tomorrow, ok?
> >>   
> >>     
> >
> > I just tried the patch in both x86 and my G5 PPC box. It works fine for
> > all samples on x86, but refuses to display any metadata on PPC. Neither
> > ffplay nor ffmpeg shows metadata. I run the code through the gdb and
> > made sure that the metadata was decoded correctly. Has someone any
> > suggestions why it doesn't shows anything?
> >   
> 
> Ping to the maintainers of the metadata API: where should I start to
> look in order to fix the problem described above?

No idea what your problem could be...
Metadata API should work the same on any arch.
I just tried on PPC and got the same result than on x86:

$ ./ffplay http://samples.mplayerhq.hu/Matroska/Mushishi24-head.mkv
FFmpeg version SVN-r23634, Copyright (c) 2000-2010 the FFmpeg developers
  built on Jun 16 2010 21:27:30 with gcc 3.3.5 (Debian 1:3.3.5-5)
  configuration: --enable-gpl
  libavutil     50.19. 0 / 50.19. 0
  libavcodec    52.76. 0 / 52.76. 0
  libavformat   52.68. 0 / 52.68. 0
  libavdevice   52. 2. 0 / 52. 2. 0
  libavfilter    1.20. 0 /  1.20. 0
  libswscale     0.11. 0 /  0.11. 0
[matroska @ 0x109c0d40]Estimating duration from bitrate, this may be
inaccurate
Input #0, matroska, from 'http://samples.mplayerhq.hu/Matroska/Mushishi24-head.mkv':
  Metadata:
    title           : Mushishi24
  Duration: 00:24:22.44, start: 0.000000, bitrate: N/A
    Chapter #0.0: start 0.000000, end 99.992000
    Metadata:
      title           : OP
    Chapter #0.1: start 99.992000, end 669.972000
    Metadata:
      title           : A part
    Chapter #0.2: start 669.972000, end 1330.018000
    Metadata:
      title           : B part
    Chapter #0.3: start 1330.018000, end 1420.027000
    Metadata:
      title           : ED
    Chapter #0.4: start 1420.027000, end 1462.449000
    Metadata:
      title           : next
    Stream #0.0(jpn): Video: mpeg4, yuv420p, 824x464 [PAR 1:1 DAR
    103:58], 23.98 fps, 23.98 tbr, 2k tbn, 23.98 tbc
    Metadata:
      BPS             : 372786
      BITSPS          : 372786
      FPS             : 23.9762
    Stream #0.1(jpn): Audio: aac, 48000 Hz, stereo, s16
    Metadata:
      FPS             : 46.1568
      BPS             : 214835
      BITSPS          : 214835
At least one output file must be specified

Aurel