[clazy] [Bug 378677] returning-void-expression false positive

2017-04-13 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #8 from oktal3...@gmail.com --- Of course it can not cover all cases, but it should at least cover one case. ;) Looks good now, thank you. -- You are receiving this mail because: You are watching all bug changes.

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-13 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #7 from Sergio Martins --- Strange, since I reproduced the bug on my system without needing to instantiate. Anyway, the test not covers all cases :) -- You are receiving this mail because: You are watching all bug

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-13 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #6 from Sergio Martins --- Git commit dcc9f740b83c1838f22981990016880295b62203 by Sergio Martins. Committed on 13/04/2017 at 18:18. Pushed by smartins into branch '1.1'. returning-void-expression: Add more unit-test

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-13 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #5 from oktal3...@gmail.com --- The unittest in the commit doesn't reproduce the bug on my system, as the function templates are not instantiated. Only after adding the following does it really test the bug: void test6() { test4();

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-13 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 Sergio Martins changed: What|Removed |Added Resolution|--- |FIXED Latest Commit|

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-13 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 Sergio Martins changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED Ever

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-12 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #3 from oktal3...@gmail.com --- I'm sorry, I jumped to a conclusion and gave you faulty information without first checking it thoroughly for myself. The false positive occurs when the function is constexpr and the return type is dependent on

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-12 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #2 from Sergio Martins --- Which clang version are you on ? I can't repro with 3.9. `-FunctionDecl 0x1f64cf0 line:1:5 test 'int (void)' `-CompoundStmt 0x1f64e70

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-12 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 --- Comment #1 from Sergio Martins --- Git commit 9bccb9a02a1ee0211e7e2b4cadd2d785a4ba792b by Sergio Martins. Committed on 12/04/2017 at 14:29. Pushed by smartins into branch '1.1'. Add unit-test for bug 378677 M +5-0

[clazy] [Bug 378677] returning-void-expression false positive

2017-04-12 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=378677 Sergio Martins changed: What|Removed |Added Assignee|unassigned-b...@kde.org |smart...@kde.org -- You