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