[svn:parrot] r44270 - trunk/src

chromatic at svn.parrot.org chromatic at svn.parrot.org
Sun Feb 21 04:06:42 UTC 2010


Author: chromatic
Date: Sun Feb 21 04:06:41 2010
New Revision: 44270
URL: https://trac.parrot.org/parrot/changeset/44270

Log:
[src] Removed unnecessary cast when parsing tracing flags in parseflags().  Not
only does it generate compiler warnings about the enumeration, but it gives a
false sense of security that we've done any argument checking there.

Modified:
   trunk/src/main.c

Modified: trunk/src/main.c
==============================================================================
--- trunk/src/main.c	Sun Feb 21 04:06:33 2010	(r44269)
+++ trunk/src/main.c	Sun Feb 21 04:06:41 2010	(r44270)
@@ -477,7 +477,7 @@
             break;
           case 't':
             if (opt.opt_arg && is_all_hex_digits(opt.opt_arg))
-                *trace = (Parrot_trace_flags)strtoul(opt.opt_arg, NULL, 16);
+                *trace = strtoul(opt.opt_arg, NULL, 16);
             else
                 *trace = PARROT_TRACE_OPS_FLAG;
             break;


More information about the parrot-commits mailing list