[PATCH] util: rename _windows_reserved_filenames and _windows_reserved_chars

Adrian Buehlmann adrian at cadifra.com
Sat May 7 17:29:05 CDT 2011


On 2011-05-07 23:38, Sune Foldager wrote:
> On 2011-05-07 23:07, Adrian Buehlmann wrote:
>> # HG changeset patch
>> # User Adrian Buehlmann <adrian at cadifra.com>
>> # Date 1304799920 -7200
>> # Node ID 4168febc29ffaee26de5cc1b1289454e4c85debf
>> # Parent  fe2153fd0fe5de49790e82d8038f5e07dd502e63
>> util: rename _windows_reserved_filenames and _windows_reserved_chars
> 
> I realize we have this no-underscore policy, which I don't agree with, but is
> this really supposed to be an improvement?:
> 
>> -_windows_reserved_filenames = '''con prn aux nul
>> +_winresnames = '''con prn aux nul
> 
> Res? Resource names?... this is the primary reason I am opposed to
> no-underscores :p.

Yeah. In *that* code, it is really hard see what _winresnames is
supposed to be hmm? You even had to delete the interesting bits, to
prove your point :-)

Oh. Yeah. Let's see. And the function is called checkwinfilename. What
the heck is win again. Hmmm.

I'm sure the next email is coming from Martin :-)


More information about the Mercurial-devel mailing list