Skip to content

Removed tzp variable from gettimeofday function for linux#26

Merged
Daivuk merged 2 commits intoDaivuk:masterfrom
scandown:master
Oct 18, 2025
Merged

Removed tzp variable from gettimeofday function for linux#26
Daivuk merged 2 commits intoDaivuk:masterfrom
scandown:master

Conversation

@scandown
Copy link
Contributor

On linux its a deprecated feature and adding this change allows for the program to compile.

In the man page for the function: "If tzp is not a null pointer, the behavior is unspecified."

@Daivuk
Copy link
Owner

Daivuk commented Oct 18, 2025

Hi, first of all: thank you so much for the contribution! I had no idea, as I do not use linux.

The file you modified is auto generated. Please edit the right header and re-generate the single header using gen_single_header.py

@scandown
Copy link
Contributor Author

Thanks for letting me know, I have now changed the code in the source directly rather than just editing the header file.

I used the generate python script, and it made a new header with the changes and it also compiles correctly into a test program.

It should now be ready to merge!

@Daivuk
Copy link
Owner

Daivuk commented Oct 18, 2025

Oops all the OSX workflow seem to fail. Probably a cmake version mismatch with latest. I will merge anyway because your changes can't cause this.

Compatibility with CMake < 3.5 has been removed from CMake

@Daivuk Daivuk merged commit c169e11 into Daivuk:master Oct 18, 2025
6 of 10 checks passed
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.

2 participants