<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Hi Anthony, Sergey,<br>
      <blockquote type="cite">Did you run KeyEvent regression tests with
        your fix (both open and closed)? Also, perhaps TextArea/TextEdit
        tests (from both AWT and Swing) might be worth running as well.
      </blockquote>
      I did, and I didn't found any new failures (on Ubuntu 12.04 at
      least).<br>
      <blockquote type="cite">I suggest to build it on jprt as
        additional check, just to be safe.<br>
      </blockquote>
      It
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      builds successfully on all platforms provided by JPRT.<br>
      <pre class="moz-signature" cols="72">Thanks,

Alexander.</pre>
      On 09/20/2013 01:23 AM, Sergey Bylokhov wrote:<br>
    </div>
    <blockquote cite="mid:523B6B3E.6010908@oracle.com" type="cite">Hi,Alexander
      <br>
      I suggest to build it on jprt as additional check, just to be
      safe.
      <br>
      <br>
      On 20.09.2013 0:03, Anthony Petrov wrote:
      <br>
      <blockquote type="cite">Hi Alexander,
        <br>
        <br>
        src/solaris/native/sun/xawt/XlibWrapper.c
        <br>
        <blockquote type="cite"> 530 XkbSetDetectableAutoRepeat((Display
          *) jlong_to_ptr(display), detectable, NULL);
          <br>
        </blockquote>
        <br>
        I suggest to convert the jboolean "detectable" to Bool
        explicitly to avoid any potential compiler warnings. No need for
        a new webrev with this change. The fix looks fine to me
        otherwise (though I see no point in modifying
        make/sun/xawt/mapfile-vers - the old build is dead.)
        <br>
        <br>
        Did you run KeyEvent regression tests with your fix (both open
        and closed)? Also, perhaps TextArea/TextEdit tests (from both
        AWT and Swing) might be worth running as well.
        <br>
        <br>
        -- <br>
        best regards,
        <br>
        Anthony
        <br>
        <br>
        On 09/19/2013 10:38 PM, Alexander Zvegintsev wrote:
        <br>
        <blockquote type="cite">Hello,
          <br>
          please review fix for
          <br>
          bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8019282">https://bugs.openjdk.java.net/browse/JDK-8019282</a>
          <br>
          webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8019282/webrev.00">http://cr.openjdk.java.net/~serb/8019282/webrev.00</a>
          <br>
          <br>
          The standard behavior of the X server is to generate a
          KeyRelease event
          <br>
          for every KeyPress event.
          <br>
          That is why we receiving KEY_RELEASED event when auto-repeat
          feature is
          <br>
          active.
          <br>
          <br>
          Fortunately, there is XkbSetDetectableAutoRepeat function:
          <br>
          <a class="moz-txt-link-freetext" href="http://linux.die.net/man/3/xkbsetdetectableautorepeat">http://linux.die.net/man/3/xkbsetdetectableautorepeat</a>
          <br>
          <br>
          It allows to receive KeyRelease events only when the key is
          physically
          <br>
          released (like on Windows).
          <br>
          <br>
        </blockquote>
      </blockquote>
      qqq
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>