[MPlayer-dev-eng] [PATCH] TOOLS/cpuinfo.c: correct k6_mtrr detection

Guillaume Poirier guillaume.poirier at etudiant.univ-rennes1.fr
Wed Nov 23 18:34:08 CET 2005


Hi,

The Wanderer wrote:
> Zuxy wrote:
>> 2005/11/23, Diego Biurrun <diego at biurrun.de>:
>>> On Wed, Nov 23, 2005 at 09:36:51PM +0800, Zuxy wrote:
>>>> Default tabstop is 8 instead of 4?
>>>
>>>
>>> 0 as in no tabs at all.
>>
>>
>> Err...but there ARE tabs in cpuinfo.c and it seems that in many
>> mplayer source files tabs and spaces are mixed for indentation.
> 
> 
> This is because a) the rule was not necessarily always in place, b) the
> rule has not always been rigorously enforced, c) people ignore the rule,
> and d) no one's taken the effort to go in and make a comprehensive
> cosmetics patch to replace all preexisting tabs with spaces.

Don't worry, I'll commit Zuxy's patch without cosmetics, and commit a
patch to use only spaces afterwards (if on one objects).

Guillaume




More information about the MPlayer-dev-eng mailing list