[FFmpeg-devel] [PATCH] Add to AVCodec a long_name field

Michael Niedermayer michaelni
Thu Apr 17 01:34:22 CEST 2008


On Thu, Apr 17, 2008 at 01:02:22AM +0200, Stefano Sabatini wrote:
> On date Tuesday 2008-04-15 14:33:34 +0200, Michael Niedermayer encoded:
> > On Tue, Apr 15, 2008 at 12:13:38PM +0200, Stefano Sabatini wrote:
> > > On date Tuesday 2008-04-15 11:47:41 +0200, Michael Niedermayer encoded:
> [...]
> > > > missing doxygen comment, all field in public structs should have one
> > > > explaining what the contain
> > > 
> > > Is it OK like this?
> > 
> > well this isnt the best way to describe the field but, yes
> 
> Well, since it didn't still got applied I gave it another try.

Yes this is better, patch looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080417/4aa029c7/attachment.pgp>



More information about the ffmpeg-devel mailing list