[FFmpeg-devel] [PATCH] Add the function libavcodec/utils.c:avcodec_type_string

Stefano Sabatini stefano.sabatini-lala
Thu Nov 8 18:02:30 CET 2007


On date Thursday 2007-11-08 17:07:35 +0100, Benoit Fouet encoded:
> Stefano Sabatini wrote:
> > On date Thursday 2007-11-08 16:43:15 +0100, Diego Biurrun encoded:
> >   
> >> On Thu, Nov 08, 2007 at 04:18:48PM +0100, Stefano Sabatini wrote:
[...]
> >>> + * @param codec_type[in] Code of the codec type. Values actually supported are:
> >>>       
> >> Supported values are
> >>
> >> Look up the meaning of actually again, it's not what you seem to think.
> >>     
> >
> > From The Collaborative International Dictionary of English v.0.48 [gcide]:
> >
> >   Actually \Ac"tu*al*ly\, adv.
> >      1. Actively. [Obs.] "Neither actually . . . nor passively."
> >         --Fuller.
> >         [1913 Webster]
> >   
> >      2. In act or in fact; really; in truth; positively.
> >         [1913 Webster]
> >
> > I meant "now", implicating that the number of supported types might
> > increment in future, trying to keep the documentation as generic as
> > possible.
> >
> >   
> 
> then why not "currently" ?
> to me, actually means "in fact"

Seconded and new patch attached. 

Best regards.
-- 
Stefano Sabatini
Linux user number 337176 (see http://counter.li.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-avcodec-type-string-function-02.patch
Type: text/x-diff
Size: 2215 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071108/e6fc5236/attachment.patch>



More information about the ffmpeg-devel mailing list