Bug 404327 - support multi-line Q_PROPERTY
Summary: support multi-line Q_PROPERTY
Status: RESOLVED FIXED
Alias: None
Product: clazy
Classification: Developer tools
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
: 423640 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-02-14 08:54 UTC by Milian Wolff
Modified: 2020-06-29 11:21 UTC (History)
2 users (show)

See Also:
Latest Commit: https://commits.kde.org/clazy/b4ad7ee09aa44138d5df04bc69ac466da8d59aac
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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. ***