[clazy] [Bug 413169] New: inefficient-qlist fails on Linux x86 (32bit)

[Pino Toscano]

https://bugs.kde.org/show_bug.cgi?id=413169

            Bug ID: 413169
           Summary: inefficient-qlist fails on Linux x86 (32bit)
           Product: clazy
           Version: unspecified
          Platform: Other
                OS: Linux
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: general
          Assignee: unassigned-bugs at kde.org
          Reporter: pino at kde.org
                CC: smartins at kde.org
  Target Milestone: ---

SUMMARY
When testing clazy 1.6, the inefficient-qlist test fails when run on x86 32bit
Linux architectures (i.e. i386 in Debian).

STEPS TO REPRODUCE
1. build clazy on Linux x86
2. run the test suite, either from the build directory, or after installation

OBSERVED RESULT
Qt version: 51103
Qt headers: /usr/include/i386-linux-gnu/qt5
clazy-standalone old-style-connect/port_qsignalspy.cpp 
-export-fixes=old-style-connect/port_qsignalspy.cpp.clazy-standalone.yaml
-checks=old-style-connect  -- -Wno-unused-value -Qunused-arguments -std=c++14
-isystem /usr/include/i386-linux-gnu/qt5 -fPIC -L /usr/lib/i386-linux-gnu 
Running: clang -Xclang -load -Xclang ClazyPlugin.so -Xclang -add-plugin -Xclang
clazy  -Wno-unused-value -Qunused-arguments -std=c++14 -isystem
/usr/include/i386-linux-gnu/qt5 -fPIC -L /usr/lib/i386-linux-gnu -c  -Xclang
-plugin-arg-clazy -Xclang inefficient-qlist inefficient-qlist/main.cpp
output_file=inefficient-qlist/main.cpp.out
[FAIL] inefficient-qlist (plugin)
diff -Naur --strip-trailing-cr inefficient-qlist/main.cpp.expected
inefficient-qlist/main.cpp.result
--- inefficient-qlist/main.cpp.expected 2019-10-12 08:39:47.000000000 +0000
+++ inefficient-qlist/main.cpp.result   2019-10-18 14:14:56.093460474 +0000
@@ -1,4 +1,4 @@
 inefficient-qlist/main.cpp:24:5: warning: Use QVector instead of QList for
type with size 9 bytes [-Wclazy-inefficient-qlist]
 inefficient-qlist/main.cpp:38:11: warning: Use QVector instead of QList for
type with size 9 bytes [-Wclazy-inefficient-qlist]
 inefficient-qlist/main.cpp:40:5: warning: Use QVector instead of QList for
type with size 9 bytes [-Wclazy-inefficient-qlist]
-inefficient-qlist/main.cpp:53:5: warning: Use QVector instead of QList for
type with size 16 bytes [-Wclazy-inefficient-qlist]
+inefficient-qlist/main.cpp:53:5: warning: Use QVector instead of QList for
type with size 12 bytes [-Wclazy-inefficient-qlist]

Log:
https://buildd.debian.org/status/fetch.php?pkg=clazy&arch=i386&ver=1.6-1&stamp=1571408147&raw=1

EXPECTED RESULT
inefficient-qlist passes on any architecture.

SOFTWARE/OS VERSIONS
clazy 1.6

ADDITIONAL INFORMATION
Basically the test properly detects QList<QVariant> as inefficient, just that
the error message does not match the baseline due to the different size of
QVariant.

This was introduced with commit 948a23ae1b1553cf67dc8b523236cc0e7dc74994.

-- 
You are receiving this mail because:
You are the assignee for the bug.