Removed some of the more ridiculous casts added after trying -Wconversion

Author: saxbophone

sxbp/begin_figure.c

@@ -55,7 +55,7 @@ static sxbp_direction_t sxbp_change_line_direction(
     sxbp_direction_t current,
     sxbp_rotation_t turn
 ) {
-    return (sxbp_direction_t)((current + (sxbp_direction_t)turn) % 4);
+    return (current + (sxbp_direction_t)turn) % 4;
 }
 
 /*
@@ -72,13 +72,13 @@ static sxbp_length_t sxbp_next_length(
     assert(direction <= SXBP_LEFT);
     switch (direction % 4u) {
         case SXBP_UP:
-            return (sxbp_length_t)labs(bounds.y_max - location.y) + 1;
+            return labs(bounds.y_max - location.y) + 1;
         case SXBP_RIGHT:
-            return (sxbp_length_t)labs(bounds.x_max - location.x) + 1;
+            return labs(bounds.x_max - location.x) + 1;
         case SXBP_DOWN:
-            return (sxbp_length_t)labs(bounds.y_min - location.y) + 1;
+            return labs(bounds.y_min - location.y) + 1;
         case SXBP_LEFT:
-            return (sxbp_length_t)labs(bounds.x_min - location.x) + 1;
+            return labs(bounds.x_min - location.x) + 1;
         default:
             /*
              * NOTE: this case should never happen
@@ -158,7 +158,7 @@ sxbp_result_t sxbp_begin_figure(
          * the number of lines is the number of bits in the buffer
          * (byte count * 8) + 1 (for the extra starting line)
          */
-        figure->size = (sxbp_figure_size_t)(data->size * 8 + 1);
+        figure->size = data->size * 8 + 1;
         // use default options if `options` is `NULL`
         if (options == NULL) {
             options = &SXBP_BEGIN_FIGURE_OPTIONS_DEFAULT;

sxbp/serialisation.c

@@ -130,7 +130,7 @@ static void sxbp_write_sxbp_data_line(
 ) {
     size_t index = *start_index;
     // the leading 2 bits of the first byte are used for the direction
-    buffer->bytes[index] = (uint8_t)(line.direction << 6u);
+    buffer->bytes[index] = line.direction << 6u;
     // the next 6 bits of the first byte are used for the first 6 bits of length
     buffer->bytes[index] |= (line.length >> 24);
     // move on to the next byte
@@ -256,7 +256,7 @@ static void sxbp_read_sxbp_data_line(
      * handle first byte on it's own as we only need least 6 bits of it
      * bit mask and shift 3 bytes to left
      */
-    line->length = (sxbp_length_t)((buffer->bytes[index] & 0x3f) << 24); // 0x3f = 0b00111111
+    line->length = (buffer->bytes[index] & 0x3f) << 24; // 0x3f = 0b00111111
     // next byte
     index++;
     // handle remaining 3 bytes in loop

sxbp/sxbp_internal.c

@@ -78,7 +78,7 @@ sxbp_bounds_t sxbp_get_bounds(const sxbp_figure_t* figure, size_t scale) {
         sxbp_move_location(
             &location,
             figure->lines[i].direction,
-            (sxbp_length_t)(figure->lines[i].length * scale)
+            figure->lines[i].length * scale
         );
         // update the bounds
         sxbp_update_bounds(location, &bounds);