-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
style: Move most of allowed_in to NonCustomPropertyId. #20046
Conversation
Heads up! This PR modifies the following files:
|
r? @nox |
@bors-servo r+ |
📌 Commit c53016a has been approved by |
style: Move most of allowed_in to NonCustomPropertyId. Also simplify the servo property id to gecko property id conversion to be faster and less repetitive. This is in preparation of some patches to properly allow disabled longhands on enabled shorthands, something that we don't contemplate right now. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/20046) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
error[E0425]: cannot find value `id` in this scope
--> /home/servo/buildbot/slave/linux-rel-wpt/build/target/release/build/style-4b63d270e8e82720/out/properties.rs:51080:44
|
51080 | let pref = match PREF_NAME[id.0] {
| ^^ not found in this scope
help: possible candidate is found in another module, you can import it into scope
|
14 | use std::process::id;
|
error: aborting due to previous error
error: Could not compile `style`. |
Also simplify the servo property id to gecko property id conversion to be faster and less repetitive.
@bors-servo r=nox |
📌 Commit 1548520 has been approved by |
style: Move most of allowed_in to NonCustomPropertyId. Also simplify the servo property id to gecko property id conversion to be faster and less repetitive. This is in preparation of some patches to properly allow disabled longhands on enabled shorthands, something that we don't contemplate right now. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/20046) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
Also simplify the servo property id to gecko property id conversion to be faster
and less repetitive.
This is in preparation of some patches to properly allow disabled longhands on enabled shorthands, something that we don't contemplate right now.
This change is