Skip to content

/ servo

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

Speed up stylist rebuilds #16659

Merged
merged 5 commits into from Apr 30, 2017
+99 −94
Merged

Speed up stylist rebuilds #16659

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b4d0591
Unbust unit tests.
bholley Apr 29, 2017
080ff12
Factor out the various things stylist does per-selector into helpers.
bholley Apr 27, 2017
71ecff8
Use a single rule hash for both lower_name and name.
bholley Apr 27, 2017
b4a2e2f
Clean up note_selector a bit and stop handling combinators in two pla…
bholley Apr 28, 2017
a6ef89a
Make note_selector more efficient.
bholley Apr 28, 2017
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Next

Unbust unit tests. 

Not sure how this managed to get past bors before.
  • Loading branch information
@bholley
bholley committed Apr 29, 2017
commit b4d05919e2b6ffe1d4cdb684c1dbf747ad840e00
2 components/selectors/parser.rs
View file
Open in desktop
@@ -1457,7 +1457,7 @@ pub mod tests {
where V: SelectorVisitor<Impl = Self::Impl> { true }
}

#[derive(PartialEq, Debug)]
#[derive(Clone, PartialEq, Debug)]
pub struct DummySelectorImpl;

#[derive(Default)]
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.