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

Enable old glyph assertion #22165

Merged
merged 1 commit into from Nov 16, 2018
Merged

Enable old glyph assertion #22165

merged 1 commit into from Nov 16, 2018

Conversation

@pyfisch
Copy link
Contributor

pyfisch commented Nov 10, 2018

The assertion had a note that it should be enabled
when something like debug_assert is available.

This must have been disabled since before Rust 1.0 as it uses invalid syntax.


This change is Reviewable

The assertion had a note that it should be enabled
when something like debug_assert is available.
@highfive
Copy link

highfive commented Nov 10, 2018

warning Warning warning

  • These commits modify gfx code, but no tests are modified. Please consider adding a test!
@pyfisch
Copy link
Contributor Author

pyfisch commented Nov 10, 2018

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Nov 10, 2018

Trying commit a093702 with merge a6dabcf...

bors-servo added a commit that referenced this pull request Nov 10, 2018
Enable old glyph assertion

The assertion had a note that it should be enabled
when something like debug_assert is available.

This must have been disabled since before Rust 1.0 as it uses invalid syntax.

<!-- 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/22165)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 10, 2018

☀️ Test successful - linux-rel-css, linux-rel-wpt
State: approved= try=True

@SimonSapin
Copy link
Member

SimonSapin commented Nov 10, 2018

r? @pcwalton who added this… in 2012. 463b800

@jdm
Copy link
Member

jdm commented Nov 15, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2018

📌 Commit a093702 has been approved by jdm

@highfive highfive assigned jdm and unassigned SimonSapin Nov 15, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2018

Testing commit a093702 with merge 5007210...

bors-servo added a commit that referenced this pull request Nov 15, 2018
Enable old glyph assertion

The assertion had a note that it should be enabled
when something like debug_assert is available.

This must have been disabled since before Rust 1.0 as it uses invalid syntax.

<!-- 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/22165)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2018

💔 Test failed - linux-rel-css

@jdm
Copy link
Member

jdm commented Nov 16, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

Testing commit a093702 with merge 1626881...

bors-servo added a commit that referenced this pull request Nov 16, 2018
Enable old glyph assertion

The assertion had a note that it should be enabled
when something like debug_assert is available.

This must have been disabled since before Rust 1.0 as it uses invalid syntax.

<!-- 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/22165)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

💔 Test failed - linux-dev

@jdm
Copy link
Member

jdm commented Nov 16, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Nov 16, 2018

Testing commit a093702 with merge 050b2bc...

bors-servo added a commit that referenced this pull request Nov 16, 2018
Enable old glyph assertion

The assertion had a note that it should be enabled
when something like debug_assert is available.

This must have been disabled since before Rust 1.0 as it uses invalid syntax.

<!-- 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/22165)
<!-- Reviewable:end -->
@bors-servo bors-servo merged commit a093702 into servo:master Nov 16, 2018
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.