[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [clazy] [Bug 379342] clazy-function-args-by-value false positive when ref constructor arg is stored 
From:       Sergio Martins <bugzilla_noreply () kde ! org>
Date:       2017-09-18 21:40:36
Message-ID: bug-379342-17878-ki9DNnxbnU () http ! bugs ! kde ! org/
[Download RAW message or body]

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

Sergio Martins <smartins@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/cla
                   |                            |zy/51ba8ddfaffbe0d15e5165b4
                   |                            |d135d9a62749dd8f
         Resolution|---                         |FIXED

--- Comment #5 from Sergio Martins <smartins@kde.org> ---
Git commit 51ba8ddfaffbe0d15e5165b4d135d9a62749dd8f by Sergio Martins.
Committed on 18/09/2017 at 21:39.
Pushed by smartins into branch '1.2'.

function-args-by-value: Fix false positive

When passing to a ctor that inits members by ref we must preserve
the ref in the original param.

M  +23   -1    src/checks/level2/function-args-by-value.cpp
A  +12   -0    tests/function-args-by-value/bug379342.cpp     [License:
UNKNOWN]  *
A  +1    -0    tests/function-args-by-value/bug379342.cpp.expected
M  +4    -2    tests/function-args-by-value/config.json
M  +0    -6    tests/function-args-by-value/main.cpp

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


https://commits.kde.org/clazy/51ba8ddfaffbe0d15e5165b4d135d9a62749dd8f

-- 
You are receiving this mail because:
You are watching all bug changes.=
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic