Bug 404327

Summary: support multi-line Q_PROPERTY
Product: [Developer tools] clazy Reporter: Milian Wolff <mail>
Component: generalAssignee: Unassigned bugs mailing-list <unassigned-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: chgans, smartins
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Milian Wolff 2019-02-14 08:54:00 UTC
clazy reports:
 Q_PROPERTY(int maxTrianglesPerTexel READ maxTrianglesPerTexel WRITE setMaxTrianglesPerTexel NOTIFY
=> Q_PROPERTY should have either NOTIFY or CONSTANT

the code is:

    Q_PROPERTY(int maxTrianglesPerTexel READ maxTrianglesPerTexel WRITE setMaxTrianglesPerTexel NOTIFY
                   maxTrianglesPerTexelChanged)
Comment 1 Milian Wolff 2019-02-14 08:54:09 UTC
clazy version: 1.5
clang version: 7.0.1
Comment 2 Sergio Martins 2019-02-15 19:15:19 UTC
Git commit b4ad7ee09aa44138d5df04bc69ac466da8d59aac by Sergio Martins.
Committed on 15/02/2019 at 19:15.
Pushed by smartins into branch 'master'.

qproperty-without-notify: Fix multi-line Q_PROPERTY

"NOTIFY" was being compared to "NOTIFY\n"

M  +2    -1    src/checks/level1/qproperty-without-notify.cpp
M  +5    -0    src/clazy_stl.h
M  +4    -0    tests/qproperty-without-notify/main.cpp

https://commits.kde.org/clazy/b4ad7ee09aa44138d5df04bc69ac466da8d59aac
Comment 3 Sergio Martins 2020-06-29 11:21:05 UTC
*** Bug 423640 has been marked as a duplicate of this bug. ***