[clazy] [Bug 355682] new check: test QList.count() must be replaced by !QList.isEmpty()

Stephane MANKOWSKI via KDE Bugzilla bugzilla_noreply at kde.org
Tue Dec 8 19:10:20 GMT 2015


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

--- Comment #2 from Stephane MANKOWSKI <stephane at mankowski.fr> ---
Hi,

Thank you, this helps me a lot.

Just one more comment: could be interesting to generate warning for the 2
followings cases too !
    if (list.count()>0) {} 
    if (list.count()==0) {}

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



More information about the Unassigned-bugs mailing list