fix: mino position, do the same as upstream

it is stored internally as i8, but used and exposed as u8, at least in the recordings!

see OpenBrickProtocolFoundation/oopetris#204

Author: Totto16

src/c_wrapper/wrapper.cpp

@@ -373,7 +373,7 @@ static OOPetrisTetrionRecord record_to_c(const recorder::Record& record) {
 }
 
 static OOPetrisMino mino_to_c(const Mino& mino) {
-    auto orig_pos = mino.position();
+    auto orig_pos = mino.position().cast<uint8_t>();
 
     auto position = OOPetrisMinoPosition{ .x = orig_pos.x, .y = orig_pos.y };
 
@@ -963,7 +963,8 @@ char* oopetris_write_to_file(OOPetrisRecordingInformation* information, const ch
 
         for (std::size_t j = 0; j < stbds_arrlenu(snapshot.mino_stack); ++j) {
             const auto mino = snapshot.mino_stack[j];
-            mino_stack.set({ mino.position.x, mino.position.y }, static_cast<helper::TetrominoType>(mino.type));
+            const auto min_pos = shapes::AbstractPoint<u8>{ mino.position.x, mino.position.y };
+            mino_stack.set(min_pos.cast<i8>(), static_cast<helper::TetrominoType>(mino.type));
         }
 
         auto core_information = std::make_unique<TetrionCoreInformation>(

src/c_wrapper/wrapper.h

@@ -147,8 +147,8 @@ typedef struct {
 } OOPetrisTetrionRecord;
 
 typedef struct {
-    int8_t x;
-    int8_t y;
+    uint8_t x;
+    uint8_t y;
 } OOPetrisMinoPosition;