IssueTracker

To reproduce:

• Create a spannable string with multiple paragraphs.
• In a paragraph other than the first one, add a styling span that starts within a word such that the line will be broken to the start of the word.

An IndexOutOfBoundsException will be thrown with the following backtrace:

E/AndroidRuntime( 3070): java.lang.IndexOutOfBoundsException
E/AndroidRuntime( 3070):        at android.graphics.Paint.getTextRunAdvances(Paint.java:1731)
E/AndroidRuntime( 3070):        at android.graphics.Paint.getTextRunAdvances(Paint.java:1704)
E/AndroidRuntime( 3070):        at android.text.MeasuredText.addStyleRun(MeasuredText.java:164)
E/AndroidRuntime( 3070):        at android.text.MeasuredText.addStyleRun(MeasuredText.java:204)
E/AndroidRuntime( 3070):        at android.text.StaticLayout.generate(StaticLayout.java:281)
...

I think the offending bit of code is this section in StaticLayout.java:

  if (here < spanStart) {
    // The text was cut before the beginning of the current span range.
    // Exit the span loop, and get spanStart to start over from here.
    measured.setPos(here);
    spanEnd = here;
    break;
  }

The 'here' variable is an offset from the start of the string. The MeasuredText.setPos method just sets the mPos member of the object. Within MeasuredText mPos is treated as an offset from the start of the current paragraph. MeasuredText has a char[] array that just contains the subsection of the text for the current paragraph. Thus when the loop is restarted addSpan will be called and it will try to index the char[] array using mPos which will be way off the end of the array.

Maybe it should be measured.setPos(here - paraStart) instead?

I'll attach an example Activity which replicates the problem. The example depends on the size of the display which in my case is WVGA800.