<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Hi Semyon, Sergey,<br>
<br>
I agree with that the modified javadoc is not good.<br>
<br>
1. When you say something is done "by calling A.b()", it means I
can write exactly "A.b()" in my code and this will do the job.
However, that's not the case with Toolkit.getDesktopProperty() (it
won't be compiled).<br>
<br>
In order to refer to a method, you can use either of the following
constructions:<br>
<br>
a) the A.b() method<br>
b) {@link A#b} <br>
c) the {@link A#b} method<br>
<br>
b/c is preferrable.<br>
<br>
2. When you say "The value is obtained by calling something", it's
not quite clear what or who obtains the value. The method itself?
Or this is an alternative way to get it for a user?<br>
<br>
3. If this is the only place in the spec where the property is
introduced, then you should somehow reflect this fact. For
instance, like this:<br>
<br>
The value is set by the "awt.mouse.numButtons" property, which can
be obtained directly with the {@link Toolkit#getDesktopProperty}
method.<br>
<br>
You don't have to _specify_ the way getNumberOfButtons() obtains
the property, unless this implementation detail should really be
specified. (For instance, if it was obtained by a method which
could be overriden in an application.)<br>
<br>
Thanks,<br>
Anton.<br>
<br>
<br>
On 12.03.2015 11:42, Semyon Sadetsky wrote:<br>
</div>
<blockquote cite="mid:55015164.6010906@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
Sorry, Sergey. Still don't understand what you mean. <br>
The issue is about<b> to do</b><b> mention</b>
"awt.mouse.numButtons". <br>
Now you are saying that there is no value to mention it for the
first time in this spec. Doesn't it contradict to the request
itself?<br>
You couldn't be more specific on what do you want, could you? <br>
The fix just adds one short statement to the spec. Maybe you'll
find it to be more productive to just rephrase as you want and
write here.<br>
<br>
Thank you!<br>
--Semyon<br>
<br>
<div class="moz-cite-prefix">On 3/12/2015 11:11 AM, Sergey
Bylokhov wrote:<br>
</div>
<blockquote cite="mid:55014A35.6070302@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8"
http-equiv="Content-Type">
<div class="moz-cite-prefix">Hi, Semyon.<br>
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.
<meta http-equiv="content-type" content="text/html;
charset=UTF-8">
areExtraMouseButtonsEnabled(). It is not necessary write so
specific specification but at least it should be clear.<br>
<br>
It would be good to rephrase it somehow.<br>
<br>
12.03.15 0:09, Semyon Sadetsky wrote:<br>
</div>
<blockquote cite="mid:55013BB2.5060104@oracle.com" type="cite">Hi
Sergey, <br>
<br>
I didn't find any mention in the new text that the method
returns the same value as Tolkit.get... returns. <br>
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. <br>
Maybe you've mixed it up with "proxy"? <br>
<br>
Thanks, <br>
--Semyon <br>
<br>
<br>
On 3/12/2015 9:47 AM, Sergey Bylokhov wrote: <br>
<blockquote type="cite">Hi, Semyon. <br>
The fix in general is correct, but it adds an assertion that
this method should return the same values as Toolkit.get...
<br>
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? <br>
<br>
11.03.15 22:52, Semyon Sadetsky wrote: <br>
<blockquote type="cite">Hello, <br>
<br>
please review fix for jdk9. <br>
<br>
Webrev: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ealexsch/semyon-sadetsky/7081580/webrev.00/">http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/7081580/webrev.00/</a>
<br>
Bug: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-7081580">https://bugs.openjdk.java.net/browse/JDK-7081580</a>
<br>
<br>
Thanks, <br>
--Semyon <br>
</blockquote>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Best regards, Sergey. </pre>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>