Skip to content

bpo-32030: Simplify _PyCoreConfig_INIT macro#4728

Merged
vstinner merged 1 commit intopython:masterfrom
vstinner:path_config_cleanup
Dec 5, 2017
Merged

bpo-32030: Simplify _PyCoreConfig_INIT macro#4728
vstinner merged 1 commit intopython:masterfrom
vstinner:path_config_cleanup

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Dec 5, 2017

  • Simplify _PyCoreConfig_INIT, _PyMainInterpreterConfig_INIT,
    _PyPathConfig_INIT macros: no need to set fields to 0/NULL, it's
    redundant (the C language sets them to 0/NULL for us).
  • Fix typo: pymain_run_statup() => pymain_run_startup()
  • Remove a few XXX/TODO

https://bugs.python.org/issue32030

* Simplify _PyCoreConfig_INIT, _PyMainInterpreterConfig_INIT,
  _PyPathConfig_INIT macros: no need to set fields to 0/NULL, it's
  redundant (the C language sets them to 0/NULL for us).
* Fix typo: pymain_run_statup() => pymain_run_startup()
* Remove a few XXX/TODO
@vstinner vstinner merged commit 33c377e into python:master Dec 5, 2017
@vstinner vstinner deleted the path_config_cleanup branch December 5, 2017 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants