[Maintenance]: Clean up Token class
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 53/100
Research direction
Locate the Token class and inspect the listed members and their usages before changing anything. Check how imageAssetMap is initialized and whether compatibility callers rely on the deprecated fields; the issue does not name tests, so find and run the relevant Token tests. Done means the unused members are removed, the image field preserves the null-key value, and existing behavior remains covered.
Written by the indexing model from the issue text.
Description
Describe the problem
There's a bunch of unused (or slightly used) stuff in the Token class that could use some cleaning up.
The improvement you'd like to see
The following unused code is removed from the Token class:
#NAME_COMPARATOR#hasMacros()#toTransferData()#getStatePropertyNames()
These fields that are ultimately unused are deprecated but retained to avoid backwards compatibility issues:
#lastX#lastY#currentImageAsset
These structural changes are be made:
#imageAssetMapis replaced with a simple fieldMD5Key image(only thenullkey was ever used). For backwards compatibility, initialize the new#imagefield withimageAsssetMap.get(null).
Finally, there are various bits of logic that could be replaced with simpler alternatives.
Expected Benefits
It will be easier to follow what is going on in the Token class, with the tiniest of performance improvements as well.
Additional Context
No response
- Dominant language
- Java
- Stars
- 927
- Forks
- 286
- Avg merge
- 3d 17h
- Merged PRs (30d)
- 94
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing 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.
More from RPTools/maptool
-
[Bug]: An initiative list with no current entry logs an error on every changePossibly taken @dwizard0 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
-
[Maintenance]: Migrate gradle dependency submission to modern alternativePossibly taken @VelpulaGangavardhan claimed this 18 days ago. Opencode-maintenance
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
RPTools/maptool#6156 · 2 comments · 1 assignee ·
Maintainers usually reply within 1 day
-
code-maintenance
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 1 day
-
feature
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
Similar issues
-
Cleanup: restore the interrupt flag in AiFailureAnalyzerPossibly taken @Rishankgupta08 claimed this today. Opengood first issue
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
seleniumboot/selenium-boot#258 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
johanhaleby/occurrent#1505 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
FasterXML/jackson-databind#6443 ·
Maintainers usually reply within 1 day
-
Docs: 2 broken linksOpen
Difficulty 1/5 Under an hour Newbie friendliness 78/100
Maintainers usually reply within 1 day