Feature/coding conventions

.clang-format

@@ -0,0 +1,30 @@
+BasedOnStyle: LLVM
+Language: Cpp
+IndentWidth: 4
+AllowShortIfStatementsOnASingleLine: false
+AlwaysBreakTemplateDeclarations: Yes
+BinPackArguments: false
+BinPackParameters: false
+ColumnLimit: 120
+DerivePointerAlignment: false
+KeepEmptyLinesAtTheStartOfBlocks: false
+PointerAlignment: Left
+SpacesBeforeTrailingComments: 1
+SortIncludes: true
+IncludeBlocks: Regroup
+IncludeCategories:
+#gtest headers
+  - Regex:            '^((<|")gtest/)'
+    Priority:         1
+#stdlib C++ headers
+  - Regex:            '^<([^.])*>'
+    Priority:         2
+#external headers
+  - Regex:            '^<([^.])*\.h>'
+    Priority:         3
+#celix C API
+  - Regex:            '^(<|")celix_.*'
+    Priority:         4
+#celix C++ API
+  - Regex:            '^(<|")celix/.*'
+    Priority:         5

CMakeLists.txt

@@ -70,7 +70,7 @@ if (ENABLE_TESTING)
 endif ()
 
 # Set C specific flags
-set(CMAKE_C_FLAGS "-D_GNU_SOURCE -std=gnu99 -fPIC ${CMAKE_C_FLAGS}")
+set(CMAKE_C_FLAGS "-D_GNU_SOURCE -std=c99 -fPIC ${CMAKE_C_FLAGS}")
 set(CMAKE_C_FLAGS "-Wall -Werror -Wformat -Wno-error=deprecated-declarations ${CMAKE_C_FLAGS}")
 
 # Set C++ specific flags

bundles/cxx_remote_services/admin/src/RemoteServiceAdmin.cc

@@ -18,6 +18,7 @@
  */
 
 #include "RemoteServiceAdmin.h"
+
 #include "celix/BundleContext.h"
 #include "celix/rsa/RemoteConstants.h"
 

documents/README.md

@@ -88,3 +88,4 @@ bundles contains binaries depending on the stdlibc++ library.
   * [Apache Celix Patterns](patterns.md)
 * [Apache Celix CMake Commands](cmake_commands)
 * [Apache Celix Sub Projects](subprojects.md)
+* [Apache Celix Coding Conventions Guide](development/README.md)