Opened 14 years ago

Closed 14 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@… 14 years ago.
Proposed patch to suppress cppcheck error

Download all attachments as: .zip

Change History (3)

by gary.buhrmaster@…, 14 years ago

Attachment: cppcheck-suppress.patch​ added

Proposed patch to suppress cppcheck error

comment:1 by danielk, 14 years ago

Owner: set to danielk
Status: new → assigned

comment:2 by danielk, 14 years ago

Resolution: → Fixed
Status: assigned → closed

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

Note: See TracTickets for help on using tickets.