<AWT Dev>  Review request for 8003965: Toolkit.beep() documentation is ambiguous
oleg.pekhovskiy at oracle.com
Mon Sep 2 06:18:28 PDT 2013
please review the second version of fix here:
On 29.08.2013 16:05, Anthony Petrov wrote:
> Note that this change will also need a CCC request filed/approved before
> the fix may be pushed.
> best regards,
> On 08/29/13 16:03, Anthony Petrov wrote:
>> Hi Oleg,
>> I believe that a javadoc shouldn't mention any particular native APIs
>> used to implement a particular functionality, because:
>> 1. There are many more platforms where Java runs, and they also may
>> implement the beep() method.
>> 2. In the future we may need to change the underlying implementation,
>> and we certainly don't want to also change the javadoc to describe the
>> low-level changes.
>> Hence, I suggest to only state something like the following:
>> "Tells the native system to emit a beep. Emitting depends on system
>> settings and hardware capabilities."
>> best regards,
>> On 08/29/13 14:52, Oleg Pekhovskiy wrote:
>>> Hi all,
>>> please review the fix for JDK-8003965 (not available on bugs.sun.com):
>>> It's just a Javadoc fix.
More information about the awt-dev