[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-compiler-dev
Subject:    Re: RFR: JDK-8201251: The tests for JDK-8187247 should be under test/langtools
From:       joe darcy <joe.darcy () oracle ! com>
Date:       2018-04-06 17:44:44
Message-ID: d46e8022-8af8-c9f2-9402-d0cc11924301 () oracle ! com
[Download RAW message or body]

Looks fine; cheers,

-Joe


On 4/6/2018 9:07 AM, Liam Miller-Cushon wrote:
> Hi,
>
> The tests for  JDK-8187247 were added to test/tools/javac instead of 
> test/langtools/tools/javac. This patch moves them to the correct location.
>
> Webrev: http://cr.openjdk.java.net/~cushon/8201251/webrev.00/ 
> <http://cr.openjdk.java.net/%7Ecushon/8201251/webrev.00/>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8201251
>
> Thanks,
> Liam


[Attachment #3 (text/html)]

<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Looks fine; cheers,</p>
    <p>-Joe<br>
    </p>
    <br>
    <div class="moz-cite-prefix">On 4/6/2018 9:07 AM, Liam Miller-Cushon
      wrote:<br>
    </div>
    <blockquote type="cite"
cite="mid:CAL4Qsgv6mP_JBNFjgz1S9obh1dEiwm1jjoHmrHCdi6cryD-Fvw@mail.gmail.com">
      <div dir="ltr">Hi,
        <div><br>
        </div>
        <div>The tests for  JDK-8187247 were added to test/tools/javac
          instead of  <span
style="color:rgb(34,34,34);font-family:sans-serif;font-size:13px;font-style:normal;fon \
t-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:nor \
mal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spaci \
ng:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">test/langtools/tools/javac.
  This patch moves them to the correct location.</span></div>
        <div>
          <div><br>
          </div>
          <div>Webrev:  <a
              href="http://cr.openjdk.java.net/%7Ecushon/8201251/webrev.00/"
              moz-do-not-send="true">http://cr.openjdk.java.net/~cushon/8201251/webrev.00/</a></div>
  <div>Bug:  <a
              href="https://bugs.openjdk.java.net/browse/JDK-8201251"
              moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8201251</a></div>
  <div><br>
          </div>
          <div>Thanks,</div>
          <div>Liam</div>
        </div>
      </div>
    </blockquote>
    <br>
  </body>
</html>



[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic