RFR: 8150564: Migrate useful ExtendedRobot methods into awt.Robot [v26]
Alisen Chung
achung at openjdk.org
Mon Jul 28 16:47:24 UTC 2025
On Fri, 25 Jul 2025 18:07:13 GMT, Phil Race <prr at openjdk.org> wrote:
>> Alisen Chung has updated the pull request incrementally with one additional commit since the last revision:
>>
>> remove static from step delay/length fields
>
> test/jdk/lib/client/ExtendedRobot.java line 253:
>
>> 251: * @see #delay(int)
>> 252: */
>> 253: public void glide(int srcX, int srcY, int destX, int destY, int stepLength, int speed) {
>
> I hadn't noticed that this over-ride is both removed here and not added to Robot.
> If this were available, then perhaps those setters and getters for the current values aren't needed !
> What was the reason for not including it ? No one ever used it or something else ?
I actually removed this when I added the setters/getters since I thought it was redundant. Do you think having this override is better than having the setters/getters? I can revert the change
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22044#discussion_r2237236118
More information about the client-libs-dev
mailing list