search

jekyll / jekyll-seo-tag

A Jekyll plugin to add metadata tags for search engines and social networks to better index and display your site's content.
https://jekyll.github.io/jekyll-seo-tag
MIT License
1.66k stars 294 forks source link

Replace read-only empty hash with private constant #418

Closed ashmaroli closed 4 years ago

ashmaroli commented 4 years ago

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

ashmaroli commented 4 years ago

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 commented 4 years ago

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