[clazy] [Bug 389055] suggest simplification: `$smartpointer.data()->` to `$smartpointer->`

Sergio Martins bugzilla_noreply at kde.org
Fri Jul 23 20:34:01 BST 2021


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

Sergio Martins <smartins at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/sdk/
                   |                            |clazy/commit/74686cc8c884db
                   |                            |cfabb12ea7a926e0eec486846d
             Status|CONFIRMED                   |RESOLVED

--- Comment #3 from Sergio Martins <smartins at kde.org> ---
Git commit 74686cc8c884dbcfabb12ea7a926e0eec486846d by Sergio Martins, on
behalf of Waqar Ahmed.
Committed on 23/07/2021 at 19:33.
Pushed by smartins into branch 'master'.

Add use-arrow-operator

Suggest to use arrow operator with qt smart pointers instead of
.data()->someFunc()

print parent class name in warning

Signed-off-by: Waqar Ahmed <waqar.17a at gmail.com>

M  +1    -0    CheckSources.cmake
M  +1    -0    ClazyTests.cmake
M  +7    -0    checks.json
M  +2    -0    src/Checks.h
A  +90   -0    src/checks/manuallevel/use-arrow-operator.cpp     [License: LGPL
(v2+)]
A  +39   -0    src/checks/manuallevel/use-arrow-operator.h     [License: LGPL
(v2+)]
A  +8    -0    tests/use-arrow-operator/config.json
A  +20   -0    tests/use-arrow-operator/main.cpp  *
A  +3    -0    tests/use-arrow-operator/main.cpp.expected

The files marked with a * at the end have a non valid license. Please read:
https://community.kde.org/Policies/Licensing_Policy and use the headers which
are listed at that page.


https://invent.kde.org/sdk/clazy/commit/74686cc8c884dbcfabb12ea7a926e0eec486846d

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


More information about the Unassigned-bugs mailing list