prepare 2.3.1 release

.circleci/config.yml

@@ -8,6 +8,7 @@ workflows:
   build_and_test_all:
     jobs:
       - build-test-linux
+      - build-test-linux-optimized
       - build-test-linux-32bit
       - build-test-osx
       - build-test-windows
@@ -33,6 +34,26 @@ jobs:
             cd build
             ctest -T memcheck
 
+  build-test-linux-optimized:
+    docker:
+      - image: ldcircleci/ld-c-sdk-ubuntu  # defined in sdks-ci-docker project
+      - image: redis
+    environment:
+      CTEST_OUTPUT_ON_FAILURE: 1
+    steps:
+      - checkout
+      - run:
+          name: Build
+          command: |
+            mkdir build && cd build
+            cmake -D CMAKE_BUILD_TYPE=Release -D REDIS_STORE=ON ..
+            cmake --build .
+      - run:
+          name: Unit test
+          command: |
+            cd build
+            make test
+
   build-test-linux-32bit:
     docker:
       - image: i386/ubuntu:18.04

CMakeLists.txt

@@ -86,9 +86,11 @@ else()
                 -pedantic
                 -Wall
                 -Wextra
+                -Werror
                 -Wstrict-prototypes
                 -Wmissing-prototypes
                 -Wmissing-declarations
+                -std=c89
     )
 endif(MSVC)
 

README.md

@@ -45,4 +45,3 @@ We encourage pull requests and other contributions from the community. Check out
     * [docs.launchdarkly.com](https://docs.launchdarkly.com/  "LaunchDarkly Documentation") for our documentation and SDK reference guides
     * [apidocs.launchdarkly.com](https://apidocs.launchdarkly.com/  "LaunchDarkly API Documentation") for our API documentation
     * [blog.launchdarkly.com](https://blog.launchdarkly.com/  "LaunchDarkly Blog Documentation") for the latest product updates
-    * [Feature Flagging Guide](https://github.com/launchdarkly/featureflags/  "Feature Flagging Guide") for best practices and strategies

c-sdk-common/.circleci/config.yml

@@ -1,10 +1,16 @@
 version: 2.1
 
+orbs:
+   win: circleci/windows@1.0.0
+
 workflows:
   version: 2
   build_and_test_all:
     jobs:
       - build-test-linux
+      - build-test-osx
+      - build-test-windows
+
 jobs:
   build-test-linux:
     docker:
@@ -13,8 +19,39 @@ jobs:
       - checkout
       - run:
           name: Build
-          command: |
-            mkdir -p build
-            cd build
-            cmake ..
-            make
+          command: mkdir -p build && cd build && cmake .. && make
+      - run:
+          name: Test
+          command: cd build && make test
+
+  build-test-osx:
+    macos:
+      xcode: "9.4.1"
+    environment:
+      CTEST_OUTPUT_ON_FAILURE: 1
+    steps:
+      - checkout
+      - run:
+          name: Install prerequisites
+          command: HOMEBREW_NO_AUTO_UPDATE=1 brew install cmake
+      - run:
+          name: Build
+          command: mkdir -p build && cd build && cmake .. && make
+      - run:
+          name: Test
+          command: cd build && make test
+
+  build-test-windows:
+    executor:
+      name: win/vs2019
+      shell: powershell.exe
+    environment:
+      CTEST_OUTPUT_ON_FAILURE: 1
+    steps:
+      - checkout
+      - run:
+          name: build
+          command: ./scripts/build.ps1
+      - run:
+          name: Test
+          command: ./scripts/test.ps1

c-sdk-common/.clang-format

@@ -0,0 +1,29 @@
+---
+ColumnLimit: 80
+IndentWidth: 4
+
+AllowShortBlocksOnASingleLine: Never
+AllowShortFunctionsOnASingleLine: false
+AllowShortLoopsOnASingleLine: false
+AllowShortIfStatementsOnASingleLine: Never
+
+BinPackArguments: false
+BinPackParameters: false 
+
+AlwaysBreakBeforeMultilineStrings: true
+AlwaysBreakAfterReturnType: All
+
+AlignConsecutiveAssignments: true
+AlignAfterOpenBracket: AlwaysBreak
+AlignConsecutiveDeclarations: true
+AllowAllArgumentsOnNextLine: true
+AllowAllParametersOfDeclarationOnNextLine: true
+
+BreakStringLiterals: true
+BreakBeforeBraces: Custom
+BraceWrapping:
+  AfterEnum: true
+  AfterStruct: true
+  SplitEmptyFunction: true
+  AfterFunction: true
+  AfterControlStatement: MultiLine