<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
<tt>Looks good to me also. :-)<br>
<br>
Dan<br>
<br>
P.S.<br>
So the missing "endif" at the end didn't cause any errors?<br>
<br>
</tt><br>
On 4/5/2011 6:29 PM, Kelly O'Hair wrote:
<blockquote cite="mid:F5C3DDD2-2286-440D-9432-A7B52ED20D5C@oracle.com"
type="cite">
<pre wrap="">
Sorry... I put dcubed down as the reviewer and meant to put dholmes. :^(
It's been a long day... :^( Need to go home and get some sleep.
Thank you for the review, you too Dan. ;^)
-kto
On Apr 5, 2011, at 5:17 PM, David Holmes wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Looks good to me.
David
Kelly O'Hair said the following on 04/06/11 09:53:
</pre>
<blockquote type="cite">
<pre wrap="">Strange... I need a reviewer on this small change.... not sure why my email never showed up. :^(
Begin forwarded message:
</pre>
<blockquote type="cite">
<pre wrap="">*From: *Kelly O'Hair <<a class="moz-txt-link-abbreviated" href="mailto:kelly.ohair@oracle.com">kelly.ohair@oracle.com</a> <a class="moz-txt-link-rfc2396E" href="mailto:kelly.ohair@oracle.com"><mailto:kelly.ohair@oracle.com></a>>
*Date: *April 5, 2011 11:56:25 AM PDT
*To: *build-dev <<a class="moz-txt-link-abbreviated" href="mailto:build-dev@openjdk.java.net">build-dev@openjdk.java.net</a> <a class="moz-txt-link-rfc2396E" href="mailto:build-dev@openjdk.java.net"><mailto:build-dev@openjdk.java.net></a>>
*Subject: **Need reviewer: Turn off warnings for mapfiles in fastdebug builds*
Need reviewer: Turn off warnings for mapfiles in fastdebug builds
The mapfiles are generally not used on fastdebug or debug builds, this just turns off the
warning in these cases, making it just an informative piece of information.
7033960: Do not check for mapfiles when fastdebug building
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-mapfile-warning/webrev/">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-mapfile-warning/webrev/</a>
-kto
</pre>
</blockquote>
</blockquote>
</blockquote>
<pre wrap=""><!---->
</pre>
</blockquote>
</body>
</html>