HaxeFlixel / HaxeFlixel/flixel
FlxTextBitmap: Update line width caching
Open
Nobody has claimed this yet.
- Dominant language
- Haxe
- Stars
- 2.2k
- Forks
- 522
- Avg merge
- 34m
- Merged PRs (30d)
- 1
Description
Discussed in https://github.com/HaxeFlixel/flixel/pull/3548
We should remove _linesWidth entirely, it doesn't seem to be useful anywhere, we may look into some new caching system but as long as no addons need this field we can remove it, beforehand
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review PR #3548 and the referenced FlxBitmapText.hx location around line 1651. Confirm whether _linesWidth is still needed, including by addons, then remove it if it is not; any replacement caching system is left for later investigation.
Written by the indexing model from the issue text.
Assessment
- Domain
- game-dev
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100