Bug 382217

Created attachment 106545 [details]
Normalize all signals and slots

When signals/slots are parsed they are "normalized" if necessary. It is thus a slight performance improvement to use the "normal" format. See:
http://doc.qt.io/qt-4.8/signalsandslots.html#advanced-signals-and-slots-usage
https://marcmutz.wordpress.com/effective-qt/prefer-to-use-normalised-signalslot-signatures/

The patch was created by running the official normalization utility on all *cpp files:
https://github.com/qt/qtrepotools/tree/master/util/normalize

Simon,

I not using your patch, but is review all Krazy static analyzer reports where signal/slot normalization is not respected, and i fixed all (mostly more than 300 reports since 3 months - not only slots normalization).

http://ebn.kde.org/krazy/reports/extragear/graphics/digikam/

I close this file now.

Gilles Caulier

• • Home
  • | New
  • | Browse
  • | Search
  • |
    [?]
  • | Reports
  • | Requests
  • | Help
  • | New Account
  • | Log In
    [x]
  • | Forgot Password
    Login: [x]