On Mon, Feb 20, 2012 at 10:55 AM, antoine.pitrou <python-checkins at python.org> wrote: > +def _relax_case(): > + """True if filenames must be checked case-insensitively.""" > + if any(map(sys.platform.startswith, CASE_INSENSITIVE_PLATFORMS)): > + def _relax_case(): > + return b'PYTHONCASEOK' in _os.environ > else: > - return True Wow, that's horrendously confusing. Please change the name of the factory function to "_make_relax_case" (or something, anything that isn't "_relax_case" would be an improvement). Also, the docstring should be on the created functions, not the factory function. Cheers, Nick. -- Nick Coghlan | ncoghlan at gmail.com | Brisbane, Australia
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4