[clazy] [Bug 487241] New: incorrect-emit confused by parentheses like emit(signal)

bugzilla_noreply at kde.org bugzilla_noreply at kde.org
Sun May 19 16:17:50 BST 2024


https://bugs.kde.org/show_bug.cgi?id=487241

            Bug ID: 487241
           Summary: incorrect-emit confused by parentheses like
                    emit(signal)
    Classification: Developer tools
           Product: clazy
           Version: unspecified
          Platform: Other
                OS: Linux
            Status: REPORTED
          Severity: minor
          Priority: NOR
         Component: general
          Assignee: unassigned-bugs at kde.org
          Reporter: foss at grueninger.de
                CC: smartins at kde.org
  Target Milestone: ---

SUMMARY
Clazy's incorrect-emit check gets confused by parentheses around the slot after
emit or Q_EMIT.

STEPS TO REPRODUCE
1. emit(slot); => warning
2. emit slot; => no warning
3. Kile has always paranthesis, e.g.
    kile/src/kileproject.h:213:16: warning: Missing emit keyword on signal call
KileProject::nameChanged [-Wclazy-incorrect-emit]
     213 |         Q_EMIT(nameChanged(name));
            |                ^



OBSERVED RESULT
Compiler warning
warning: Missing emit keyword on signal call KileProject::nameChanged
[-Wclazy-incorrect-emit]

EXPECTED RESULT
No compiler warning

SOFTWARE/OS VERSIONS
clazy version: 1.12 (Git master)
clang version: 18.1.5

-- 
You are receiving this mail because:
You are the assignee for the bug.


More information about the Unassigned-bugs mailing list