fix: Replace C-style casts with C++ static/reinterpret casts

C-style casts should be replaced with appropriate C++ cast operators
for better type safety and code clarity. This change fixes cppcheck
style warnings about C-style pointer casting by using:
- static_cast for simple pointer type conversions
- reinterpret_cast for memory buffer pointer arithmetic

The changes maintain identical functionality while following modern
C++ best practices for explicit casting.

2 files changed, 33 insertions, 33 deletions

diff --git a/kms++util/src/colorbar.cpp b/kms++util/src/colorbar.cpp
index c727306..ec59806 100644
--- a/kms++util/src/colorbar.cpp
+++ b/kms++util/src/colorbar.cpp
@@ -39,7 +39,7 @@ static void drm_draw_color_bar_rgb888(IFramebuffer& buf, int old_xpos, int xpos,
 {
 	for (unsigned y = 0; y < buf.height(); ++y) {
 		RGB bcol = colors32[y * ARRAY_SIZE(colors32) / buf.height()];
-		uint32_t* line = (uint32_t*)(buf.map(0) + buf.stride(0) * y);
+		uint32_t* line = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y);
 
 		if (old_xpos >= 0) {
 			for (int x = old_xpos; x < old_xpos + width; ++x)
@@ -57,7 +57,7 @@ static void drm_draw_color_bar_rgb565(IFramebuffer& buf, int old_xpos, int xpos,
 
 	for (unsigned y = 0; y < buf.height(); ++y) {
 		uint16_t bcol = colors16[y * ARRAY_SIZE(colors16) / buf.height()];
-		uint16_t* line = (uint16_t*)(buf.map(0) + buf.stride(0) * y);
+		uint16_t* line = reinterpret_cast<uint16_t*>(buf.map(0) + buf.stride(0) * y);
 
 		if (old_xpos >= 0) {
 			for (int x = old_xpos; x < old_xpos + width; ++x)
@@ -85,7 +85,7 @@ static void drm_draw_color_bar_semiplanar_yuv(IFramebuffer& buf, int old_xpos, i
 
 	for (unsigned y = 0; y < buf.height(); ++y) {
 		unsigned int bcol = colors[y * ARRAY_SIZE(colors) / buf.height()];
-		uint8_t* line = (uint8_t*)(buf.map(0) + buf.stride(0) * y);
+		uint8_t* line = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * y);
 
 		if (old_xpos >= 0) {
 			for (int x = old_xpos; x < old_xpos + width; ++x)
diff --git a/kms++util/src/drawing.cpp b/kms++util/src/drawing.cpp
index 7e3ca97..df02485 100644
--- a/kms++util/src/drawing.cpp
+++ b/kms++util/src/drawing.cpp
@@ -17,49 +17,49 @@ void draw_rgb_pixel(IFramebuffer& buf, unsigned x, unsigned y, RGB color)
 	switch (buf.format()) {
 	case PixelFormat::XRGB8888:
 	case PixelFormat::ARGB8888: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.argb8888();
 		break;
 	}
 	case PixelFormat::XBGR8888:
 	case PixelFormat::ABGR8888: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.abgr8888();
 		break;
 	}
 	case PixelFormat::RGBX8888:
 	case PixelFormat::RGBA8888: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.rgba8888();
 		break;
 	}
 	case PixelFormat::BGRX8888:
 	case PixelFormat::BGRA8888: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.bgra8888();
 		break;
 	}
 	case PixelFormat::XRGB2101010:
 	case PixelFormat::ARGB2101010: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.argb2101010();
 		break;
 	}
 	case PixelFormat::XBGR2101010:
 	case PixelFormat::ABGR2101010: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.abgr2101010();
 		break;
 	}
 	case PixelFormat::RGBX1010102:
 	case PixelFormat::RGBA1010102: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.rgba1010102();
 		break;
 	}
 	case PixelFormat::BGRX1010102:
 	case PixelFormat::BGRA1010102: {
-		uint32_t* p = (uint32_t*)(buf.map(0) + buf.stride(0) * y + x * 4);
+		uint32_t* p = reinterpret_cast<uint32_t*>(buf.map(0) + buf.stride(0) * y + x * 4);
 		*p = color.bgra1010102();
 		break;
 	}
@@ -78,29 +78,29 @@ void draw_rgb_pixel(IFramebuffer& buf, unsigned x, unsigned y, RGB color)
 		break;
 	}
 	case PixelFormat::RGB332: {
-		uint8_t* p = (uint8_t*)(buf.map(0) + buf.stride(0) * y + x);
+		uint8_t* p = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * y + x);
 		*p = color.rgb332();
 		break;
 	}
 	case PixelFormat::RGB565: {
-		uint16_t* p = (uint16_t*)(buf.map(0) + buf.stride(0) * y + x * 2);
+		uint16_t* p = reinterpret_cast<uint16_t*>(buf.map(0) + buf.stride(0) * y + x * 2);
 		*p = color.rgb565();
 		break;
 	}
 	case PixelFormat::BGR565: {
-		uint16_t* p = (uint16_t*)(buf.map(0) + buf.stride(0) * y + x * 2);
+		uint16_t* p = reinterpret_cast<uint16_t*>(buf.map(0) + buf.stride(0) * y + x * 2);
 		*p = color.bgr565();
 		break;
 	}
 	case PixelFormat::XRGB4444:
 	case PixelFormat::ARGB4444: {
-		uint16_t* p = (uint16_t*)(buf.map(0) + buf.stride(0) * y + x * 2);
+		uint16_t* p = reinterpret_cast<uint16_t*>(buf.map(0) + buf.stride(0) * y + x * 2);
 		*p = color.argb4444();
 		break;
 	}
 	case PixelFormat::XRGB1555:
 	case PixelFormat::ARGB1555: {
-		uint16_t* p = (uint16_t*)(buf.map(0) + buf.stride(0) * y + x * 2);
+		uint16_t* p = reinterpret_cast<uint16_t*>(buf.map(0) + buf.stride(0) * y + x * 2);
 		*p = color.argb1555();
 		break;
 	}
@@ -114,9 +114,9 @@ void draw_yuv444_pixel(IFramebuffer& buf, unsigned x, unsigned y, YUV yuv)
 	if (x >= buf.width() || y >= buf.height())
 		throw runtime_error("attempt to draw outside the buffer");
 
-	uint8_t* py = (uint8_t*)(buf.map(0) + buf.stride(0) * y + x);
-	uint8_t* pu = (uint8_t*)(buf.map(1) + buf.stride(1) * y + x);
-	uint8_t* pv = (uint8_t*)(buf.map(2) + buf.stride(2) * y + x);
+	uint8_t* py = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * y + x);
+	uint8_t* pu = reinterpret_cast<uint8_t*>(buf.map(1) + buf.stride(1) * y + x);
+	uint8_t* pv = reinterpret_cast<uint8_t*>(buf.map(2) + buf.stride(2) * y + x);
 
 	switch (buf.format()) {
 	case PixelFormat::YUV444:
@@ -139,7 +139,7 @@ void draw_yuv444_pixel(IFramebuffer& buf, unsigned x, unsigned y, YUV yuv)
 static void draw_yuv422_packed_macropixel(IFramebuffer& buf, unsigned x, unsigned y,
 					  YUV yuv1, YUV yuv2)
 {
-	uint8_t* p = (uint8_t*)(buf.map(0) + buf.stride(0) * y + x * 2);
+	uint8_t* p = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * y + x * 2);
 
 	uint8_t y0 = yuv1.y;
 	uint8_t y1 = yuv2.y;
@@ -184,7 +184,7 @@ static void draw_y2xx_packed_macropixel(IFramebuffer& buf, unsigned x, unsigned
 					YUV yuv1, YUV yuv2)
 {
 	const uint32_t macro_size = 4;
-	uint16_t* p = (uint16_t*)(buf.map(0) + buf.stride(0) * y + x * macro_size);
+	uint16_t* p = reinterpret_cast<uint16_t*>(buf.map(0) + buf.stride(0) * y + x * macro_size);
 
 	switch (buf.format()) {
 	case PixelFormat::Y210: {
@@ -239,8 +239,8 @@ static void draw_y2xx_packed_macropixel(IFramebuffer& buf, unsigned x, unsigned
 static void draw_yuv422_semiplanar_macropixel(IFramebuffer& buf, unsigned x, unsigned y,
 					      YUV yuv1, YUV yuv2)
 {
-	uint8_t* py = (uint8_t*)(buf.map(0) + buf.stride(0) * y + x);
-	uint8_t* puv = (uint8_t*)(buf.map(1) + buf.stride(1) * y + x);
+	uint8_t* py = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * y + x);
+	uint8_t* puv = reinterpret_cast<uint8_t*>(buf.map(1) + buf.stride(1) * y + x);
 
 	uint8_t y0 = yuv1.y;
 	uint8_t y1 = yuv2.y;
@@ -270,9 +270,9 @@ static void draw_yuv422_semiplanar_macropixel(IFramebuffer& buf, unsigned x, uns
 static void draw_yuv422_planar_macropixel(IFramebuffer& buf, unsigned x, unsigned y,
 					  YUV yuv1, YUV yuv2)
 {
-	uint8_t* py = (uint8_t*)(buf.map(0) + buf.stride(0) * y + x);
-	uint8_t* pu = (uint8_t*)(buf.map(1) + buf.stride(1) * y + x / 2);
-	uint8_t* pv = (uint8_t*)(buf.map(2) + buf.stride(2) * y + x / 2);
+	uint8_t* py = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * y + x);
+	uint8_t* pu = reinterpret_cast<uint8_t*>(buf.map(1) + buf.stride(1) * y + x / 2);
+	uint8_t* pv = reinterpret_cast<uint8_t*>(buf.map(2) + buf.stride(2) * y + x / 2);
 
 	uint8_t y0 = yuv1.y;
 	uint8_t y1 = yuv2.y;
@@ -338,10 +338,10 @@ void draw_yuv422_macropixel(IFramebuffer& buf, unsigned x, unsigned y, YUV yuv1,
 static void draw_yuv420_semiplanar_macropixel(IFramebuffer& buf, unsigned x, unsigned y,
 					      YUV yuv1, YUV yuv2, YUV yuv3, YUV yuv4)
 {
-	uint8_t* py1 = (uint8_t*)(buf.map(0) + buf.stride(0) * (y + 0) + x);
-	uint8_t* py2 = (uint8_t*)(buf.map(0) + buf.stride(0) * (y + 1) + x);
+	uint8_t* py1 = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * (y + 0) + x);
+	uint8_t* py2 = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * (y + 1) + x);
 
-	uint8_t* puv = (uint8_t*)(buf.map(1) + buf.stride(1) * (y / 2) + x);
+	uint8_t* puv = reinterpret_cast<uint8_t*>(buf.map(1) + buf.stride(1) * (y / 2) + x);
 
 	uint8_t y0 = yuv1.y;
 	uint8_t y1 = yuv2.y;
@@ -377,11 +377,11 @@ static void draw_yuv420_semiplanar_macropixel(IFramebuffer& buf, unsigned x, uns
 static void draw_yuv420_planar_macropixel(IFramebuffer& buf, unsigned x, unsigned y,
 					  YUV yuv1, YUV yuv2, YUV yuv3, YUV yuv4)
 {
-	uint8_t* py1 = (uint8_t*)(buf.map(0) + buf.stride(0) * (y + 0) + x);
-	uint8_t* py2 = (uint8_t*)(buf.map(0) + buf.stride(0) * (y + 1) + x);
+	uint8_t* py1 = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * (y + 0) + x);
+	uint8_t* py2 = reinterpret_cast<uint8_t*>(buf.map(0) + buf.stride(0) * (y + 1) + x);
 
-	uint8_t* pu = (uint8_t*)(buf.map(1) + buf.stride(1) * (y / 2) + x / 2);
-	uint8_t* pv = (uint8_t*)(buf.map(2) + buf.stride(2) * (y / 2) + x / 2);
+	uint8_t* pu = reinterpret_cast<uint8_t*>(buf.map(1) + buf.stride(1) * (y / 2) + x / 2);
+	uint8_t* pv = reinterpret_cast<uint8_t*>(buf.map(2) + buf.stride(2) * (y / 2) + x / 2);
 
 	uint8_t y0 = yuv1.y;
 	uint8_t y1 = yuv2.y;