Message124766
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. |
|
Date |
User |
Action |
Args |
2010-12-28 07:53:12 | amaury.forgeotdarc | set | recipients:
+ amaury.forgeotdarc, jaraco, eric.smith, brian.curtin |
2010-12-28 07:53:12 | amaury.forgeotdarc | set | messageid: <1293522792.04.0.744697879165.issue9333@psf.upfronthosting.co.za> |
2010-12-28 07:53:06 | amaury.forgeotdarc | link | issue9333 messages |
2010-12-28 07:53:06 | amaury.forgeotdarc | create | |
|