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

Make it fast! #164

Closed
melroy89 opened this issue Jun 25, 2017 · 2 comments
Closed

Make it fast! #164

melroy89 opened this issue Jun 25, 2017 · 2 comments

Comments

@melroy89
Copy link

melroy89 commented Jun 25, 2017

As stated in the readme, this is still an open item.

So I created an issue, to higher the importance and make CSS parsing faster in servo. I think at some point you need to focus on code quality, maintenance and performance, instead of (mainly) adding new features.

But as always, keep up the good work.

Regards,
Melroy

@SimonSapin
Copy link
Member

Thanks for pointing this out. This section of README is 4 years old, and largely obsolete. I’ve removed it.

“Mainly adding new features” is not representative at all of what’s been happening in cssparser. If fact the feature set grew little since the initial implementation. Most changes have been correctness fixes and, yes, performance improvements. Some of those are #66, #102, #110, #112, #118, #122, #124, #132, #157, #159; and we have more planned. CSS parsing performance is one of the focuses of Stylo, which brings this code into Firefox.

@melroy89
Copy link
Author

Ah great. Good job.

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

2 participants