Skip to content

Include time.h explicitly#57

Closed
aroningruber wants to merge 1 commit intopyranges:masterfrom
aroningruber:master
Closed

Include time.h explicitly#57
aroningruber wants to merge 1 commit intopyranges:masterfrom
aroningruber:master

Conversation

@aroningruber
Copy link
Copy Markdown

Pull Request Description

This pull request addresses a build issue related to the use of clock_t and CLOCKS_PER_SEC in the find_intervals_stack functions, which appear in the following files:

The existence of these symbols is implicitly assumed, which can cause the build to fail in environments where time.h is not automatically included (#56).

Changes Made

  • Added an explicit #include <time.h> directive to defaults.h to ensure clock_t and CLOCKS_PER_SEC are always defined.

Issue Fixed

@endrebak
Copy link
Copy Markdown
Collaborator

endrebak commented Jul 4, 2025

I think these were leftovers from some benchmarking. I can just delete these references.

@endrebak
Copy link
Copy Markdown
Collaborator

Thanks for making me aware of this bw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows build fails for CPython 3.13

2 participants