[Python-checkins] r53489 - in python/trunk: Include/Python-ast.h Misc/NEWS Parser/asdl_c.py

["Martin v. Löwis"]

Jim Jewett schrieb:
> How does this fix the issue?

As Thomas explains: the issue (#1637022) was a run-time (actually,
link-time) issue. By renaming the symbols at compile-time, the
issue is fixed.

> Adding the _Py_ names is a start, but I thought a fix would require
> removing the non-_Py_ names.  This code still #defines them, as an
> alias of the new name.

Why do you think #define's could cause conflicts at link time?

> I would understand not removing the old names (or at least doing it
> only conditionally) in python2.5, but why keep them on the trunk?

Thomas also guessed correctly here: it improves readability to
keep the original names in the source code.

Regards,
Martin