Opened 7 years ago

Closed 7 years ago

#10303 closed Patch - Bug Fix (Fixed)

Proposed cppcheck error suppression

Reported by: gary.buhrmaster@… Owned by: danielk
Priority: trivial Milestone: unknown
Component: MythTV - General Version: Unspecified
Severity: low Keywords:
Cc: Ticket locked: no

Description

While cppcheck is a wonderful tool, it sometimes gets it wrong.

This proposed patch adds in the cppcheck hint to a code segment that says in the comments that cl is not leaking. If cl is not leaking, we do not want the cppcheck message.

For this patch to be useful, one will also need to change the invocation of cppcheck to include the "--inline-suppr" flag.

There are more cppcheck errors/warnings that should probably be suppressed if this approach is accepted.

Attachments (1)

cppcheck-suppress.patch (520 bytes) - added by gary.buhrmaster@… 7 years ago.
Proposed patch to suppress cppcheck error

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by gary.buhrmaster@…

Attachment: cppcheck-suppress.patch added

Proposed patch to suppress cppcheck error

comment:1 Changed 7 years ago by danielk

Owner: set to danielk
Status: newassigned

comment:2 Changed 7 years ago by danielk

Resolution: Fixed
Status: assignedclosed

Gary, I decided to suppress this with the suppressions.txt for now.

Note: See TracTickets for help on using tickets.