Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace read-only empty hash with private constant #418

Merged
merged 1 commit into from
Oct 15, 2020

Conversation

ashmaroli
Copy link
Member

To avoid allocating multiple empty Hash objects for every instance of the Drop class.

@ashmaroli
Copy link
Member Author

Third-party Repo Memory Profile Summary

--- master https://github.com/jekyll/jekyll-seo-tag/runs/1226684766
+++ PR     https://github.com/jekyll/jekyll-seo-tag/runs/1258882894

- Total allocated: 26.40 MB (214878 objects)
- Total retained:  19.52 kB (167 objects)
+ Total allocated: 25.18 MB (209612 objects)
+ Total retained:  19.39 kB (167 objects)

@ashmaroli ashmaroli marked this pull request as ready for review October 15, 2020 13:13
@ashmaroli ashmaroli requested a review from a team October 15, 2020 13:13
@ashmaroli
Copy link
Member Author

Thank you Matt and Frank :)
@jekyllbot: merge +fix

@jekyllbot jekyllbot merged commit 80da6a7 into jekyll:master Oct 15, 2020
jekyllbot added a commit that referenced this pull request Oct 15, 2020
@ashmaroli ashmaroli deleted the constant-empty-read-only-hash branch October 15, 2020 14:00
@ashmaroli ashmaroli removed the bug label Oct 15, 2020
@jekyll jekyll locked and limited conversation to collaborators Oct 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants