From b3cdb38c0b2ebab71a75b4fe8329b2084313b8d9 Mon Sep 17 00:00:00 2001
From: Evgenii Zheltonozhskii <zheltonozhskiy@gmail.com>
Date: Tue, 5 Jun 2018 19:10:28 +0300
Subject: [PATCH] Add clang-format target to CMake and Travis

---
 .clang-format                     | 43 +++++++++++++++++++++++++++++++
 .travis/build.sh                  |  3 +++
 .travis/os_build.sh               |  4 +--
 CMakeLists.txt                    |  3 +++
 modules/clang-cxx-dev-tools.cmake | 43 +++++++++++++++++++++++++++++++
 5 files changed, 94 insertions(+), 2 deletions(-)
 create mode 100644 .clang-format
 create mode 100644 modules/clang-cxx-dev-tools.cmake

diff --git a/.clang-format b/.clang-format
new file mode 100644
index 000000000..a5fc00726
--- /dev/null
+++ b/.clang-format
@@ -0,0 +1,43 @@
+---
+AccessModifierOffset: -4
+AlignAfterOpenBracket: Align
+AlignConsecutiveAssignments: false
+AlignConsecutiveDeclarations: false
+AlignEscapedNewlinesLeft: false
+AlignEscapedNewlines: false
+AlignOperands: true
+AlignTrailingComments: false
+AllowShortBlocksOnASingleLine: false
+AllowShortCaseLabelsOnASingleLine: true
+AllowShortFunctionsOnASingleLine: Empty
+AllowShortIfStatementsOnASingleLine: true
+AllowShortLoopsOnASingleLine: true
+AlwaysBreakAfterDefinitionReturnType: None
+AlwaysBreakAfterReturnType: None
+AlwaysBreakBeforeMultilineStrings: true
+BinPackArguments: true
+BinPackParameters: true
+BreakBeforeBraces: Attach
+BreakBeforeTernaryOperators: false
+BreakConstructorInitializers: BeforeComma
+BreakStringLiterals: true
+ColumnLimit: 120
+ConstructorInitializerAllOnOneLineOrOnePerLine: false
+Cpp11BracedListStyle: true
+DerivePointerAlignment: true
+IndentCaseLabels: false
+IndentWidth: 4
+KeepEmptyLinesAtTheStartOfBlocks: false
+MaxEmptyLinesToKeep: 2
+NamespaceIndentation: None
+PointerAlignment: Right
+SortIncludes: true
+SpaceAfterCStyleCast: false
+SpaceBeforeAssignmentOperators: true
+SpaceBeforeParens: ControlStatements
+SpaceInEmptyParentheses: false
+Standard: Cpp11
+TabWidth: 4
+UseTab: ForIndentation
+ForEachMacros: [ for_const, foreach, Q_FOREACH, BOOST_FOREACH ]
+...
diff --git a/.travis/build.sh b/.travis/build.sh
index a91387fd5..5eff2fa6f 100755
--- a/.travis/build.sh
+++ b/.travis/build.sh
@@ -3,5 +3,8 @@
 set -x
 
 cmake -G Ninja -DCMAKE_BUILD_TYPE=$BUILD_TYPE $EXTRA_CMAKE_FLAGS -DBUILD_TESTING=ON .. || exit 1
+# grep returns number of items found. each change is enclosed into <replacement>
+# tag in the xml. Thus if no changes needed, 0 will be returned
+cmake --build . --target clang-format-ci -- -v | grep -c "<replacement " && exit 1
 cmake --build . -- -v || exit 1
 ASAN_OPTIONS=alloc_dealloc_mismatch=0 ctest . || exit 1
diff --git a/.travis/os_build.sh b/.travis/os_build.sh
index 48e83d297..b8fe0c17f 100755
--- a/.travis/os_build.sh
+++ b/.travis/os_build.sh
@@ -3,7 +3,7 @@
 set -x
 
 if [ "$TRAVIS_OS_NAME" == "linux" ]; then
-    docker run --rm -v $PWD:/repo -v ~/.travis:/travis berkus/docker-cpp-ci /bin/sh -c "cd /repo/_build_; conan install .. --build missing; /repo/.travis/build.sh"
+    docker run --rm -v $PWD:/repo -v ~/.travis:/travis berkus/docker-cpp-ci /bin/sh -c "cd /repo/_build_; conan install .. --build missing; /repo/.travis/build.sh" || exit 1
 fi
 
 if [ "$TRAVIS_OS_NAME" == "osx" ]; then
@@ -12,5 +12,5 @@ if [ "$TRAVIS_OS_NAME" == "osx" ]; then
     export CXX=clang++
     export EXTRA_CMAKE_FLAGS=-DCMAKE_PREFIX_PATH='/usr/local/opt/qt5/;/usr/local/opt/openal-soft'
     conan install -s compiler=apple-clang .. --build missing
-    ../.travis/build.sh
+    ../.travis/build.sh || exit 1
 fi
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 1f62e9d03..e0dbb3f47 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -73,3 +73,6 @@ if (CCACHE)
 endif()
 
 add_subdirectory(Telegram)
+
+# clang-format
+include(modules/clang-cxx-dev-tools.cmake)
diff --git a/modules/clang-cxx-dev-tools.cmake b/modules/clang-cxx-dev-tools.cmake
new file mode 100644
index 000000000..3190dd56f
--- /dev/null
+++ b/modules/clang-cxx-dev-tools.cmake
@@ -0,0 +1,43 @@
+# http://www.labri.fr/perso/fleury/posts/programming/using-clang-tidy-and-clang-format.html
+
+# Additional target to perform clang-format/clang-tidy run
+# Requires clang-format and clang-tidy
+file(GLOB_RECURSE
+        ALL_CXX_SOURCE_FILES
+        ${CMAKE_CURRENT_SOURCE_DIR}/Telegram/SourceFiles/*.cpp
+        ${CMAKE_CURRENT_SOURCE_DIR}/Telegram/SourceFiles/*.h
+        )
+set(clang-format "clang-format")
+
+# Adding clang-format target if executable is found
+find_program(CLANG_FORMAT "clang-format")
+if(CLANG_FORMAT)
+    add_custom_target(
+            clang-format
+            COMMAND clang-format
+            -i
+            -style=file
+            ${ALL_CXX_SOURCE_FILES}
+    )
+    add_custom_target(
+            clang-format-ci
+            COMMAND clang-format
+            -output-replacements-xml
+            -style=file
+            ${ALL_CXX_SOURCE_FILES}
+    )
+endif()
+
+# Adding clang-tidy target if executable is found
+find_program(CLANG_TIDY "clang-tidy")
+if(CLANG_TIDY)
+    add_custom_target(
+            clang-tidy
+            COMMAND /usr/bin/clang-tidy
+            ${ALL_CXX_SOURCE_FILES}
+            -config=''
+            --
+            -std=c++11
+            ${INCLUDE_DIRECTORIES}
+    )
+endif()