[MPlayer-cvslog] CVS: main/DOCS/tech swscaler_methods.txt, 1.1, 1.2
Diego Biurrun
diego at biurrun.de
Wed Aug 24 02:41:39 CEST 2005
On Tue, Aug 23, 2005 at 12:40:05PM -0400, The Wanderer wrote:
>
> (if I'd realized that there were any files left with that many problems,
> I'd have gone hunting them down long since)
Point your text editor at DOCS/tech/* and let the fun begin...
> (While I'm at this: there is at least one place in that file where there
> is a line which is entirely blank except for roughly a tab-width of
> spaces. I can understand such being useful for cursor-positioning
> convenience in actual source code, but it doesn't seem to serve any
> purpose here, especially since nothing in the file is directly indented;
> would it be acceptable to remove those useless spaces, in the name of
> reducing filesize slightly?)
>
> (And, on loosely the same subject, there are several lines on which
> there are trailing spaces which serve no obvious function; I'd be
> inclined to remove them as well, unless someone objects.)
I dislike trailing whitespace as well. I remove it when I change the
line anyway. It is a purely cosmetic change, so I mostly just don't
bother. IIRC I have removed trailing whitespace from some XML docs in
separate commits. If you feel strongly about this, go ahead, I won't
mind.
Diego
More information about the MPlayer-cvslog
mailing list