[MPlayer-dev-eng] [PATCH] charset checking for TOOLS/checktree.sh

The Wanderer inverseparadox at comcast.net
Mon Dec 5 01:29:34 CET 2005


The Wanderer wrote:

> Ivo wrote:
> 
>> On Friday 11 November 2005 19:15, Ivo wrote:
>> 
>>> I once thought of adding an option, so each check could either
>>> print a list of filenames and just that, or it could also dump
>>> the offending lines per file, but I never got around to making
>>> it. If you want, you can add that :)
>> 
>> Ok, maybe something like this patch.
>> 
>> And then TOOLS/checktree.sh -all -showcont
> 
> It's been two weeks - any comments on this patch?

Another week-plus later without any comments: the patch looks okay to my
(comparatively inexperienced) eyes, still applies without errors, and
appears to behave correctly; I'll commit it in a day or two if no one
speaks up against it.

(Although, I'll note, there is apparently one UNHOLY SHITLOAD of
trailing whitespace in the source tree...)

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

Secrecy is the beginning of tyranny.




More information about the MPlayer-dev-eng mailing list