[issue9333] Expose a way to enable os.symlink on Windows

Amaury Forgeot d'Arc report at bugs.python.org
Tue Dec 28 08:53:12 CET 2010


Amaury Forgeot d'Arc <amauryfa at gmail.com> added the comment:

The patch looks good, I only have stylistic remarks:
- We normally don't use windows-specific types in CPython code. Please use int instead of BOOL. And C variables are usually lowercase, even module globals. I suggest something like "static int win32_can_symlink;"
- the enable_symlink() function should be "static" as well.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue9333>
_______________________________________


More information about the Python-bugs-list mailing list