Skip to content

gh-113791: Expose CLOCK_MONOTONIC_RAW_APPROX and CLOCK_UPTIME_RAW_APROX on macOS in the time module#113792

Merged
ronaldoussoren merged 2 commits into
python:mainfrom
ronaldoussoren:gh-113791
Jan 8, 2024
Merged

gh-113791: Expose CLOCK_MONOTONIC_RAW_APPROX and CLOCK_UPTIME_RAW_APROX on macOS in the time module#113792
ronaldoussoren merged 2 commits into
python:mainfrom
ronaldoussoren:gh-113791

Conversation

@ronaldoussoren

@ronaldoussoren ronaldoussoren commented Jan 7, 2024

Copy link
Copy Markdown
Contributor

@ronaldoussoren

Copy link
Copy Markdown
Contributor Author

There are no test updates because existing CLOCK_* constants aren't tested either (other than indirectly testing some of them through tests of functions in the module)

@corona10 corona10 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, But let's follow convention of other constants :)

Comment thread Modules/timemodule.c Outdated
Comment thread Modules/timemodule.c Outdated
@corona10

corona10 commented Jan 7, 2024

Copy link
Copy Markdown
Member

LGTM, But let's follow convention of other constants :)

Ah, some of the other constants have empty space. Can we unify them through this PR?

@ronaldoussoren

Copy link
Copy Markdown
Contributor Author

LGTM, But let's follow convention of other constants :)

Ah, some of the other constants have empty space. Can we unify them through this PR?

That's why my entries have empty lines, I copied the last entry in the list :-) . I'll remove the blank lines.

@ronaldoussoren
ronaldoussoren merged commit c6ca562 into python:main Jan 8, 2024
@ronaldoussoren
ronaldoussoren deleted the gh-113791 branch January 8, 2024 19:44
kulikjak pushed a commit to kulikjak/cpython that referenced this pull request Jan 22, 2024
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants