Hi Kirrill,<br><br>Yeah, I figured those out independently - and I think I&#39;ve got those patches to apply against the current bsd port with some manual hackery.<br><br>I&#39;m testing it now to check it took OK and will let the list know how I get on.<br>
<br>Thanks,<br><br>Ben<br><br><div class="gmail_quote">On Fri, Dec 26, 2008 at 9:35 PM, Kirill Shirokov <span dir="ltr">&lt;<a href="mailto:Kirill.Shirokov@sun.com" target="_blank">Kirill.Shirokov@sun.com</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;">


  

<div bgcolor="#ffffff" text="#000000">
Hi Ben,<br>
<br>
It seems that the following is a correct patch for these 2 files
(Parser.java and Name.java):<br>
<br>
diff -r 871c1708de94
src/share/classes/com/sun/tools/javac/parser/JavacParser.java<br>
--- a/src/share/classes/com/sun/tools/javac/parser/JavacParser.java&nbsp;&nbsp;&nbsp;&nbsp;
Tue Dec 23 05:50:59 2008 +0300<br>
+++ b/src/share/classes/com/sun/tools/javac/parser/JavacParser.java&nbsp;&nbsp;&nbsp;&nbsp;
Sat Dec 27 00:29:06 2008 +0300<br>
@@ -1027,7 +1027,13 @@<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return illegal(pos);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; } else {<br>
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return illegal();<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; // Support the corner case of
myMethodHandle.&lt;void&gt;invoke() by passing<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; // a void type (like other primitive types) to the
next phase.<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; // The error will be reported in Attr.attribTypes or
Attr.visitApply.<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; JCPrimitiveTypeTree ti =
to(F.at(pos).TypeIdent(TypeTags.VOID));<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; S.nextToken();<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return ti;<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; //return illegal();<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; break;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; default:<br>
diff -r 871c1708de94
src/share/classes/com/sun/tools/javac/util/Names.java<br>
--- a/src/share/classes/com/sun/tools/javac/util/Names.java&nbsp;&nbsp;&nbsp;&nbsp; Tue Dec
23 05:50:59 2008 +0300<br>
+++ b/src/share/classes/com/sun/tools/javac/util/Names.java&nbsp;&nbsp;&nbsp;&nbsp; Sat Dec
27 00:29:07 2008 +0300<br>
@@ -73,6 +73,8 @@<div><br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name java_io_Serializable;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name serialVersionUID;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name java_lang_Enum;<br>
+&nbsp;&nbsp;&nbsp; public final Name java_dyn_MethodHandle;<br>
+&nbsp;&nbsp;&nbsp; public final Name java_dyn_Dynamic;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name package_info;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name ConstantValue;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name LineNumberTable;<br></div>
@@ -111,6 +113,7 @@<div><br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name value;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name getMessage;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name getClass;<br>
+&nbsp;&nbsp;&nbsp; public final Name invoke;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name TYPE;<br>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name FIELD;<br></div>
&nbsp;&nbsp;&nbsp;&nbsp; public final Name METHOD;<br>
@@ -175,6 +178,8 @@<div><br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_lang_Cloneable = fromString(&quot;java.lang.Cloneable&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_io_Serializable = fromString(&quot;java.io.Serializable&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_lang_Enum = fromString(&quot;java.lang.Enum&quot;);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_dyn_MethodHandle = fromString(&quot;java.dyn.MethodHandle&quot;);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_dyn_Dynamic = fromString(&quot;java.dyn.Dynamic&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; package_info = fromString(&quot;package-info&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; serialVersionUID = fromString(&quot;serialVersionUID&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ConstantValue = fromString(&quot;ConstantValue&quot;);<br></div>
@@ -216,6 +221,7 @@<div><br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; value = fromString(&quot;value&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; getMessage = fromString(&quot;getMessage&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; getClass = fromString(&quot;getClass&quot;);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; invoke = fromString(&quot;invoke&quot;);<br>
<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; TYPE = fromString(&quot;TYPE&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; FIELD = fromString(&quot;FIELD&quot;);<br>
<br></div>
Best regards,<br>
Kirill<br>
<br>
Ben Evans пишет:
<div><div></div><div><blockquote type="cite">Merry Christmas John (and everyone),<br>
  <br>
I&#39;m starting to look at the delta between the current bsd-port and b42,
to see what needs doing to get this building on Mac and I&#39;ve noticed
something which seems a bit odd about meth.patch<br>
  <br>
It contains this hunk:<br>
  <br>
diff --git a/src/share/classes/com/sun/tools/javac/util/Name.java
b/src/share/classes/com/sun/tools/javac/util/Name.java<br>
--- a/src/share/classes/com/sun/tools/javac/util/Name.java<br>
+++ b/src/share/classes/com/sun/tools/javac/util/Name.java<br>
@@ -437,6 +437,8 @@<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_lang_Cloneable = fromString(&quot;java.lang.Cloneable&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_io_Serializable = fromString(&quot;java.io.Serializable&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_lang_Enum = fromString(&quot;java.lang.Enum&quot;);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_dyn_MethodHandle =
fromString(&quot;java.dyn.MethodHandle&quot;);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; java_dyn_Dynamic = fromString(&quot;java.dyn.Dynamic&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; package_info = fromString(&quot;package-info&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; serialVersionUID = fromString(&quot;serialVersionUID&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ConstantValue = fromString(&quot;ConstantValue&quot;);<br>
@@ -478,6 +480,7 @@<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; value = fromString(&quot;value&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; getMessage = fromString(&quot;getMessage&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; getClass = fromString(&quot;getClass&quot;);<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; invoke = fromString(&quot;invoke&quot;);<br>
&nbsp;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; TYPE = fromString(&quot;TYPE&quot;);<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; FIELD = fromString(&quot;FIELD&quot;);<br>
@@ -571,6 +574,8 @@<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name java_io_Serializable;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name serialVersionUID;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name java_lang_Enum;<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name java_dyn_MethodHandle;<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name java_dyn_Dynamic;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name package_info;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name ConstantValue;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name LineNumberTable;<br>
@@ -612,6 +617,7 @@<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name value;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name getMessage;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name getClass;<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name invoke;<br>
&nbsp;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name TYPE;<br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public final Name FIELD;<br>
  <br>
but in the code which I fclone&#39;d today, those additions look like they
should be applied to Names.java rather than Name.java as the diff line
would suggest.<br>
  <br>
Am I making some schoolboy error which currently eludes me?<br>
  <br>
Thanks,<br>
  <br>
Ben<br>
  <br>
  <div class="gmail_quote">On Fri, Dec 26, 2008 at 9:03 AM, John Rose <span dir="ltr">&lt;<a href="mailto:John.Rose@sun.com" target="_blank">John.Rose@sun.com</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;">
    <div>I&#39;ve spent the last 24+ hours enjoying more
traditional forms of merrymaking. &nbsp;(Though I was indeed having a wild
ride with that meth.patch stuff.)
    <div><br>
    </div>
    <div>Merry Christmas (x 12 days of); enjoy all the good things the
holidays bring!</div>
    <div><br>
    </div>
    <div>Warm best wishes,</div>
    <div>-- John
    <div>
    <div><br>
    <div>
    <div>On Dec 25, 2008, at 6:36 AM, David Pollak wrote:</div>
    <blockquote type="cite">
      <p>(F)rebasing meth... a new kind of Christmas cheer.&nbsp; :-)</p>
      <p>Sorry... it&#39;s t-minus 30 minutes to kids waking up and I spent
many hours last night doing &quot;some assembly required&quot;</p>
      <blockquote type="cite">
        <div><br>
        </div>
        <blockquote>On Dec 24, 2008 8:22 PM, &lt;<a href="mailto:john.rose@sun.com" target="_blank">john.rose@sun.com</a>&gt;
wrote:<br>
          <br>
Changeset: 1d42dc25d9e3<br>
Author: &nbsp; &nbsp;jrose<br>
Date: &nbsp; &nbsp; &nbsp;2008-12-24 19:53 -0800<br>
URL: &nbsp; &nbsp; &nbsp;&nbsp;<a href="http://hg.openjdk.java.net/mlvm/mlvm/hotspot/rev/1d42dc25d9e3" target="_blank">http://hg.openjdk.java.net/mlvm/mlvm/hotspot/rev/1d42dc25d9e3</a><br>
          <br>
meth: rebase to b42; improve implementation<br>
          <br>
! meth.patch<br>
! meth.txt<br>
! series<br>
          <br>
_____________</blockquote>
      </blockquote>
    </blockquote>
    </div>
    </div>
    </div>
    </div>
    </div>
    <br>
_______________________________________________<br>
mlvm-dev mailing list<br>
    <a href="mailto:mlvm-dev@openjdk.java.net" target="_blank">mlvm-dev@openjdk.java.net</a><br>
    <a href="http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev" target="_blank">http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev</a><br>
    <br>
  </blockquote>
  </div>
  <br>
</blockquote>
<br>
</div></div></div>

</blockquote></div><br>