chromatic - this cast is required to build with g++<br><br>I had to restore it in r44276<br><br>I am sure there could be another solution to this problem.<br><br><div class="gmail_quote">On 21 February 2010 04:06, <span dir="ltr"><<a href="mailto:chromatic@svn.parrot.org">chromatic@svn.parrot.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Author: chromatic<br>
Date: Sun Feb 21 04:06:41 2010<br>
New Revision: 44270<br>
URL: <a href="https://trac.parrot.org/parrot/changeset/44270" target="_blank">https://trac.parrot.org/parrot/changeset/44270</a><br>
<br>
Log:<br>
[src] Removed unnecessary cast when parsing tracing flags in parseflags(). Not<br>
only does it generate compiler warnings about the enumeration, but it gives a<br>
false sense of security that we've done any argument checking there.<br>
<br>
Modified:<br>
trunk/src/main.c<br>
<br>
Modified: trunk/src/main.c<br>
==============================================================================<br>
--- trunk/src/main.c Sun Feb 21 04:06:33 2010 (r44269)<br>
+++ trunk/src/main.c Sun Feb 21 04:06:41 2010 (r44270)<br>
@@ -477,7 +477,7 @@<br>
break;<br>
case 't':<br>
if (opt.opt_arg && is_all_hex_digits(opt.opt_arg))<br>
- *trace = (Parrot_trace_flags)strtoul(opt.opt_arg, NULL, 16);<br>
+ *trace = strtoul(opt.opt_arg, NULL, 16);<br>
else<br>
*trace = PARROT_TRACE_OPS_FLAG;<br>
break;<br>
_______________________________________________<br>
<a href="http://lists.parrot.org/mailman/listinfo/parrot-commits" target="_blank">http://lists.parrot.org/mailman/listinfo/parrot-commits</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Michael H. Hind<br><br>Tel: +44 (0) 1224 631 581<br>Cell: +44 (0) 7877 224 745<br>