ogt_vox: added ogt_warn for non fatal errors

Author: mgerhardy

src/ogt_vox.h

@@ -176,6 +176,7 @@
     eg.
         #include "my_assert.h"
         #define ogt_assert(condition, message_str)    my_assert(condition, message_str)
+        #define ogt_assert_warn(condition, message_str)      my_assert(condition, message_str)
 
         #define OGT_VOX_IMPLEMENTATION
         #include "path/to/ogt_vox.h"
@@ -515,6 +516,9 @@
 #ifndef ogt_assert
     #include <assert.h>
     #define ogt_assert(x, msg_str)      do { assert((x) && (msg_str)); } while(0)
+#endif
+#ifndef ogt_assert_warn
+    #define ogt_assert_warn(x, msg_str) ogt_assert(x, msg_str)
 #endif
     #include <stdlib.h>
     #include <string.h>
@@ -1282,7 +1286,7 @@
             }
             default:
             {
-                ogt_assert(0, "unhandled node type");
+                ogt_assert_warn(0, "unhandled node type");
             }
         }
     }
@@ -1577,8 +1581,9 @@
                             uint8_t y = packed_voxel_data[i * 4 + 1];
                             uint8_t z = packed_voxel_data[i * 4 + 2];
                             uint8_t color_index = packed_voxel_data[i * 4 + 3];
-                            ogt_assert(x < size_x && y < size_y && z < size_z, "invalid data in XYZI chunk");
-                            if(x < size_x && y < size_y && z < size_z ) {
+                            const bool inside_region = x < size_x && y < size_y && z < size_z;
+                            ogt_assert_warn(inside_region, "invalid data in XYZI chunk");
+                            if (inside_region) {
                                 voxel_data[(x * k_stride_x) + (y * k_stride_y) + (z * k_stride_z)] = color_index;
                             }
                         }
@@ -1618,8 +1623,8 @@
                     _vox_file_read_uint32(fp, &reserved_id);
                     _vox_file_read_uint32(fp, &layer_id);
                     _vox_file_read_uint32(fp, &num_frames);
-                    ogt_assert(reserved_id == UINT32_MAX, "unexpected values for reserved_id in nTRN chunk");
-                    ogt_assert(num_frames > 0, "must have at least 1 frame in nTRN chunk");
+                    ogt_assert_warn(reserved_id == UINT32_MAX, "unexpected values for reserved_id in nTRN chunk");
+                    ogt_assert_warn(num_frames > 0, "must have at least 1 frame in nTRN chunk");
 
                     // make space in misc_data array for the number of transforms we'll need for this node
                     ogt_vox_keyframe_transform* keyframes = misc_data.alloc_many<ogt_vox_keyframe_transform>(num_frames);
@@ -1733,7 +1738,7 @@
                     _vox_file_read_int32(fp, &layer_id);
                     _vox_file_read_dict(&dict, fp);
                     _vox_file_read_int32(fp, &reserved_id);
-                    ogt_assert(reserved_id == -1, "unexpected value for reserved_id in LAYR chunk");
+                    ogt_assert_warn(reserved_id == -1, "unexpected value for reserved_id in LAYR chunk");
 
                     layers.grow_to_fit_index(layer_id);
                     layers[layer_id].name   = NULL;
@@ -2056,8 +2061,8 @@
 
             if (g_progress_callback_func) {
                 // we indicate progress as 0.8f * amount of buffer read + 0.2f at end after processing
-                if (!g_progress_callback_func(0.8f*(float)(fp->offset)/(float)(fp->buffer_size), g_progress_callback_user_data))
-                {
+                const float progress = 0.8f * (float)(fp->offset) / (float)(fp->buffer_size);
+                if (!g_progress_callback_func(progress, g_progress_callback_user_data)) {
                     return 0;
                 }
             }
@@ -2483,10 +2488,10 @@
                 is_negative = f[i] < 0.0f ? true : false;
             }
             else {
-                ogt_assert(f[i] == 0.0f, "rotation vector should contain only 0.0f, 1.0f, or -1.0f");
+                ogt_assert_warn(f[i] == 0.0f, "rotation vector should contain only 0.0f, 1.0f, or -1.0f");
             }
         }
-        ogt_assert(out_index != 3, "rotation vector was all zeroes but it should be a cardinal axis vector");
+        ogt_assert_warn(out_index != 3, "rotation vector was all zeroes but it should be a cardinal axis vector");
         return is_negative;
     }
 
@@ -2499,7 +2504,7 @@
         bool row0_negative = _vox_get_vec3_rotation_bits(row0, row0_index);
         bool row1_negative = _vox_get_vec3_rotation_bits(row1, row1_index);
         bool row2_negative = _vox_get_vec3_rotation_bits(row2, row2_index);
-        ogt_assert(((1 << row0_index) | (1 << row1_index) | (1 << row2_index)) == 7, "non orthogonal rows found in transform"); // check that rows are orthogonal. There must be a non-zero entry in column 0, 1 and 2 across these 3 rows.
+        ogt_assert_warn(((1 << row0_index) | (1 << row1_index) | (1 << row2_index)) == 7, "non orthogonal rows found in transform"); // check that rows are orthogonal. There must be a non-zero entry in column 0, 1 and 2 across these 3 rows.
         return (row0_index) | (row1_index << 2) | (row0_negative ? 1 << 4 : 0) | (row1_negative ? 1 << 5 : 0) | (row2_negative ? 1 << 6 : 0);
     }
 
@@ -3054,7 +3059,7 @@
         // check that the buffer is not larger than the maximum file size, return nothing if would overflow
         if (fp->data.count > UINT32_MAX ||  (fp->data.count - offset_post_main_chunk) > UINT32_MAX)
         {
-            ogt_assert(0, "Generated file size exceeded 4GiB, which is too large for Magicavoxel to parse.");
+            ogt_assert_warn(0, "Generated file size exceeded 4GiB, which is too large for Magicavoxel to parse.");
             *buffer_size = 0;
             return NULL;  // note: fp will be freed in dtor on exit
         }