[FFmpeg-devel] [PATCH] run level decode function for wma &?wmapro

Diego Biurrun diego
Fri Jun 12 15:12:07 CEST 2009


On Fri, Jun 12, 2009 at 11:47:54AM +0200, Sascha Sommer wrote:
> 
> On Freitag, 12. Juni 2009, Diego Biurrun wrote:
> > On Fri, Jun 12, 2009 at 11:13:32AM +0200, Sascha Sommer wrote:
> > > On Donnerstag, 11. Juni 2009, Michael Niedermayer wrote:
> > > > On Thu, Jun 11, 2009 at 10:17:19PM +0200, Sascha Sommer wrote:
> > > > > attached patch adds a run level decode function that can be used for
> > > > > both wma12 and wmapro.
> > > >
> > > > could you split it into a patch factorizig the code out and one
> > > > changing it ?
> > > > so that code moving around is seperate from functional changes ...
> > > >
> > > > [...]
> > >
> > > Attached patch finally fixes a few remaining style problems.
> >
> > Looks good, but of course there are more issues in libavcodec/wma.c ...
> 
> Yes. However, my first priority is wmapro.

File reformatted, this should save you some work.

Diego



More information about the ffmpeg-devel mailing list