Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v3]

2023-03-09 Thread Andy Goryachev
On Wed, 8 Mar 2023 19:36:54 GMT, Phil Race wrote: >> This fixes an the AIOOBE when finding a line break point in RTL laid out >> glyphs. >> The comment in the bug report explains how we can end up trying to find an >> unachievable break point and yet there's no "stop" on the search when we've

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v3]

2023-03-09 Thread Kevin Rushforth
On Wed, 8 Mar 2023 19:36:54 GMT, Phil Race wrote: >> This fixes an the AIOOBE when finding a line break point in RTL laid out >> glyphs. >> The comment in the bug report explains how we can end up trying to find an >> unachievable break point and yet there's no "stop" on the search when we've

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-09 Thread Kevin Rushforth
On Wed, 8 Mar 2023 19:20:21 GMT, Phil Race wrote: >> tests/system/src/test/java/test/javafx/scene/text/ArabicWrappingTest.java >> line 46: >> >>> 44: import java.util.TimerTask; >>> 45: >>> 46: public class ArabicWrappingTest extends Application { >> >> The main test class should not extend A

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v3]

2023-03-08 Thread Andy Goryachev
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Wed, 8 Mar 2023 19:36:54 GMT, Phil Race wrote:

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v3]

2023-03-08 Thread Phil Race
> This fixes an the AIOOBE when finding a line break point in RTL laid out > glyphs. > The comment in the bug report explains how we can end up trying to find an > unachievable break point and yet there's no "stop" on the search when we've > run out of glyphs so hence the exception. > > The fix

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-08 Thread Phil Race
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Wed, 8 Mar 2023 12:29:45 GMT, Kevin Rushforth wr

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-08 Thread Kevin Rushforth
On Tue, 7 Mar 2023 22:03:12 GMT, Phil Race wrote: >> This fixes an the AIOOBE when finding a line break point in RTL laid out >> glyphs. >> The comment in the bug report explains how we can end up trying to find an >> unachievable break point and yet there's no "stop" on the search when we've

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-08 Thread Ajit Ghaisas
On Tue, 7 Mar 2023 22:03:12 GMT, Phil Race wrote: >> This fixes an the AIOOBE when finding a line break point in RTL laid out >> glyphs. >> The comment in the bug report explains how we can end up trying to find an >> unachievable break point and yet there's no "stop" on the search when we've

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-07 Thread Andy Goryachev
On Tue, 7 Mar 2023 22:03:12 GMT, Phil Race wrote: >> This fixes an the AIOOBE when finding a line break point in RTL laid out >> glyphs. >> The comment in the bug report explains how we can end up trying to find an >> unachievable break point and yet there's no "stop" on the search when we've

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex()

2023-03-07 Thread Phil Race
On Tue, 7 Mar 2023 20:18:59 GMT, Andy Goryachev wrote: >> This fixes an the AIOOBE when finding a line break point in RTL laid out >> glyphs. >> The comment in the bug report explains how we can end up trying to find an >> unachievable break point and yet there's no "stop" on the search when we

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-07 Thread Phil Race
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Tue, 7 Mar 2023 21:29:20 GMT, Phil Race wrote:

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex() [v2]

2023-03-07 Thread Phil Race
> This fixes an the AIOOBE when finding a line break point in RTL laid out > glyphs. > The comment in the bug report explains how we can end up trying to find an > unachievable break point and yet there's no "stop" on the search when we've > run out of glyphs so hence the exception. > > The fix

Re: RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex()

2023-03-07 Thread Andy Goryachev
On Tue, 7 Mar 2023 19:54:15 GMT, Phil Race wrote: > This fixes an the AIOOBE when finding a line break point in RTL laid out > glyphs. > The comment in the bug report explains how we can end up trying to find an > unachievable break point and yet there's no "stop" on the search when we've > ru

RFR: 8302797: ArrayIndexOutOfBoundsException in TextRun.getWrapIndex()

2023-03-07 Thread Phil Race
This fixes an the AIOOBE when finding a line break point in RTL laid out glyphs. The comment in the bug report explains how we can end up trying to find an unachievable break point and yet there's no "stop" on the search when we've run out of glyphs so hence the exception. The fix uses a differe