[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: New santizer warning in KF 5.98 headers
From: Ingo =?ISO-8859-1?Q?Kl=F6cker?= <kloecker () kde ! org>
Date: 2023-01-11 10:26:42
Message-ID: 2271372.ElGaqSPkdT () daneel
[Download RAW message or body]
On Mittwoch, 11. Januar 2023 11:02:04 CET Milian Wolff wrote:
> On Dienstag, 10. Januar 2023 23:45:26 CET Michael Reeves wrote:
> > Thanks. I would say your right there this would definitely have caught
> > someone's attention if didn't work in practice with what kde needs.
> > Santizers are by design quite pedantic as serves there purpose well.
>
> I agree, the code is clearly wrong and it's unclear what it's trying to
> achieve here. Does anyone know what this is trying to do?
>
> Qt::ConnectionType connectionType = static_cast<Qt::ConnectionType>(-1)
>
> Should this maybe just be changed to use Qt::AutoConnection?
The code:
https://invent.kde.org/frameworks/kconfigwidgets/-/blob/master/src/
kstandardaction.h#L253
What the code does:
The default value `static_cast<Qt::ConnectionType>(-1)` serves as hint that
the function should decide what kind of connection type to use and for some
reason the `ConfigureToolbars` action explicitly needs to use a
`Qt::QueuedConnection` instead of a `Qt::AutoConnection`. This could be
changed to a std::optional (for KF6) to make the intention clear.
Why the code does what it does:
One could question whether this special casing for `ConfigureToolbars` is still
necessary. The bug report about the crash that this seems to have fixed is from
2009: https://bugs.kde.org/show_bug.cgi?id=200815
Regards,
Ingo
["signature.asc" (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic