<AWT Dev> [13] RFR: JDK-8215910: Typo in AWT InvocationEvent Method Documentation

Krishna Addepalli krishna.addepalli at oracle.com
Thu Jan 3 14:25:04 UTC 2019


Thank you Alexey. I understood that the second Runnable was not needed, hence I made changes.
So, I believe you are endorsing webrev01 right?

Thanks
Krishna

-----Original Message-----
From: Alexey Ivanov 
Sent: Thursday, January 3, 2019 3:14 PM
To: Krishna Addepalli <krishna.addepalli at oracle.com>; awt-dev <awt-dev at openjdk.java.net>
Subject: Re: <AWT Dev> [13] RFR: JDK-8215910: Typo in AWT InvocationEvent Method Documentation

Hi Krishna,

On 31/12/2018 16:09, Alexey Ivanov wrote:
> Hi Krishna,
>
> But the bug hasn't been fixed yet:

Sorry I might have sounded rude. I didn't intend to.

> - 230      * @param listener          The {@code Runnable}Runnable 
> whose
> + 230      * @param listener          The {@code Runnable} Runnable 
> + whose

You did exactly what was requested in the bug report [1].

> The second "Runnable" is not needed at all.

But I believe the typo is in the repeated "Runnable" word rather than the missing space between "Runnable Runnable".

Thank you for cleaning up the uses of run() to make them consistent across the documentation of the class. I really appreciate it.


Regards,
Alexey

[1] https://bugs.openjdk.java.net/browse/JDK-8215910

>
> Regards,
> Alexey
>
> On 31/12/2018 07:50, Krishna Addepalli wrote:
>> Hi All,
>>
>> Please review a trivial doc fix for JDK-8215910.
>> Webrev: http://cr.openjdk.java.net/~kaddepalli/8215910/webrev00/
>>
>> Thanks,
>> Krishna 


More information about the awt-dev mailing list