diff options
| author | Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> | 2025-09-13 17:40:42 +0300 |
|---|---|---|
| committer | Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> | 2025-12-18 12:23:57 +0200 |
| commit | 7cc1bdd06e68ab81612e8feee2a1dedf0e392886 (patch) | |
| tree | 99415217574312a3ac7b22a8fcb84aa27ab156fa /kms++/src/dmabufframebuffer.cpp | |
| parent | 2efdd2583da9575242091bb53c57a311c3eacbc6 (diff) | |
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.
Diffstat (limited to 'kms++/src/dmabufframebuffer.cpp')
| -rw-r--r-- | kms++/src/dmabufframebuffer.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kms++/src/dmabufframebuffer.cpp b/kms++/src/dmabufframebuffer.cpp index 939c025..33d65f7 100644 --- a/kms++/src/dmabufframebuffer.cpp +++ b/kms++/src/dmabufframebuffer.cpp @@ -97,8 +97,8 @@ uint8_t* DmabufFramebuffer::map(unsigned plane) if (p.map) return p.map; - p.map = (uint8_t*)mmap(0, p.size, PROT_READ | PROT_WRITE, MAP_SHARED, - p.prime_fd, 0); + p.map = static_cast<uint8_t*>(mmap(0, p.size, PROT_READ | PROT_WRITE, MAP_SHARED, + p.prime_fd, 0)); if (p.map == MAP_FAILED) throw invalid_argument(string("mmap failed: ") + strerror(errno)); |