[FFmpeg-cvslog] r15288 - trunk/libavcodec/mace.c

Diego Biurrun diego
Wed Sep 10 01:15:32 CEST 2008


On Tue, Sep 09, 2008 at 05:27:13PM -0400, The Wanderer wrote:
> Vitor Sessak wrote:
> 
> > Diego Biurrun wrote:
> > 
> >> On Tue, Sep 09, 2008 at 06:51:39PM +0200, vitor wrote:
> >> 
> >>> Log:
> >>> Cosmetics: remove useless parenthesis
> >> 
> >> singular: parenthesis, plural: parentheses
> > 
> > Done.
> 
> I commented on this against an earlier commit (15222), which still
> hasn't been changed. I wasn't sure it needed changing, but if this one
> does...

It's changed already.

> > PS: Have a look also at the comments in my code. It's been quite some
> > time since anyone complained of the english in my comments and it
> > would surprise me that it is better there than in the logs...
> 
> A comprehensive review of the state of the English in all code comments
> is on my to-do list, but it's a major task and I don't have a lot of
> time to spend on such things

It is a major task that is quite easy to break into small pieces: Just
do it one file at a time.  Every little bit is an improvement.

Diego




More information about the ffmpeg-cvslog mailing list