[Ffmpeg-devel] Re: [patch] const-correctness

Diego Biurrun diego
Sat Jun 17 17:54:00 CEST 2006


On Sat, Jun 17, 2006 at 05:15:04PM +0200, Baptiste Coudurier wrote:
> Diego Biurrun wrote:
> > On Sat, Jun 17, 2006 at 02:14:51PM +0200, Baptiste COUDURIER wrote:
> >> Michael Niedermayer wrote:
> >>> On Sat, Jun 17, 2006 at 12:28:08AM +0200, Stefan Huehner wrote:
> >>>>                                                                                 
> >>>> attached patch add (mostly) const to variable and parameter declaration,        
> >>>> where a char* was used and and 'const char*' should be. Additionally            
> >>>> some function declarations are made static if they aren't used outside          
> >>>> its declaring source file.                                                      
> >>>> Only files under the libavformat directory are affected by this patch.          
> >>>>                                                                                 
> >>>> Please review and consider for inclusion.                                       
> >>> patch looks ok assuming it compiles & regression tests pass ...
> >> Compiles fine and pass regression tests here.
> > 
> > What's stopping you from committing it then? :)
> 
> Another test from someone on another arch maybe ? :)

Compiles and passes regression tests here on PPC, applied.

Diego




More information about the ffmpeg-devel mailing list