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

Pre-process emoji. #3

Open
SamCB opened this issue Dec 17, 2019 · 0 comments
Open

Pre-process emoji. #3

SamCB opened this issue Dec 17, 2019 · 0 comments

Comments

@SamCB
Copy link
Owner

SamCB commented Dec 17, 2019

A problem we have is that any time the library is loaded, it goes through a process of reading the emoji-test.txt file and loading from there. On most systems, this is very quick. I chose this because it meant future updates to the emoji spec would simply require replacing the emoji-test.txt file with whatever the most up-to-date version was.

However, on slower systems... such as those which actually have use for this library... this actually takes quite a while to do. As such, we should create a script that pre-processes the emoji-test.txt file, crease some kind of serializable object out of it, and include that object in the final library. I expect this will speed up the loading process phenomanly. As a bonus, we won't need to package the emoji-test.txt file, which will save ~400kb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant