Another reason to avoid new API is that we don't have to involve the CCC to get this
"bug fix" in...
...jim
On 5/13/16 3:50 PM, Jim Graham wrote:
That looks very tight. The only issue I'd have is that it would be better if
this could be done with non-public API for
That looks very tight. The only issue I'd have is that it would be better if this could be done with non-public API for
now - the map() methods could live on one of the sun.awt.image classes or even in a Swing implementation utility class
and still work just fine. When we have more time to
On 05/11/2016 06:58 AM, Alexander Scherbatiy wrote:
Hello,
Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/8132119/webrev.12
From an API perspective this seems fine with the caveats that you have
follow-on implementation work to do and that I am not familiar
The fix looks good to me.
Thanks,
Alexandr.
On 10/05/16 20:42, Semyon Sadetsky wrote:
On 4/28/2016 7:46 PM, Sergey Bylokhov wrote:
On 28.04.16 19:30, Semyon Sadetsky wrote:
ok, then just make the new field private and do not use the interned
string as a key.
Why do you think it should be
Hello Semyon, Sergey and Alexander,
Thank you very much for review of this fix. The new version of the fix
was created. Could you please review the second version of the fix.
Webrev (the 2nd version):
http://cr.openjdk.java.net/~alitvinov/8041694/jdk9/webrev.01
The second version of the
On 5/13/2016 7:54 PM, Alexander Potochkin wrote:
Hello
I just checked the code in JDK9 and found that it still syncs only
with the EDT
and completely ignores the toolkit thread.
I cannot agree. Robot uses native SunToolkit.syncNativeQueue() to
syncs it with the native event queue.
On 10.05.16 19:42, Semyon Sadetsky wrote:
I made the flag key private. Please review
http://cr.openjdk.java.net/~ssadetsky/8046031/webrev.01/
Looks fine.
--
Best regards, Sergey.
Hi Mikhail,
Thanks!
Looks great now!
Regards,
Alexey
On 12.05.2016 18:54, Alexander Potochkin wrote:
Hello Mikhail
Looks great!
Thanks
alexp
On 5/12/2016 5:53 PM, Alexey Ivanov wrote:
56 while (!parsingDone && System.currentTimeMillis() - s < 5_000) {
Check for exception was added:
