Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-36142: Move command line parsing to coreconfig.c #12123

Merged
merged 2 commits into from
Mar 1, 2019
Merged

bpo-36142: Move command line parsing to coreconfig.c #12123

merged 2 commits into from
Mar 1, 2019

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Mar 1, 2019

  • Add _PyCoreConfig_ReadFromArgv() function which parses command line
    options: move code from main.c to coreconfig.c.
  • Add _PyCoreConfig_Write() to write the new configuration: coerce
    the LC_CTYPE locale, set Py_xxx global configuration variables,
    etc.
  • _PyCoreConfig_ReadFromArgv() now only changes the LC_CTYPE locale
    temporarily. _PyCoreConfig_Write() becomes responsible to set the
    LC_CTYPE locale.
  • Add _Py_SetArgcArgv() and _Py_ClearArgcArgv() functions
  • Rename many "pymain_xxx()" functions
  • Add "const" to some function parameters
  • Reorganize main.c to declare functions in the order in which they
    are called.

https://bugs.python.org/issue36142

* Add _PyCoreConfig_ReadFromArgv() function which parses command line
  options: move code from main.c to coreconfig.c.
* Add _PyCoreConfig_Write() to write the new configuration: coerce
  the LC_CTYPE locale, set Py_xxx global configuration variables,
  etc.
* _PyCoreConfig_ReadFromArgv() now only changes the LC_CTYPE locale
  temporarily. _PyCoreConfig_Write() becomes responsible to set the
  LC_CTYPE locale.
* Add _Py_SetArgcArgv() and _Py_ClearArgcArgv() functions
* Rename many "pymain_xxx()" functions
* Add "const" to some function parameters
* Reorganize main.c to declare functions in the order in which they
  are called.
@vstinner vstinner merged commit 95e2cbf into python:master Mar 1, 2019
@vstinner vstinner deleted the preparse5 branch March 1, 2019 15:25
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