<div dir="ltr"><div>Webrev regenerated</div><a href="http://cr.openjdk.java.net/~martin/webrevs/openjdk9/usr-dt-environment/usr-dt-environment.patch">http://cr.openjdk.java.net/~martin/webrevs/openjdk9/usr-dt-environment/usr-dt-environment.patch</a><div>and is now the best kind of change, a pure-deletion change.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 16, 2015 at 9:00 AM, Phil Race <span dir="ltr"><<a href="mailto:philip.race@oracle.com" target="_blank">philip.race@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>That is fine by me. If any one on
      awt-dev knows of a reason to keep it<br>
      they should speak up.<span class="HOEnZb"><font color="#888888"><br>
      <br>
      -phil.</font></span><div><div class="h5"><br>
      <br>
      On 9/15/15 5:09 PM, Martin Buchholz wrote:<br>
    </div></div></div><div><div class="h5">
    <blockquote type="cite">
      <div dir="ltr">We would be entirely happy if the environment
        frobbing code were to be deleted.
        <div>Should I change my code to do that?</div>
      </div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On Tue, Sep 15, 2015 at 3:49 PM, Phil
          Race <span dir="ltr"><<a href="mailto:philip.race@oracle.com" target="_blank">philip.race@oracle.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't
            understand that original assessment.<br>
            Switching to XAWT had no impact on this code except to make
            it pointless.<br>
            i.e it did not prevent its execution.<br>
            <br>
            I doubt there is any code left in the JDK that will derive
            any<br>
            benefit from it still being there. All CDE/Motif & Xt
            related code is gone.<br>
            This should have been removed along with it but was
            doubtless not<br>
            in plain sight to whoever did that.<br>
            <br>
            So unless I am missing something you could go further and
            just delete it.<br>
            <br>
            -phil.
            <div>
              <div><br>
                <br>
                On 9/15/2015 3:13 PM, Martin Buchholz wrote:<br>
              </div>
            </div>
            <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
              <div>
                <div>
                  I reported this bug 12 years ago, but it was closed
                  Will Not Fix, and this year I see java programs
                  crashing because of it!<br>
                  This is a partial fix.<br>
                  <br>
                  <a href="https://bugs.openjdk.java.net/browse/JDK-8136570" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8136570</a><br>
                  <a href="https://bugs.openjdk.java.net/browse/JDK-4953367" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-4953367</a><br>
                </div>
              </div>
              <a href="http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/usr-dt-environment/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~martin/webrevs/openjdk9/usr-dt-environment/</a>
              <<a href="http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/usr-dt-environment/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/usr-dt-environment/</a>><br>
            </blockquote>
            <br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </div></div></div>

</blockquote></div><br></div>