Changes between Version 22 and Version 23 of Guidelines/MaintenanceGuidelines

Timestamp:

Nov 9, 2009, 4:09:34 PM (13 years ago)

Author:

Hartmut Kaiser

Comment:

--

Guidelines/MaintenanceGuidelines

@@ -687 +687 @@
 
 
-||C4100|| unreferenced formal parameter||Either surround the parameter with C comments, for example: int */ my_variable */)or just delete if the variable name is uninformative. If in other's module(s), or you are too busy/idle, at least suppress warning.||#  pragma warning(disable: 4100) // 'name' : unreferenced formal parameter||
+||C4100|| unreferenced formal parameter||Either surround the parameter with C comments, for example: int */ my_variable */)or just delete if the variable name is uninformative. If in other's module(s), or you are too busy/idle, at least suppress warning. In generic code you might not be able to comment the variable name as it might actually be used to call a static function. In this case simply referencing the variable (varname;) in the function body eliminates the warning.||#  pragma warning(disable: 4100) // 'name' : unreferenced formal parameter||
 
 ||C4127|| conditional expression is constant||Very common and many believe unhelpful, but a few find it informative, so do not suppress globally.  Even while(true) can trigger this!  Suppress. while(true) can be rewritten as for(;;) eliminating the warning. || #  pragma warning(disable: 4127) // conditional expression is constant.||