[clazy] [Bug 359306] new check idea: avoid using Q_FOREACH to iterate over QMap/QHash keys and retrieve values inside loop

[Sergio Martins via KDE Bugzilla]

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

Sergio Martins <smart...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED

--- Comment #1 from Sergio Martins <smart...@kde.org> ---
The container-anti-pattern will warn when you iterate on keys(), which you
should fix with:

Q_FOREACH( const QString& value, map) {}, or using iterator if you need both
value and key.
There might be corner-cases where people are doing something very stupid, but
it's difficult to make a check for more imaginative cases, so I consider this
done

-- 
You are receiving this mail because:
You are watching all bug changes.