<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    This works on my system.<br>
    <br>
    +1<br>
    <br>
    -phil.<br>
    <br>
    <div class="moz-cite-prefix">On 05/10/2018 01:30 PM, Pankaj Bansal
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:e8e76bd6-a2d7-4eaf-b7b3-41bc3649de01@default">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <meta name="Generator" content="Microsoft Word 15 (filtered
        medium)">
      <style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Consolas;
        panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri","sans-serif";
        color:black;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
pre
        {mso-style-priority:99;
        mso-style-link:"HTML Preformatted Char";
        margin:0in;
        margin-bottom:.0001pt;
        font-size:10.0pt;
        font-family:"Courier New";
        color:black;}
span.HTMLPreformattedChar
        {mso-style-name:"HTML Preformatted Char";
        mso-style-priority:99;
        mso-style-link:"HTML Preformatted";
        font-family:Consolas;
        color:black;}
span.new
        {mso-style-name:new;}
span.EmailStyle20
        {mso-style-type:personal;
        font-family:"Calibri","sans-serif";
        color:windowtext;}
span.EmailStyle21
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
      <div class="WordSection1">
        <p class="MsoNormal"><span style="color:#1F497D">Hi Phil,<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Thanks for the
            quick review.<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><<Execing xrandr twice ? Can't we
          either just re-write the code to not use takeWhile() which<br>
          <<is part of the problem, or less desirably store the
          results into an internal buffer<br>
          <<and read it from there using StringReader/StringWriter
          - wrapped in BufferedReader/BufferedWriter ?<span
            style="color:#1F497D"><o:p></o:p></span></p>
        <p class="MsoNormal"><a name="_MailEndCompose"
            moz-do-not-send="true"><span style="color:#1F497D">Yes that
              makes much more sense. I have removed takeWhile as I think
              filter should do the work here. There will not be need to
              ignore any line.<o:p></o:p></span></a></p>
        <p class="MsoNormal"><span style="color:#1F497D">I have also
            used the try-with-resource as suggested.<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Webrev:<o:p></o:p></span></p>
        <p class="MsoNormal"><a
            href="http://cr.openjdk.java.net/%7Epbansal/8196616/webrev.01/"
            moz-do-not-send="true">http://cr.openjdk.java.net/~pbansal/8196616/webrev.01/</a><span
            style="color:#1F497D"><o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Regards,<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Pankaj Bansal<o:p></o:p></span></p>
        <div>
          <div style="border:none;border-top:solid #E1E1E1
            1.0pt;padding:3.0pt 0in 0in 0in">
            <p class="MsoNormal"><b><span style="color:windowtext">From:</span></b><span
                style="color:windowtext"> Phil Race <br>
                <b>Sent:</b> Friday, May 11, 2018 12:59 AM<br>
                <b>To:</b> Pankaj Bansal; <a class="moz-txt-link-abbreviated" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a><br>
                <b>Subject:</b> Re: [11] Review Request: JDK-8196616
                java/awt/GraphicsDevice/DisplayModes/CompareToXrandrTest.java
                fails<o:p></o:p></span></p>
          </div>
        </div>
        <p class="MsoNormal"><o:p> </o:p></p>
        <p class="MsoNormal">Execing xrandr twice ? Can't we either just
          re-write the code to not use takeWhile() which<br>
          is part of the problem, or less desirably store the results
          into an internal buffer<br>
          and read it from there using StringReader/StringWriter -
          wrapped in BufferedReader/BufferedWriter ?<br>
          <br>
          Also I think you should use try-with-resources on the reading
          from the "real" stream.<br>
          <br>
          And a style point :<span style="font-size:12.0pt"><o:p></o:p></span></p>
        <pre><span class="new">while((line = reader.readLine()) != null) {<o:p></o:p></span></pre>
        <pre><span class="new">-><o:p></o:p></span></pre>
        <pre><span class="new">while ((line = reader.readLine()) != null) {</span><o:p></o:p></pre>
        <p class="MsoNormal" style="margin-bottom:12.0pt"><br>
          -phil.<br>
          <br>
          <br>
          <o:p></o:p></p>
        <div>
          <p class="MsoNormal">On 05/10/2018 12:21 PM, Pankaj Bansal
            wrote:<o:p></o:p></p>
        </div>
        <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
          <p class="MsoNormal">Hi All, Please review test fix for the
            below bug:<o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal">Bug: <a
              href="https://bugs.openjdk.java.net/browse/JDK-8196616"
              moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8196616</a><o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal">Webrev: <a
              href="http://cr.openjdk.java.net/%7Epbansal/8196616/webrev.00/"
              moz-do-not-send="true">http://cr.openjdk.java.net/~pbansal/8196616/webrev.00/</a><o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal">The test case create a BufferedReader
             and this BufferedReader is used to find XRanderModes and
            JavaModes and compare them.<o:p></o:p></p>
          <p class="MsoNormal">It is assumed in this test that the first
            two lines of the  BufferedReader don’t have the useful mode
            data and contain some other information. So it ignores first
            two lines. But this is not always true and test may have to
            ignore more or less lines depending upon the data. <o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal">Made changes to the test to check how
            many lines should be ignored, instead of hard coding the
            number of lines to ignore.<o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal">Regards,<br>
            Pankaj Bansal<o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
        </blockquote>
        <p class="MsoNormal"><span
            style="font-size:12.0pt;font-family:"Times New
            Roman","serif""><o:p> </o:p></span></p>
      </div>
    </blockquote>
    <br>
  </body>
</html>