[mythtv-commits] Ticket #12996: Fix compiler warnings in trunk

MythTV noreply at mythtv.org
Fri Feb 10 16:50:29 UTC 2017


#12996: Fix compiler warnings in trunk
--------------------------------------+-----------------------------
 Reporter:  David Hampton <mythtv@…>  |          Owner:
     Type:  Patch - Bug Fix           |         Status:  new
 Priority:  minor                     |      Milestone:  29.0
Component:  MythTV - General          |        Version:  Master Head
 Severity:  medium                    |     Resolution:
 Keywords:                            |  Ticket locked:  0
--------------------------------------+-----------------------------

Comment (by Gary Buhrmaster <gary.buhrmaster@…>):

 Re: 0006-...., some (although not necessarily all) of the unused
 parameters fixes should likely be modified to follow the "Coding
 Standards" document which states:

 '''Unused parameters''' Functions that have to satisfy a given interface
 definition sometimes have parameters that are not used. The name of an
 unused parameter should be put in a comment to document that the parameter
 is intentionally unused in such a way that using it anyway will not work.
 If the function does not have to follow such an interface definition the
 parameter should simply be removed.

 So, for a specific example, a "func(int a, int b)" should turn into a
 "func(int /* a */, int /* b */)" if a and b are not used, OR the coding
 standard document needs to be updated.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/12996#comment:3>
MythTV <http://www.mythtv.org>
MythTV Media Center


More information about the mythtv-commits mailing list