[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