Re: New santizer warning in KF 5.98 headers

2023-01-11 Thread Michael Reeves
That looks a lot better. Weird magic numbers are what enums are ment to avoid. On Wed, Jan 11, 2023 at 7:26 AM Nicolas Fella wrote: > On 1/10/23 22:49, Michael Reeves wrote: > > /usr/include/KF5/KConfigWidgets/kstandardaction.h:261:64: runtime > > error: load of value 4294967295, which is not a

Re: New santizer warning in KF 5.98 headers

2023-01-11 Thread Nicolas Fella
On 1/10/23 22:49, Michael Reeves wrote: /usr/include/KF5/KConfigWidgets/kstandardaction.h:261:64: runtime error: load of value 4294967295, which is not a valid value for type 'Qt::ConnectionType' SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /usr/include/KF5/KConfigWidgets/kstandardact

Re: New santizer warning in KF 5.98 headers

2023-01-11 Thread Milian Wolff
On Mittwoch, 11. Januar 2023 11:26:42 CET Ingo Klöcker wrote: > 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

Re: New santizer warning in KF 5.98 headers

2023-01-11 Thread Ingo Klöcker
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 des

Re: New santizer warning in KF 5.98 headers

2023-01-11 Thread Milian Wolff
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