Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Commits

Ticket #10303: Proposed cppcheck error suppression

 

 

MythTV commits RSS feed   Index | Next | Previous | View Threaded


noreply at mythtv

Feb 2, 2012, 4:11 AM

Post #1 of 3 (35 views)
Permalink
Ticket #10303: Proposed cppcheck error suppression

#10303: Proposed cppcheck error suppression
---------------------------------------+----------------------------------
Reporter: gary.buhrmaster@… | Type: Patch - Bug Fix
Status: new | Priority: trivial
Milestone: unknown | Component: MythTV - General
Version: Unspecified | Severity: low
Keywords: | Ticket locked: 0
---------------------------------------+----------------------------------
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.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10303>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Feb 2, 2012, 7:32 AM

Post #2 of 3 (30 views)
Permalink
Re: Ticket #10303: Proposed cppcheck error suppression [In reply to]

#10303: Proposed cppcheck error suppression
---------------------------------------+-----------------------------
Reporter: gary.buhrmaster@… | Owner: danielk
Type: Patch - Bug Fix | Status: assigned
Priority: trivial | Milestone: unknown
Component: MythTV - General | Version: Unspecified
Severity: low | Resolution:
Keywords: | Ticket locked: 0
---------------------------------------+-----------------------------
Changes (by danielk):

* owner: => danielk
* status: new => assigned


--
Ticket URL: <http://code.mythtv.org/trac/ticket/10303#comment:1>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Feb 6, 2012, 3:03 PM

Post #3 of 3 (29 views)
Permalink
Re: Ticket #10303: Proposed cppcheck error suppression [In reply to]

#10303: Proposed cppcheck error suppression
---------------------------------------+-----------------------------
Reporter: gary.buhrmaster@… | Owner: danielk
Type: Patch - Bug Fix | Status: closed
Priority: trivial | Milestone: unknown
Component: MythTV - General | Version: Unspecified
Severity: low | Resolution: Fixed
Keywords: | Ticket locked: 0
---------------------------------------+-----------------------------
Changes (by danielk):

* status: assigned => closed
* resolution: => Fixed


Comment:

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

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10303#comment:2>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits

MythTV commits RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.