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

Harald Sitter <sit...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/netw
                   |                            |ork/kio-extras/commit/07d54
                   |                            |af0b6e3f074a9e679db506fc538
                   |                            |f52cc410
         Resolution|---                         |FIXED
             Status|REPORTED                    |RESOLVED

--- Comment #2 from Harald Sitter <sit...@kde.org> ---
Git commit 07d54af0b6e3f074a9e679db506fc538f52cc410 by Harald Sitter.
Committed on 15/07/2021 at 14:01.
Pushed by sitter into branch 'release/21.08'.

don't auto deduce QStringBuilder

see 'auto QString(Builder) considered VERY HARMFUL' thread on kde-devel
https://markmail.org/message/mbrp5ypyhcfk7zjn

What happens is that the type of str ends up being QStringBuilder
instead of QString. The QStringBuilder is destroyed after the semicolon,
and all access to "str" produces undefined behavior.

Never use auto for QString concatenation!

M  +1    -1    activities/fileitemplugin/FileItemLinkingPluginActionLoader.cpp

https://invent.kde.org/network/kio-extras/commit/07d54af0b6e3f074a9e679db506fc538f52cc410

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

Reply via email to