ArrayList.subList IllegalArgumentException when rendering text input widget after pasting code

[GreyMario]

Issue type:

• 🐛 Bug

---

Short description:

Bringing a crashlog from the Discord where user encountered crash when pasting script.

User believes they erased existing script manually and selected topmost line before pasting new script. User claims TPS of server is suffering, so server performance may be related.

Description: Rendering screen

java.lang.IllegalArgumentException: fromIndex(33) > toIndex(26)
	at java.base/java.util.AbstractList.subListRangeCheck(AbstractList.java:511) ~[?:?] {re:mixin}
	at java.base/java.util.ArrayList.subList(ArrayList.java:1190) ~[?:?] {re:mixin}
	at TRANSFORMER/integratedscripting@1.0.19/org.cyclops.integratedscripting.client.gui.component.input.WidgetTextArea.rebuildDisplayCache(WidgetTextArea.java:581) ~[integratedscripting-1.21.1-neoforge-1.0.19-295.jar%23915!/:1.0.19]

---

Versions:

• This mod: IntegratedScripting v1.0.19
• Minecraft: 1.21.1
• Mod loader version: NeoForge v21.1.215

Log file:

https://mclo.gs/kCyCJ2q

[rubensworks]

Thanks for reporting!

[GreyMario]

Further conversation and a look at https://github.com/CyclopsMC/IntegratedScripting/blob/master-1.21-lts/src/main/java/org/cyclops/integratedscripting/client/gui/component/input/WidgetTextArea.java#L581 is leading me to believe that this might happen when replacing a tall text with one that doesn't scroll? Needs further testing.

GitHub

IntegratedScripting/src/main/java/org/cyclops/integratedscripting/client/gui/component/input/WidgetTextArea.java at master-1.21-lts · CyclopsMC/IntegratedScripting

Create scripts for handling complex operations in Integrated Dynamics - CyclopsMC/IntegratedScripting