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.addepa...@oracle.com>; awt-dev
<awt-dev@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
