Merge pull request #1 from Zer0-One/ubsan

Add ubsan to debug builds

.github/workflows/build_ship.yml

@@ -2,9 +2,9 @@ name: Build and Ship
 
 on:
   push:
-    branches: [ "master" ]
+    branches: [ "**" ]
   pull_request:
-    branches: [ "master" ]
+    branches: [ "**" ]
 
 jobs:
   build-linux:

CMakeLists.txt

@@ -82,7 +82,7 @@ set(CMAKE_C_FLAGS_RELEASE "-O2")
 if(WIN32)
     set(CMAKE_C_FLAGS_DEBUG "-Z7")
 else()
-    set(CMAKE_C_FLAGS_DEBUG "-O0 -g3 -fsanitize=address")
+    set(CMAKE_C_FLAGS_DEBUG "-O0 -g3 -fsanitize=address -fsanitize=undefined") 
 endif()
 
 # If tests are enabled, make tests

README.md

@@ -1,6 +1,6 @@
 # libtmj
 
-A library for loading Tiled maps in JSON format
+A library for loading [Tiled](https://www.mapeditor.org) maps in JSON format
 
 Bear in mind that this library is currently in beta and not yet stable.  If you
 find bugs, or find that the library is missing an important feature, please
@@ -98,7 +98,7 @@ void log_cb(tmj_log_priority priority, const char* msg){
     }
 }
 
-// Layers are oganized into a tree, so descend recursively
+// Layers are organized into a tree, so descend recursively
 void enumerate_layers(Layer* layers, size_t count){
     for(size_t i = 0; i < count; i++){
         printf("Layer %d: %s\n", layers[i].id, layers[i].name);

include/tmj.h

@@ -540,8 +540,8 @@ typedef enum TMJ_LOG_PRIORITY{
  *
  * @param debug If set to true, the given callback function will receive debug
  * messages and information in addition to the higher-priority messages.
- * @param callback A function that takes a LOG_PRIORITY and a char* and returns
- * nothing.
+ * @param callback A function that takes a TMJ_LOG_PRIORITY and a char* and
+ * returns nothing.
  */
 void tmj_log_regcb(bool debug, void (*callback)(tmj_log_priority, const char*));