[clazy] [Bug 410714] clazy-detaching-temporary: Don't warn on detaching pointer values

2019-08-25 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=410714 Sergio Martins changed: What|Removed |Added Resolution|--- |NOT A BUG Status|REPORTED

[clazy] [Bug 410714] clazy-detaching-temporary: Don't warn on detaching pointer values

2019-08-08 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=410714 --- Comment #3 from Sergio Martins --- It's indeed safe, but the warning is not about safety, it's about an unneeded QList copy when detaching a temporary. The performance impact can vary from negligeable if the list is small and detach called just a

[clazy] [Bug 410714] clazy-detaching-temporary: Don't warn on detaching pointer values

2019-08-08 Thread Markus
https://bugs.kde.org/show_bug.cgi?id=410714 --- Comment #2 from Markus --- (In reply to Sergio Martins from comment #1) > That QList is not a pointer > > It's doing axes(Qt::Horizontal).first(), not axes(Qt::Horizontal)->first() This is of course correct, sorry for being unclear in my

[clazy] [Bug 410714] clazy-detaching-temporary: Don't warn on detaching pointer values

2019-08-08 Thread Sergio Martins
https://bugs.kde.org/show_bug.cgi?id=410714 --- Comment #1 from Sergio Martins --- That QList is not a pointer It's doing axes(Qt::Horizontal).first(), not axes(Qt::Horizontal)->first() -- You are receiving this mail because: You are watching all bug changes.