<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Looks OK to me, although (micro-nit) it would ne nice to see white
    space around == and before ? in this part of the line<br>
    &nbsp;&nbsp;&nbsp; <br>
    <pre><span class="changed">tree.body.type==null?


</span></pre>
    <span class="changed">-- Jon</span><br>
    <br>
    <br>
    <span class="changed"></span>
    <div class="moz-cite-prefix">On 12/06/2013 01:54 PM, Robert Field
      wrote:<br>
    </div>
    <blockquote cite="mid:52A24791.20204@oracle.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      <big><br>
        Please review compiler (javac) fix for:<br>
        <br>
        &nbsp;&nbsp;&nbsp; <a moz-do-not-send="true"
          href="https://bugs.openjdk.java.net/browse/JDK-8029558">https://bugs.openjdk.java.net/browse/JDK-8029558</a><br>
        <br>
        Webrev:<br>
        <br>
        &nbsp;&nbsp;&nbsp; <a moz-do-not-send="true"
          href="http://cr.openjdk.java.net/%7Erfield/8029558v0/">http://cr.openjdk.java.net/~rfield/8029558v0/</a><br>
        <br>
        Fix is to correctly treat the lambda body as a value returning
        entity rather than a void entity so that the correct type
        converting casts are generated.&nbsp; This is done by passing the
        erased type rather than null as the expected type.&nbsp; <br>
        <br>
        Thanks,<br>
        Robert<br>
        <br>
        <br>
      </big>
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      <br>
    </blockquote>
    <br>
  </body>
</html>