Do a Garbage Pass (on Non-Animation)

[edwardrowe]

There could be areas we are generating needless garbage. Especially consider:

• String concatenation
• Animating text mesh
• any case of "new"

[edwardrowe]

This actually won't be too bad. Found the following areas to tackle:

Tag Parsing:

• ProcessCustomTags uses new Lists.
• TextTagParser.CreateSymbolListFromText Creates a list of TextSymbols
• RemoveTagsFromString uses substring which appears to allocate a new string
• Lots of Substrings in RichTextTag. It stores the tag just as a full string instead of splitting it out, so every property call uses substring to figure out what part of the rich text tag it is... Best fix is to probably save these out. Could make RichTextTags read only so that they can easily cache the values.

Text Typing:

• Pool Coroutine yields - could make a class that stores them and creates them if they're new. Maybe it's a queue and we pop them off when there are too many.

Animation:

• Pool Animation Components (or don't use components at all?)

  unity-text-typer/Assets/RedBlueGames/TextTyper/TextTyper.cs

  Line 121 in 20cbfba

  // TODO - Would be better to pool/reuse these components

[edwardrowe]

Captured allocations when printing the color tags. Appears to be 334 KB (did it twice, same result both times).