On Mon, 28 Sep 2026 07:21:03 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

> > > the method returns -1 (since -1 is already established as “not 
> > > computable/no visual position” as per the 
> > > Utilities.getPositionAbove/Below spec)
> > 
> > 
> > However, for EAST/WEST, it never needs to use target and might currently 
> > return something more useful than -1. Should the return -1 only be in the 
> > NORTH/SOUTH code path ?
> 
> I am returning -1 in NORTH/SOUTH codepath only, no?

you are right. The window github gave me lead me to mis-read the context of 
where the change was.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/33049#issuecomment-5879015322

Reply via email to