Skip to content

Conversation

earlephilhower
Copy link
Collaborator

sntp_get_real_time takes a time_t in the core, but in the header it
was listed as long. Make them both match by changing the header.

sntp_get_real_time takes a `time_t` in the core, but in the header it
was listed as `long`.  Make them both match by changing the header.
@earlephilhower earlephilhower requested a review from d-a-v November 14, 2020 02:02
@earlephilhower
Copy link
Collaborator Author

This was spotted while doing the newlib 3.3 port where time_t has gone to 64-bit (i.e. a long long) to avoid the year 2038 problem. On the current core, this is just a cleanup and shouldn't change any code.

@d-a-v d-a-v merged commit 417aacc into esp8266:master Nov 14, 2020
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