Skip to content

Feature/project update and refactor #135

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.

Sign up for GitHub

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

Merged
merged 9 commits into from
Mar 22, 2020
Merged

Feature/project update and refactor #135

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
35f72dc
Update googletest submodule to release 1.8.1
Liastre Mar 21, 2020
e2b319a
Add status section for compiler configuration
Liastre Mar 21, 2020
ccb0df3
Add gitignore section
Liastre Mar 21, 2020
6a8e550
Fix compiler flags for modern compilers
Liastre Mar 21, 2020
6b44b8d
Fix path doc description
Liastre Mar 21, 2020
b6737d8
Fix recursion on some compilers due to wrong copy_part method used
Liastre Mar 21, 2020
0046c4f
Fix clang-format, remove useless states, update style according to us…
Liastre Mar 21, 2020
cc58c81
Apply codestyle to all files
Liastre Mar 21, 2020
f2f44d7
Apply codestyle to leftovers sources
Liastre Mar 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 3 additions & 5 deletions .clang-format
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
BasedOnStyle: Google
AccessModifierOffset: -1
ConstructorInitializerIndentWidth: 4
AlignEscapedNewlinesLeft: true
AlignTrailingComments: true
AllowAllParametersOfDeclarationOnNextLine: true
AllowShortIfStatementsOnASingleLine: true
Expand All @@ -13,7 +12,7 @@ BreakConstructorInitializersBeforeComma: false
BinPackParameters: true
ColumnLimit: 80
ConstructorInitializerAllOnOneLineOrOnePerLine: true
DerivePointerBinding: true
DerivePointerAlignment: false
ExperimentalAutoDetectBinPacking: false
IndentCaseLabels: true
MaxEmptyLinesToKeep: 1
Expand All @@ -23,16 +22,15 @@ PenaltyBreakString: 1000
PenaltyBreakFirstLessLess: 120
PenaltyExcessCharacter: 1000000
PenaltyReturnTypeOnItsOwnLine: 200
PointerBindsToType: true
PointerAlignment: Right
SpacesBeforeTrailingComments: 2
Cpp11BracedListStyle: true
Standard: Auto
IndentWidth: 2
TabWidth: 4
UseTab: false
BreakBeforeBraces: Attach
IndentFunctionDeclarationAfterType: true
SortIncludes: false
SpacesInParentheses: false
SpaceInEmptyParentheses: false
SpacesInCStyleCastParentheses: false
SpaceAfterControlStatementKeyword: true
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# Build artifacts
_build
cmake-build-*
.idea
6 changes: 4 additions & 2 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ find_package(Threads REQUIRED)

set(CMAKE_VERBOSE_MAKEFILE true)

message(STATUS "Configure compiler")
message("Using ${CMAKE_CXX_COMPILER_ID}")
if (${CMAKE_CXX_COMPILER_ID} MATCHES GNU)
INCLUDE(CheckCXXCompilerFlag)
CHECK_CXX_COMPILER_FLAG(-std=c++11 HAVE_STD11)
Expand All @@ -33,7 +35,7 @@ if (${CMAKE_CXX_COMPILER_ID} MATCHES GNU)
endif()

if (Uri_WARNINGS_AS_ERRORS)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror -Wno-parentheses")
endif()

message("C++ Flags: ${CMAKE_CXX_FLAGS} link flags: ${CMAKE_CXX_LINK_FLAGS}")
Expand All @@ -49,7 +51,7 @@ elseif(${CMAKE_CXX_COMPILER_ID} MATCHES Clang)
endif()

if (Uri_WARNINGS_AS_ERRORS)
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror -Wno-parentheses")
endif()

message("C++ Flags: ${CMAKE_CXX_FLAGS} link flags: ${CMAKE_CXX_LINK_FLAGS}")
Expand Down
2 changes: 1 addition & 1 deletion deps/googletest
Open in desktop
Submodule googletest updated 336 files
Loading