| Summary: | Support QOverload<>::of in fix-old-style-connect | ||
|---|---|---|---|
| Product: | [Developer tools] clazy | Reporter: | Alexander <ashaduri> |
| Component: | general | Assignee: | Unassigned bugs <unassigned-bugs-null> |
| Status: | CONFIRMED --- | ||
| Severity: | normal | CC: | alexander.lohnau, smartins, woebbeking |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
I don't have much time now, so don't hold your breath. But it's a good suggestion Would be really nice to have. Though nowadays most of those signals are un-overloaded. Even when using Qt5, one can disable the overloads when disabling deprecated API. |
Hi, Please support QOverload<> syntax in fix-old-style-connect. Right now half of my connect() calls fail to convert to the new syntax because clazy complains somethink like: FixIt failed, requires manual intervention: Too many overloads (2) for method currentIndexChanged for record QComboBox [-Wclazy-old-style-connect] This is very common for QComboBox, QSpinBox, etc... Qt docs show how to do it: http://doc.qt.io/qt-5/qtglobal.html#qOverload Ideally, this should be configurable, to choose between "qOverload<>" (C++14), "QOverload<>::of" (C++11), or "static_cast<void (QComboBox::*)(int)>" (Qt 5.6 and older). Thanks