[clazy] [Bug 407258] clazy-standalone/clang-tidy substitution
https://bugs.kde.org/show_bug.cgi?id=407258 --- Comment #2 from Christian Gagneraud --- Hi, Is there any plan to merge clazy in clang-tidy? I don't have any plan to work on the issue #407258 unfortunately. As a user of both tools, i find clang-tidy to be easier to use and integrate (eg CI), for example run-clang-tidy allows to run checks in parallel, something that not even clazy-standalone provides. IMHO, merging clazy in clang-tidy would make the user happier and will lessen the maintenance and improvement (catching up with clang-tid) burden. Chris -- You are receiving this mail because: You are watching all bug changes.
[clazy] [Bug 423780] New: clazy-fully-qualified-moc-types not detecting missing namespace
https://bugs.kde.org/show_bug.cgi?id=423780
Bug ID: 423780
Summary: clazy-fully-qualified-moc-types not detecting missing
namespace
Product: clazy
Version: unspecified
Platform: Other
OS: Linux
Status: REPORTED
Severity: normal
Priority: NOR
Component: general
Assignee: unassigned-b...@kde.org
Reporter: chg...@gmail.com
CC: smart...@kde.org
Target Milestone: ---
SUMMARY
Given an enum E in a class C in a namespace N, clazy reports an error when the
enum is used fully-unqualified (E value), but doesn't report an error if used
partially-qualified (C:E value).
The only valid usage should be "N:C:E value"
Or the documentation is not accurate, here is the given example:
namespace MyNameSpace {
struct MyType { (...) };
class MyObject : public QObject
{
Q_OBJECT
Q_PROPERTY(MyGadget myprop READ myprop); // Wrong, needs namespace
Q_SIGNALS:
void mySignal(MyType); // Wrong
void mySignal(MyNameSpace::MyType); // OK
};
}
STEPS TO REPRODUCE
run clazy on:
namespace N {
class C: QObject
{
Enum E {};
signals:
void function(C:E value) {}
}
}
OBSERVED RESULT
No error reported
EXPECTED RESULT
"signal arguments need to be fully-qualified" is emitted
SOFTWARE/OS VERSIONS
Windows:
macOS:
Linux/KDE Plasma:
(available in About System)
KDE Plasma Version:
KDE Frameworks Version:
Qt Version:
ADDITIONAL INFORMATION
Stock clazy on Ubuntu 18.04, based on LLVM-6.0.0
--
You are receiving this mail because:
You are watching all bug changes.
[clazy] [Bug 423640] New: Wrong diagnostic with multi-lines Q_PROPERTY
https://bugs.kde.org/show_bug.cgi?id=423640 Bug ID: 423640 Summary: Wrong diagnostic with multi-lines Q_PROPERTY Product: clazy Version: unspecified Platform: Other OS: Linux Status: REPORTED Severity: normal Priority: NOR Component: general Assignee: unassigned-b...@kde.org Reporter: chg...@gmail.com CC: smart...@kde.org Target Milestone: --- SUMMARY Wrong diagnostic with multi-lines Q_PROPERTY STEPS TO REPRODUCE 1. Use QtCreator with built in clazy 2. Multi-line Q_PROPERTY, eg: Q_PROPERTY(QString controllerFirmwareCrc READ controllerFirmwareCrc NOTIFY controllerFirmwareCrcChanged) OBSERVED RESULT /home/chgans/Projects/neon-factory-support/task/NTA-108-touchscreen-module/sources/Testing/Modules/TouchscreenModule.h:79:5: warning: Q_PROPERTY should have either NOTIFY or CONSTANT [-Wclazy-qproperty-without-notify] EXPECTED RESULT No issue reported SOFTWARE/OS VERSIONS Ubuntu 18.04, 64 bits Qt Version: 5.12 Unknown version of clazy ADDITIONAL INFORMATION -- You are receiving this mail because: You are watching all bug changes.
[clazy] [Bug 407258] New: clazy-standalone/clang-tidy substitution
https://bugs.kde.org/show_bug.cgi?id=407258 Bug ID: 407258 Summary: clazy-standalone/clang-tidy substitution Product: clazy Version: unspecified Platform: Other OS: Linux Status: REPORTED Severity: normal Priority: NOR Component: general Assignee: unassigned-b...@kde.org Reporter: chg...@gmail.com CC: smart...@kde.org Target Milestone: --- SUMMARY Currently there are cases where clazy-standalone cannot be substituted to clang-tidy. To make clazy-standalone command-line options par with clang-tidy, there is a need for: - Support for filter selection (eg, using `-` and `*`) - Support for config file [1] - Support for listing [1] at least these properties: - Checks - WarningsAsErrors - HeaderFilterRegex I haven't look at clang-tidy code, but it doesn't seem to be something over-complicated. Will investigate and report. -- You are receiving this mail because: You are watching all bug changes.
[clazy] [Bug 406645] yaml diagnostics export
https://bugs.kde.org/show_bug.cgi?id=406645 --- Comment #6 from Christian Gagneraud --- Thanks a lot for integrating my initial work. I'll surely report any issue as i start to use this new feature on our code base. -- You are receiving this mail because: You are watching all bug changes.
