Message75812
The patch no longer applies to trunk, but that would be trivial to fix.
Changes like these have been accepted with little resistance in the past
(see e.g. issue651362), so I don't see why this patch has been pending
for so long. (In general, I am -0 on adding const modifiers because
they tend to trigger an avalanche of changes, but arguably this is the
"right thing to do.")
A few comments:
* Documentation needs to be updated in Doc/c-api/sys.rst.
* Any reason why PySys_SetPath(char *) is left out?
* Same for PySys_SetArgv(int, char **)
* Similar changes are proposed in issue1772673. Maybe these two issues
can be dealt with together. |
|
Date |
User |
Action |
Args |
2008-11-13 04:26:04 | belopolsky | set | recipients:
+ belopolsky, christian.heimes, sebastinas |
2008-11-13 04:26:04 | belopolsky | set | messageid: <1226550364.53.0.39048335955.issue1699259@psf.upfronthosting.co.za> |
2008-11-13 04:26:03 | belopolsky | link | issue1699259 messages |
2008-11-13 04:26:03 | belopolsky | create | |
|