Merge pull request #6 from Gadgetoid/feature/ci

CI: Continuous integration and build fixes.

.github/workflows/cmake.yml

@@ -0,0 +1,21 @@
+name: CMake
+
+on:
+  push:
+  pull_request:
+
+env:
+  # Customize the CMake build type here (Release, Debug, RelWithDebInfo, etc.)
+  BUILD_TYPE: Release
+
+jobs:
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        submodules: true
+
+    - name: Build
+      run: ci/build.sh

CMakeLists.txt

@@ -9,4 +9,6 @@ include_directories(
   .
 )
 
+set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Werror")
+
 add_subdirectory(examples)

ci/build.sh

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+cmake -S . -B build
+cmake --build build -j

examples/c/character.cmake

@@ -1,3 +1,6 @@
 add_executable(
   character
-  character.c)
+  character.c
+)
+
+target_link_libraries(character m)

examples/c/geometry.cmake

@@ -1,3 +1,6 @@
 add_executable(
   geometry
-  geometry.c)
+  geometry.c
+)
+
+target_link_libraries(geometry m)

examples/c/logo.c

@@ -322,7 +322,7 @@ int main() {
   }
 
   uint64_t end = time_ms();
-  printf("render time: %llums", end - start);
+  printf("render time: %lums", end - start);
 
 
 

examples/c/logo.cmake

@@ -2,3 +2,5 @@ add_executable(
   logo
   logo.c
 )
+
+target_link_libraries(logo m)

pretty-poly.h

@@ -274,8 +274,7 @@ pp_rect_t pp_polygon_bounds(pp_poly_t *p) {
 // buffer that each tile is rendered into before callback
 // allocate one extra byte to allow a small optimization in the row renderer
 const uint32_t tile_buffer_size = PP_TILE_BUFFER_SIZE;
-//uint8_t tile_buffer[tile_buffer_size + 1];
-uint8_t tile_buffer[tile_buffer_size + 1];
+uint8_t tile_buffer[PP_TILE_BUFFER_SIZE + 1];
 
 // polygon node buffer handles at most 16 line intersections per scanline
 // is this enough for cjk/emoji? (requires a 2kB buffer)