[RFC] using `if WINDOWS` instead of `if sys.platform == 'win32'`?
Vincent Ladeuil
v.ladeuil+lp at free.fr
Thu Sep 6 11:22:31 BST 2007
>>>>> "bialix" == Alexander Belchenko <bialix at ukr.net> writes:
bialix> There are around 100 places where bzrlib code try to check
bialix> if it run on Windows by comparing `sys.platform == 'win32'`
bialix> or similar. About the half of this checks is inside core code,
bialix> and another half in tests.
bialix> I'd like to change this places by defining in bzrlib/__init__.py
bialix> constant with name 'WINDOWS' or something similar and use
bialix> this constant where needed.
bialix> WINDOWS = sys.platform == 'win32'
bialix> ...
bialix> if WINDOWS:
bialix> do_something_crazy()
bialix> if not WINDOWS:
bialix> do_another_thing()
bialix> Proposed change should make code more clear.
bialix> Anyone have objections or suggestions?
Can you try to reduce the 100 places to functions inside osutils
instead ?
Vincent
More information about the bazaar
mailing list