RFR: 8319844 : Text/TextFlow.hitTest() is incorrect in RTL orientation [v13]

[Andy Goryachev]

On Thu, 29 Feb 2024 17:33:14 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:

>> @hjohn how do you get this coverage diagram?
>> 
>> The BreakIterator is a part of the existing code, we should probably have this discussion outside of this PR.  I agree, there might be a situation when the app wants to select a specific locale for the text component instead of relying on the default locale.
>
> @andy-goryachev-oracle  The coverage comes from EclEmma, an Eclipse plugin. Once installed, there is another way to run tests called `Coverage as...` just above `Run as...`.  It's very useful to use it on a JUnit test to see if there are major branches you missed in testing that may require additional cases.

thank you @hjohn good to know about EclEmma

you might be right about dead code: I could not hit the break points in L461 or L471 with the MT.  There might be a scenario which we are not testing.  I'd say leave it as is perhaps?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1323#discussion_r1508028426