Merge branch 'feature/grid_map_static_analysis' into 'master'

[grid_map] Enabled static analysis without code formatting on all packages GitOrigin-RevId: 78cf1b49f972725e9adc4faebb6f236f4255143e
BruceChanJianLe · Nov 18, 2020 · ea0759a · ea0759a
1 parent 8d0a3b4
commit ea0759a
Show file tree

Hide file tree

Showing 23 changed files with 154 additions and 13 deletions.
diff --git a/grid_map_core/CMakeLists.txt b/grid_map_core/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_core)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED)
 
@@ -146,4 +148,14 @@ if(CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_core/package.xml b/grid_map_core/package.xml
@@ -10,6 +10,7 @@
   <url type="bugtracker">http://github.com/anybotics/grid_map/issues</url>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>eigen</depend>
 <!--   <test_depend>cmake_code_coverage</test_depend> -->
   <test_depend>gtest</test_depend>

diff --git a/grid_map_costmap_2d/CMakeLists.txt b/grid_map_costmap_2d/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_costmap_2d)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED
   COMPONENTS
@@ -91,4 +93,14 @@ if(CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_costmap_2d/package.xml b/grid_map_costmap_2d/package.xml
@@ -10,6 +10,7 @@
   <url type="bugtracker">http://github.com/anybotics/grid_map/issues</url>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>grid_map_core</depend>
   <depend>costmap_2d</depend>
   <depend>tf</depend>

diff --git a/grid_map_cv/CMakeLists.txt b/grid_map_cv/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_cv)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED COMPONENTS
   grid_map_core
@@ -125,3 +127,12 @@ if(CATKIN_ENABLE_TESTING)
   endif()
 endif()
 
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_cv/package.xml b/grid_map_cv/package.xml
@@ -11,6 +11,7 @@
   <author email="[email protected]">Péter Fankhauser</author>
   <author email="[email protected]">Magnus Gärtner</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>grid_map_core</depend>
   <depend>cv_bridge</depend>
   <depend>filters</depend>

diff --git a/grid_map_demos/CMakeLists.txt b/grid_map_demos/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_demos)
 
 set(CMAKE_CXX_STANDARD 11)
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED COMPONENTS
   roscpp
@@ -265,4 +267,14 @@ if(CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_demos/package.xml b/grid_map_demos/package.xml
@@ -10,6 +10,7 @@
   <url type="bugtracker">http://github.com/anybotics/grid_map/issues</url>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>roscpp</depend>
   <depend>grid_map_core</depend>
   <depend>grid_map_ros</depend>

diff --git a/grid_map_filters/CMakeLists.txt b/grid_map_filters/CMakeLists.txt
@@ -10,6 +10,8 @@ set(CMAKE_CXX_STANDARD 11)
 # Therefore, we disable the finite-math-only flag that was set by fast-math.  
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -ffast-math -fno-finite-math-only")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 find_package(Boost REQUIRED)
 
 # Other possible options.
@@ -160,4 +162,14 @@ if (CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_filters/package.xml b/grid_map_filters/package.xml
@@ -11,6 +11,7 @@
   <author email="[email protected]">Péter Fankhauser</author>
   <author email="[email protected]">Martin Wermelinger</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>boost</depend>
   <depend>filters</depend>
   <depend>grid_map_core</depend>

diff --git a/grid_map_loader/CMakeLists.txt b/grid_map_loader/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_loader)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED COMPONENTS
   roscpp
@@ -95,4 +97,14 @@ if (CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_loader/package.xml b/grid_map_loader/package.xml
@@ -10,6 +10,7 @@
   <url type="bugtracker">http://github.com/anybotics/grid_map/issues</url>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>roscpp</depend>
   <depend>grid_map_ros</depend>
   <depend>grid_map_msgs</depend>

diff --git a/grid_map_octomap/CMakeLists.txt b/grid_map_octomap/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_octomap)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED
   COMPONENTS
@@ -121,4 +123,14 @@ if(CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_octomap/package.xml b/grid_map_octomap/package.xml
@@ -11,6 +11,7 @@
   <author email="[email protected]">Jeff Delmerico</author>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>grid_map_core</depend>
   <depend>octomap</depend>
 <!--   <test_depend>cmake_code_coverage</test_depend> -->

diff --git a/grid_map_pcl/CMakeLists.txt b/grid_map_pcl/CMakeLists.txt
@@ -3,6 +3,7 @@ project(grid_map_pcl)
 
 set(CMAKE_CXX_STANDARD 14)
 add_compile_options(-Wall -Wextra -Wpedantic)
+
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
 
 set(SRC_FILES
@@ -195,10 +196,9 @@ if(CATKIN_ENABLE_TESTING)
   endif()
 endif()
 
-###########
-## Clang ##
-###########
-
+#################
+## Clang_tools ##
+#################
 find_package(cmake_clang_tools QUIET)
 if(cmake_clang_tools_FOUND)
   add_default_clang_tooling()

diff --git a/grid_map_ros/CMakeLists.txt b/grid_map_ros/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_ros)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED COMPONENTS
   roscpp
@@ -133,3 +135,12 @@ if(CATKIN_ENABLE_TESTING)
   endif()
 endif()
 
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_ros/package.xml b/grid_map_ros/package.xml
@@ -10,6 +10,7 @@
   <url type="bugtracker">http://github.com/anybotics/grid_map/issues</url>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>roscpp</depend>
   <depend>grid_map_core</depend>
   <depend>grid_map_msgs</depend>

diff --git a/grid_map_rviz_plugin/CMakeLists.txt b/grid_map_rviz_plugin/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_rviz_plugin)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 find_package(catkin REQUIRED
   COMPONENTS
     rviz
@@ -145,4 +147,14 @@ if (CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_rviz_plugin/package.xml b/grid_map_rviz_plugin/package.xml
@@ -10,6 +10,7 @@
   <author>Péter Fankhauser</author>
 
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <build_depend>qtbase5-dev</build_depend>
   <build_depend>rviz</build_depend>
   <build_depend>grid_map_ros</build_depend>

diff --git a/grid_map_sdf/CMakeLists.txt b/grid_map_sdf/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_sdf)
 
 set(CMAKE_CXX_FLAGS "-std=c++14 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED COMPONENTS
   grid_map_core
@@ -102,4 +104,14 @@ if(CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_sdf/package.xml b/grid_map_sdf/package.xml
@@ -11,6 +11,7 @@
   <author email="[email protected]">Takahiro Miki</author>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>grid_map_core</depend>
   <depend>pcl_ros</depend>
 <!--   <test_depend>cmake_code_coverage</test_depend> -->

diff --git a/grid_map_visualization/CMakeLists.txt b/grid_map_visualization/CMakeLists.txt
@@ -3,6 +3,8 @@ project(grid_map_visualization)
 
 set(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 ## Find catkin macros and libraries
 find_package(catkin REQUIRED COMPONENTS
   roscpp
@@ -133,4 +135,14 @@ if (CATKIN_ENABLE_TESTING)
         ${PROJECT_NAME}-test
     )
   endif()
-endif()
+endif()
+
+#################
+## Clang_tools ##
+#################
+find_package(cmake_clang_tools QUIET)
+if(cmake_clang_tools_FOUND)
+  add_default_clang_tooling(
+    DISABLE_CLANG_FORMAT
+  )
+endif(cmake_clang_tools_FOUND)
diff --git a/grid_map_visualization/package.xml b/grid_map_visualization/package.xml
@@ -10,6 +10,7 @@
   <url type="bugtracker">http://github.com/anybotics/grid_map/issues</url>
   <author email="[email protected]">Péter Fankhauser</author>
   <buildtool_depend>catkin</buildtool_depend>
+<!--   <build_depend>cmake_clang_tools</build_depend> -->
   <depend>roscpp</depend>
   <depend>grid_map_core</depend>
   <depend>grid_map_ros</depend>