[FFmpeg-soc] [soc]: r3957 - wmapro/wma3dec.c

[Sascha Sommer]

Hi,

On Donnerstag, 15. Januar 2009, Kostya wrote:
> On Thu, Jan 15, 2009 at 11:09:14AM -0800, Baptiste Coudurier wrote:
> > Hi,
> >
> > > Modified: wmapro/wma3dec.c
> > > =======================================================================
> > >======= --- wmapro/wma3dec.c	Thu Jan 15 19:45:48 2009	(r3956)
> > > +++ wmapro/wma3dec.c	Thu Jan 15 19:47:58 2009	(r3957)
> > > @@ -1616,7 +1616,7 @@ static int wma3_decode_packet(AVCodecCon
> > >   */
> > >  AVCodec wmapro_decoder =
> > >  {
> > > -    "wmav3pro",
> > > +    "wmapro",
> > >      CODEC_TYPE_AUDIO,
> > >      CODEC_ID_WMAPRO,
> > >      sizeof(WMA3DecodeContext),
> >
> > Considering we have wmav1 and wmav2, I think wmav3pro was a better name,
> > what do other think ?
>
> Personally I'd rather use simple wmav3, do we have non-pro version anyway?

Afaik there is no non-pro version that is not compatible to wmav2.
I think I'll leave it as it is for now and we decide this before the merge.

Regards

Sascha