Update a comment about hack used to prevent gcc "unused_result" warning for strtoul...
authorwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 10 Aug 2011 04:42:42 +0000 (04:42 +0000)
committerwmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 10 Aug 2011 04:42:42 +0000 (04:42 +0000)
Specifically: The following hack prevents a gcc "unused_result" warning and
is not flagged (currently) by either Coverity or CLang.
   if (strtoul(...){}

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@38441 f5534014-38df-0310-8fa8-9805f1628bb7

gtk/prefs_dlg.c

index 6dae47ff5462b5ccbdd36034483954df9ce8eaaa..01eabe7bc92c2f4591ea02437a0991fa101177d8 100644 (file)
@@ -910,10 +910,10 @@ pref_check(pref_t *pref, gpointer user_data)
 
     /* XXX: The following ugly hack prevents a gcc warning
        "ignoring return value of 'strtoul', declared with attribute warn_unused_result"
-       which can occur when using certain gcc configurations (see _FORTIFY_SOURCE).
+       which can occur when using certain gcc configurations (see -D_FORTIFY_SOURCE).
        A dummy variable is not used because when using gcc 4.6 with -Wextra a
        "set but not used [-Wunused-but-set-variable]" warning will occur.
-       TBD: will this hack pass muster with other validators such as Coverity, CLang, & etc
+       (Coverity & CLang apparently do not object to this hack).
 
        [Guy Harris comment:
         "... perhaps either using spin buttons for numeric preferences, or otherwise making