cstylecasts and suppressions

diff --git a/src/hwlm/hwlm_dump.cpp b/src/hwlm/hwlm_dump.cpp
index b3a681de290404b31f198a301dba427549769d1b..69463519068d9a8900fcddfe1f8a59083dfed7f1 100644 (file)
--- a/src/hwlm/hwlm_dump.cpp
+++ b/src/hwlm/hwlm_dump.cpp
@@ -53,9 +53,11 @@ void hwlmGenerateDumpFiles(const HWLM *h, const string &base) {
 
     switch (h->type) {
     case HWLM_ENGINE_NOOD:
+        // cppcheck-suppress cstyleCast
         noodPrintStats(reinterpret_cast<const noodTable *>(HWLM_C_DATA(h)), f);
         break;
     case HWLM_ENGINE_FDR:
+        // cppcheck-suppress cstyleCast
         fdrPrintStats(reinterpret_cast<const FDR *>(HWLM_C_DATA(h)), f);
         break;
     default:

diff --git a/src/nfa/goughdump.cpp b/src/nfa/goughdump.cpp
index 060d6046f7c488662138ffba2324b957de88f438..8717d06a009ce5662b1dde76d4cbf39a08c9154d 100644 (file)
--- a/src/nfa/goughdump.cpp
+++ b/src/nfa/goughdump.cpp
@@ -123,7 +123,7 @@ void describeNode(const NFA *n, const mcclellan *m, u16 i, FILE *f) {
 
     if (aux->accel_offset) {
         dumpAccelDot(f, i,
-          &((const gough_accel *)(reinterpret_cast<const char *>(m) + aux->accel_offset))->accel);
+          &(reinterpret_cast<const gough_accel *>(reinterpret_cast<const char *>(m) + aux->accel_offset))->accel);
     }
 
     if (aux->accept_eod) {

diff --git a/src/nfa/lbr_dump.cpp b/src/nfa/lbr_dump.cpp
index 813af084c8712293f73881d2073a2289bd588f1f..102b655c5d6b5c8f100099aa414904c1860bd72e 100644 (file)
--- a/src/nfa/lbr_dump.cpp
+++ b/src/nfa/lbr_dump.cpp
@@ -126,8 +126,8 @@ void nfaExecLbrTruf_dump(const NFA *nfa, const string &base) {
     const lbr_truf *lt = reinterpret_cast<const lbr_truf *>(getImplNfa(nfa));
     lbrDumpCommon(&lt->common, f);
 
-    CharReach cr = truffle2cr((const u8 *)(&lt->mask1),
-                              (const u8 *)(&lt->mask2));
+    CharReach cr = truffle2cr(reinterpret_cast<const u8 *>(&lt->mask1),
+                              reinterpret_cast<const u8 *>(&lt->mask2));
     fprintf(f, "TRUFFLE model, scanning for: %s (%zu chars)\n",
             describeClass(cr, 20, CC_OUT_TEXT).c_str(), cr.count());
     fprintf(f, "\n");

diff --git a/src/parser/Parser.rl b/src/parser/Parser.rl
index ae419ec84d57dcee0a402c0dee2327ea23a8877e..668f3778c53064330bdfff3ce05639e58e16d04e 100644 (file)
--- a/src/parser/Parser.rl
+++ b/src/parser/Parser.rl
@@ -237,6 +237,7 @@ unichar readUtf8CodePoint2c(const char *s) {
 
 static
 unichar readUtf8CodePoint3c(const char *s) {
+    // cppcheck-suppress cstyleCast
     auto *ts = (const u8 *)s;
     assert(ts[0] >= 0xe0 && ts[0] < 0xf0);
     assert(ts[1] >= 0x80 && ts[1] < 0xc0);
@@ -253,6 +254,7 @@ unichar readUtf8CodePoint3c(const char *s) {
 
 static
 unichar readUtf8CodePoint4c(const char *s) {
+    // cppcheck-suppress cstyleCast
     auto *ts = (const u8 *)s;
     assert(ts[0] >= 0xf0 && ts[0] < 0xf8);
     assert(ts[1] >= 0x80 && ts[1] < 0xc0);

diff --git a/unit/internal/uniform_ops.cpp b/unit/internal/uniform_ops.cpp
index 17a3f668f836f3b5f4cc33fc1fa1a30634e60894..31f75e2db5556f0d18926cb61997e6eda070c6a2 100644 (file)
--- a/unit/internal/uniform_ops.cpp
+++ b/unit/internal/uniform_ops.cpp
@@ -52,6 +52,7 @@ TEST(Uniform, loadstore_u8) {
     for (int i = 0; i < 8; i++) {
         u8 in = 1 << i;
         const char *cin = reinterpret_cast<const char *>(&in);
+        // cppcheck-suppress cstyleCast
         u8 out = load_u8(cin);
         EXPECT_EQ(in, out);
         char ALIGN_DIRECTIVE stored[1];
@@ -64,6 +65,7 @@ TEST(Uniform, loadstore_u16) {
     for (int i = 0; i < 16; i++) {
         u16 in = 1 << i;
         const char *cin = reinterpret_cast<const char *>(&in);
+        // cppcheck-suppress cstyleCast
         u16 out = load_u16(cin);
         EXPECT_EQ(in, out);
         void *stored = aligned_zmalloc(2);
@@ -77,6 +79,7 @@ TEST(Uniform, loadstore_u32) {
     for (int i = 0; i < 32; i++) {
         u32 in = 1U << i;
         const char *cin = reinterpret_cast<const char *>(&in);
+        // cppcheck-suppress cstyleCast
         u32 out = load_u32(cin);
         EXPECT_EQ(in, out);
         void *stored = aligned_zmalloc(32/8);
@@ -90,6 +93,7 @@ TEST(Uniform, loadstore_u64a) {
     for (int i = 0; i < 64; i++) {
         u64a in = 1ULL << i;
         const char *cin = reinterpret_cast<const char *>(&in);
+        // cppcheck-suppress cstyleCast
         u64a out = load_u64a(cin);
         EXPECT_EQ(in, out);
         void *stored = aligned_zmalloc(64/8);