<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>EnableInvokeDynamic and related switches may be useful for diagnostic testing, for root cause analysis. &nbsp;And they are needed to differentiate behavior&nbsp;of 1.6&nbsp;from 1.7. &nbsp;But they are probably not useful as full product switches. &nbsp;Unless there's a reason against it, we should categorize them as diagnostic switches. &nbsp;This will affect the messages in methodHandles.cpp as well as the defs in globals.hpp.</div><div><br></div><div>Also, in arguments.cpp, I suggest replacing "supported since 1.7" by "not supported before 1.7", which is logically more precise.</div><div><br></div><div>Thanks!</div><div><br></div><div>-- John</div><div><br></div><div>On Mar 28, 2011, at 10:41 AM, Christian Thalinger wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div><a href="http://cr.openjdk.java.net/~twisti/6817525/">http://cr.openjdk.java.net/~twisti/6817525/</a><br><br>6817525: turn on method handle functionality by default for JSR 292<br>Summary: After appropriate testing, we need to turn on EnableMethodHandles and EnableInvokeDynamic by default.<br>Reviewed-by:<br><br>After appropriate testing, we need to turn on EnableMethodHandles and<br>EnableInvokeDynamic by default.<font class="Apple-style-span" color="#000000"><font class="Apple-style-span" color="#144FAE"><br></font></font></div></blockquote></div></body></html>