IssueTracker

This should be moved to the Android Public Tracker > Text component, but I don't have permissions so I'm redirecting to the appropriate owner.

So after updating to 1.4.1 and all deps too, the crash was a little different see below.

The repro for that one is easy

1. create a new app from AS 7.1 RC 1
2. update deps to (Appcompat 1.4.1/Material 1.5.0/Constraintlayout 2.1.3)
3. Add 2 textViews

    <TextView
        android:id="@+id/text1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Hello World!"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <TextView
        android:id="@+id/text2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Hello World!"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/text1" />```

4. In onCreate:

        val t1 = findViewById<TextView>(R.id.text1)
        val t2 = findViewById<TextView>(R.id.text2)

        val charBuffer1 = CharArrayBuffer("AAA".toCharArray())
        charBuffer1.sizeCopied = 3
        val charBuffer2 = CharArrayBuffer("AAA".toCharArray())
        charBuffer2.sizeCopied = 3

        t1.setText(charBuffer1.data, 0, charBuffer1.sizeCopied)
        t2.setText(charBuffer2.data, 0, charBuffer2.sizeCopied)

5. Run see the texts updated then crash with the following. (Tested on P6 Pro Android 12)

Process: org.debug.myapplication, PID: 32493
    java.lang.NullPointerException: src == null
        at java.lang.System.arraycopy(Native Method)
        at android.widget.TextView$CharWrapper.getChars(TextView.java:13588)
        at android.text.TextUtils.getChars(TextUtils.java:155)
        at android.text.BoringLayout.hasAnyInterestingChars(BoringLayout.java:314)
        at android.text.BoringLayout.isBoring(BoringLayout.java:340)
        at android.widget.TextView.onMeasure(TextView.java:9413)
        at androidx.appcompat.widget.AppCompatTextView.onMeasure(AppCompatTextView.java:607)
        at android.view.View.measure(View.java:25774)
        at androidx.constraintlayout.widget.ConstraintLayout$Measurer.measure(ConstraintLayout.java:811)
        at androidx.constraintlayout.core.widgets.analyzer.BasicMeasure.measure(BasicMeasure.java:466)
        at androidx.constraintlayout.core.widgets.analyzer.BasicMeasure.measureChildren(BasicMeasure.java:134)
        at androidx.constraintlayout.core.widgets.analyzer.BasicMeasure.solverMeasure(BasicMeasure.java:278)
        at androidx.constraintlayout.core.widgets.ConstraintWidgetContainer.measure(ConstraintWidgetContainer.java:120)
        at androidx.constraintlayout.widget.ConstraintLayout.resolveSystem(ConstraintLayout.java:1594)
        at androidx.constraintlayout.widget.ConstraintLayout.onMeasure(ConstraintLayout.java:1708)
        at android.view.View.measure(View.java:25774)
        at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:6980)
        at android.widget.FrameLayout.onMeasure(FrameLayout.java:194)
        at androidx.appcompat.widget.ContentFrameLayout.onMeasure(ContentFrameLayout.java:145)
        at android.view.View.measure(View.java:25774)
        at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:6980)
        at androidx.appcompat.widget.ActionBarOverlayLayout.onMeasure(ActionBarOverlayLayout.java:496)
        at android.view.View.measure(View.java:25774)
        at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:6980)
        at android.widget.FrameLayout.onMeasure(FrameLayout.java:194)
        at android.view.View.measure(View.java:25774)
        at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:6980)
        at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1552)
        at android.widget.LinearLayout.measureVertical(LinearLayout.java:842)
        at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
        at android.view.View.measure(View.java:25774)
        at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:6980)
        at android.widget.FrameLayout.onMeasure(FrameLayout.java:194)
        at com.android.internal.policy.DecorView.onMeasure(DecorView.java:761)
        at android.view.View.measure(View.java:25774)
        at android.view.ViewRootImpl.performMeasure(ViewRootImpl.java:3628)
        at android.view.ViewRootImpl.measureHierarchy(ViewRootImpl.java:2424)
        at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2694)
        at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:2143)
        at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:8665)
        at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1037)
        at android.view.Choreographer.doCallbacks(Choreographer.java:845)
        at android.view.Choreographer.doFrame(Choreographer.java:780)
        at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1022)
        at android.os.Handler.handleCallback(Handler.java:938)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loopOnce(Looper.java:201)
        at android.os.Looper.loop(Looper.java:288)
        at android.app.ActivityThread.main(ActivityThread.java:7839)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)

since it is possibly related to emojicompat reassigned

Thanks for the report.

What version of Android are you testing this on?

Android 12 P6 pro P3a and emulator A 11.

Can test on others the last repro is easy to test.

Thanks! That's helpful to know it's reproing on both 11 and 12 - no need to test on others. We'll have to explore a bit.

Any feedback on this? Appcompat 1.6 is around the corner with Android T. Migrating away from CharArrayBuffer is a ton of work to try to keep the huge performance gain it provide for my use case.

Going out in next release.

There's no code workaround that enables emoji and allows that setText overload to be used in the current version.

Other options:

1. emojiCompatEnabled=false in your theme will skip emoji processing (please enable it again once fix goes out)
2. Don't use setText(char[]) (sounds like this is not really an option for you)
3. Hold back appcompat upgrade until the fix goes out.

The fix will go out in emoji2:emoji2-views-helpers 1.2-alpha02 so you may be able to bump that dependency early to test it.

Thanks for the fix I can wait as nothing mandatory in appcompat 1.4 for my use case.

Is there some compatibility matrix between appcompat and emoji ? (Mostly will emoji 1.2 work with app compat 1.4 in case there's blocking stuff to support T and appcompat 1.6?)

Great to hear!

I don't have a compat spreadsheet, but 1.2 is going to be 1.1 with bugfixes like this so no concerns here.