Hi, Semyon.
That's a specification which should be read as written. But if you mean
this is not the same things, then it is unclear what value will be added
to the description of "awt.mouse.numButtons" property, which mentions in
the specification for the first time. Since getNumberOfButtons obtain
something not specified from the getToolkit, modify it somehow(w/o
specification) and returns. See for example Toolkit.getToolkit and
Toolkit. areExtraMouseButtonsEnabled(). It is not necessary write so
specific specification but at least it should be clear.
It would be good to rephrase it somehow.
12.03.15 0:09, Semyon Sadetsky wrote:
Hi Sergey,
I didn't find any mention in the new text that the method returns the
same value as Tolkit.get... returns.
I'm not an expert in English but in my opinion "obtained by" verb
doesn't state that the same value is returned without any handling.
Maybe you've mixed it up with "proxy"?
Thanks,
--Semyon
On 3/12/2015 9:47 AM, Sergey Bylokhov wrote:
Hi, Semyon.
The fix in general is correct, but it adds an assertion that this
method should return the same values as Toolkit.get...
And this is incorrect, and we can get a new CR that implementation
don't follow the specification. Probably we can simplify it and state
that we use numeric value from desktop property or something like that?
11.03.15 22:52, Semyon Sadetsky wrote:
Hello,
please review fix for jdk9.
Webrev:
http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/7081580/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-7081580
Thanks,
--Semyon
--
Best regards, Sergey.