[MPlayer-dev-eng] [PATCH] replacement for internal mpg123 fork (mp3lib)

Uoti Urpala uoti.urpala at pp1.inet.fi
Wed May 12 16:47:41 CEST 2010


On Wed, 2010-05-12 at 16:44 +0200, Thomas Orgis wrote:
> Am Wed, 12 May 2010 14:03:12 +0200
> schrieb Diego Biurrun <diego at biurrun.de>: 
> 
> > On Wed, May 12, 2010 at 01:27:39PM +0200, Thomas Orgis wrote:
> > > The next iteration, fixing stream input handling for large frames (I
> > > got one bit wrong in the transition to the old-style API). CBR 320
> > > works for me now.
> > 
> > You've extensively commented your code.  Converting those comments to
> > Doxygen syntax would be even better.
> 
> Any hints on how it should be structured? Example files that fit into
> the existing doxygen setup?

I think you shouldn't waste effort on this. Overall generating doxygen
output for MPlayer is not really useful. Non-doxygen syntax is fine.




More information about the MPlayer-dev-eng mailing list