Skip to content

Add C++11 to time_stamp test#192

Merged
carstene1ns merged 2 commits intomasterfrom
unknown repository
Oct 18, 2016
Merged

Add C++11 to time_stamp test#192
carstene1ns merged 2 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Oct 17, 2016

No description provided.

@Ghabry
Copy link
Copy Markdown
Member

Ghabry commented Oct 18, 2016

The lines look fine but delegating this to @carstene1ns because I have no idea if this is the best way to do it or to place it.

@carstene1ns
Copy link
Copy Markdown
Member

Yeah, looks good to me.

Some history: wanted to initially solve this with AX_CXX_COMPILE_STDCXX(11), because gcc < 4.8 does not support -std=c++11 and gcc > 6 does not need this flag anymore, so it would be seamlessly enabled, if needed.

@carstene1ns carstene1ns merged commit c50dff8 into EasyRPG:master Oct 18, 2016
@ghost ghost deleted the issue_177 branch October 19, 2016 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants