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">&lt;<a href="mailto:chromatic@svn.parrot.org">chromatic@svn.parrot.org</a>&gt;</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&#39;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 &#39;t&#39;:<br>
             if (opt.opt_arg &amp;&amp; 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>