Skip to content

Commit

Permalink
Enable C++11 by default
Browse files Browse the repository at this point in the history
  • Loading branch information
m-kuhn committed Oct 20, 2016
1 parent 8064148 commit a6a4f2e
Showing 1 changed file with 2 additions and 17 deletions.
19 changes: 2 additions & 17 deletions CMakeLists.txt
Expand Up @@ -346,23 +346,8 @@ FIND_PROGRAM(QT_LRELEASE_EXECUTABLE
# or definitely Apple LLVM 5.0 (based on LLVM 3.3svn, in Xcode 5+):
# https://gist.github.com/yamaya/2924292

IF (CMAKE_CXX_COMPILER_ID MATCHES "GNU")
EXECUTE_PROCESS(COMMAND ${CMAKE_CXX_COMPILER} -dumpversion OUTPUT_VARIABLE GCC_VERSION)
IF (GCC_VERSION VERSION_GREATER 4.7 OR GCC_VERSION VERSION_EQUAL 4.7)
SET(USE_CXX_11 TRUE)
SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
ENDIF()
ELSEIF (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
IF ((NOT APPLE AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER "3.2")
OR (APPLE AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER "4.1"))
SET(USE_CXX_11 TRUE)
SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wno-error=c++11-narrowing")
ENDIF()
ELSEIF (MSVC AND MSVC_VERSION GREATER 1600)
SET(USE_CXX_11 TRUE)
ELSE()
SET(USE_CXX_11 FALSE)
ENDIF()
SET(CMAKE_CXX_STANDARD 11)
SET(USE_CXX_11 TRUE)

#allow override keyword if available
IF(NOT USE_CXX_11)

This comment has been minimized.

Copy link
@3nids

3nids Oct 20, 2016

Member

I suppose this can be removed too?

This comment has been minimized.

Copy link
@m-kuhn

m-kuhn Oct 20, 2016

Author Member

done, thx

Expand Down

1 comment on commit a6a4f2e

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay!

Please sign in to comment.