jonobr1 / jonobr1/two.js

[Enhancement] Cache garbage collectible variables for each file

Open
#802 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
JavaScript
Stars
8.7k
Forks
463
Avg merge
2d 1h
Merged PRs (30d)
8

Description

Describe the solution you'd like
In this PR I introduced the pattern to have a cache object at the top of Group.js. I'd like to do this to every file to cut down on any garbage collection that happens on "per method invocations".

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the cache object pattern introduced in Group.js and review PR #801. Inspect the other project files for variables created during method invocations, then apply the requested per-file caching consistently and verify that behavior is unchanged while reducing temporary allocations.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
performance
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.