Changes between Version 28 and Version 29 of Guidelines/MaintenanceGuidelines


Ignore:
Timestamp:
Nov 18, 2009, 10:41:36 AM (13 years ago)
Author:
Paul A. Bristow
Comment:

more on C4800

Legend:

Unmodified
Added
Removed
Modified

  • Guidelines/MaintenanceGuidelines

    v28 v29  
    750750||C4710||'function' : function not inlined||If variable is volatile, suppress.||#pragma warning(disable: 4510) // function not inlined.||
    751751
    752 ||C4800|| int' : forcing value to bool 'true' or 'false'||Use a bool valued expression, or static_cast. Writing (intexpr ? true : false) helps as well. Or suppress.||#  pragma warning(disable: 4800) // int' : forcing value to bool 'true' or 'false'
     752||C4800|| int' : forcing value to bool 'true' or 'false'||Use a bool valued expression. Write out expressions, for example: "value >= 0" or "ptr != 0" (Boost prefers clarity to curtness). Take care if using templates that constants are of the right type, for example you may need "static_cast<T>(1)". Or use static_cast<bool>(expression). Or suppress.||#  pragma warning(disable: 4800) // int' : forcing value to bool 'true' or 'false'
    753753
    754754||C4996||'putenv': The POSIX name for this item is deprecated (but NOT by POSIX!).||Suppress.||#  pragma warning(disable: 4996) // 'putenv' was declared deprecated.||