[MPlayer-cvslog] CVS: main AUTHORS,1.128,1.129

Diego Biurrun diego at biurrun.de
Sun Nov 14 17:57:44 CET 2004


The Wanderer writes:
> Diego Biurrun CVS wrote:
> 
> > -    * mga_vid fixes, module option etc.
> > -    * fbdev support in libvo
> > +    * mga_vid fixes, module option etc
> 
> Missing comma.

Yes.

> Also, because "etc." is an abbreviation for "et cetera", the period
> probably is in fact appropriate even though this is not a full sentence.

Dunno, maybe, but it's written without period everywhere else.

> > -    * PNG file output support in libvo
> > +    * png video output driver
> 
> Sure about your capitalization here? (IOW, are you in fact referring to
> the string which replaces the wildcard in vo_*, and not the actual name
> of the format? Because at the very least, you didn't do the same thing
> with the line about ALSA-in-libao2 above...)

Your theory is correct, same below ;-)

> > -    * SUN audio driver in libao2
> > +    * SUN audio output driver in libao2
> 
> Is this correctly capitalized? I'm 99% certain it's not an acronym, and
> I don't *think* it's properly spelled in all caps.

You're right.

Everything fixed in my local tree, will commit eventually.

Diego




More information about the MPlayer-cvslog mailing list