Skip to content

bpo-45412: Add _PY_SHORT_FLOAT_REPR macro#31171

Merged
vstinner merged 1 commit intopython:mainfrom
vstinner:pycore_pymath
Feb 23, 2022
Merged

bpo-45412: Add _PY_SHORT_FLOAT_REPR macro#31171
vstinner merged 1 commit intopython:mainfrom
vstinner:pycore_pymath

Conversation

@vstinner
Copy link
Copy Markdown
Member

@vstinner vstinner commented Feb 6, 2022 •
edited by bedevere-bot
Loading

Remove the HAVE_PY_SET_53BIT_PRECISION macro (moved to the internal
C API).

  • Move HAVE_PY_SET_53BIT_PRECISION macro to pycore_pymath.h.
  • Replace PY_NO_SHORT_FLOAT_REPR macro with _PY_SHORT_FLOAT_REPR
    macro which is always defined. gcc -Wundef emits a warning when
    using _PY_SHORT_FLOAT_REPR but the macro is not defined, if
    pycore_pymath.h include was forgotten.

https://bugs.python.org/issue45412

@vstinner
bpo-45412: Add _PY_SHORT_FLOAT_REPR macro
5ee49fa 
Remove the HAVE_PY_SET_53BIT_PRECISION macro (moved to the internal
C API).

* Move HAVE_PY_SET_53BIT_PRECISION macro to pycore_pymath.h.
* Replace PY_NO_SHORT_FLOAT_REPR macro with _PY_SHORT_FLOAT_REPR
  macro which is always defined. gcc -Wundef emits a warning when
  using _PY_SHORT_FLOAT_REPR but the macro is not defined, if
  pycore_pymath.h include was forgotten.
@vstinner vstinner mentioned this pull request Feb 6, 2022
Merged
@vstinner
Copy link
Copy Markdown
Member Author

vstinner commented Feb 6, 2022

In my PR GH-28882, I tried to avoid making these changes because:

  • It removes the public HAVE_PY_SET_53BIT_PRECISION macro.
  • pycore_pymath.h must be included, otherwise _PY_SHORT_FLOAT_REPR is not included. I replaced PY_NO_SHORT_FLOAT_REPR with _PY_SHORT_FLOAT_REPR to be able to detect such bug using gcc -Wundef.

I searched for HAVE_PY_SET_53BIT_PRECISION in the source code of top 5000 PyPI project: no project use it. But I documented the change, since it impacts the public C API.

@vstinner
Copy link
Copy Markdown
Member Author

vstinner commented Feb 6, 2022

@mdickinson: Here you have.

@vstinner vstinner mentioned this pull request Feb 7, 2022
Merged
@vstinner vstinner merged commit 9bbdde2 into python:main Feb 23, 2022
@vstinner vstinner deleted the pycore_pymath branch February 23, 2022 17:16
@seberg seberg mentioned this pull request May 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner @the-knights-who-say-ni @bedevere-bot