Performance issue with 2.7.4
- Dominant language
- JavaScript
- Stars
- 2.9k
- Forks
- 461
- PR merge metrics
- No merged PRs in 30d
Description
Hi,
I've found an performance issue in the current 2.7 branch and would fix that. I also found out that the code in master is already restructured and may not be affected anymore. Will the current master be merged in 2.7 or is it worth it to fix that 2.7 branch?
(v8 does not optimise that function)
https://github.com/linkedin/dustjs/blob/2.7/lib/parser.js#L268
Marc
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at lib/parser.js line 268 in the 2.7 branch and compare the corresponding code in master, since the issue says the parser was restructured there. Reproduce the reported V8 optimization problem and determine whether the 2.7 branch still needs a fix; done means the branch decision and performance impact are verified.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- web-dev
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100