Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing ctime include #6269

Merged
merged 1 commit into from
Aug 18, 2017
Merged

Conversation

adrido
Copy link
Contributor

@adrido adrido commented Aug 18, 2017

Without that include MSVC will fail, because time_t is not in the namespace std.

@SmallJoker SmallJoker added Bug Issues that were confirmed to be a bug One approval ✅ ◻️ Windows labels Aug 18, 2017
@SmallJoker
Copy link
Member

👍

@nerzhul
Copy link
Member

nerzhul commented Aug 18, 2017

removed the last comment, okay for me (i cleaned my glasses :p)

@nerzhul nerzhul merged commit 1d055aa into minetest:master Aug 18, 2017
osjc pushed a commit to osjc/minetest that referenced this pull request Jan 11, 2019
osjc pushed a commit to osjc/minetest that referenced this pull request Jan 23, 2019
@adrido adrido deleted the Add-missing-ctime-include branch February 27, 2019 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues that were confirmed to be a bug >= Two approvals ✅ ✅ Windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants