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

Use the better TokenStream API in dom_struct #18340

Merged
merged 1 commit into from Sep 2, 2017
Merged

Conversation

@nox
Copy link
Member

nox commented Sep 1, 2017

Before:

error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:25:1
   |
25 | #[dom_struct]
   | ^^^^^^^^^^^^^ did you mean `SourceBufferList`?

After:

error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:28:39
   |
28 |     source_buffers: DOMRefCell<Vec<JS<SourceBuffer>>>,
   |                                       ^^^^^^^^^^^^ did you mean `SourceBufferList`?

This change is Reviewable

Before:

```
error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:25:1
   |
25 | #[dom_struct]
   | ^^^^^^^^^^^^^ did you mean `SourceBufferList`?
```

After:

```
error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:28:39
   |
28 |     source_buffers: DOMRefCell<Vec<JS<SourceBuffer>>>,
   |                                       ^^^^^^^^^^^^ did you mean `SourceBufferList`?
```
@nox nox force-pushed the dom-struct-errors branch from cd032b9 to 5dad4c8 Sep 1, 2017
@SimonSapin
Copy link
Member

SimonSapin commented Sep 1, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2017

📌 Commit 5dad4c8 has been approved by SimonSapin

@highfive highfive assigned SimonSapin and unassigned glennw Sep 1, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2017

Testing commit 5dad4c8 with merge 3e05fd47eaa52f21451f5fc566b9d5dfdf665023...

@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2017

💔 Test failed - mac-rel-wpt1

@jdm
Copy link
Member

jdm commented Sep 1, 2017

bors-servo added a commit that referenced this pull request Sep 2, 2017
Use the better TokenStream API in dom_struct

Before:

```rust
error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:25:1
   |
25 | #[dom_struct]
   | ^^^^^^^^^^^^^ did you mean `SourceBufferList`?
```

After:

```rust
error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:28:39
   |
28 |     source_buffers: DOMRefCell<Vec<JS<SourceBuffer>>>,
   |                                       ^^^^^^^^^^^^ did you mean `SourceBufferList`?
```

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

bors-servo commented Sep 2, 2017

Testing commit 5dad4c8 with merge 782aa34...

@bors-servo
Copy link
Contributor

bors-servo commented Sep 2, 2017

💔 Test failed - mac-rel-wpt4

@SimonSapin
Copy link
Member

SimonSapin commented Sep 2, 2017

@bors-servo retry

command timed out: 1200 seconds without output running ['env', 'PKG_CONFIG_PATH=/usr/local/opt/zlib/lib/pkgconfig', './mach', 'build', '--release'], attempting to kill
@bors-servo
Copy link
Contributor

bors-servo commented Sep 2, 2017

Testing commit 5dad4c8 with merge 32b04b3...

bors-servo added a commit that referenced this pull request Sep 2, 2017
Use the better TokenStream API in dom_struct

Before:

```rust
error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:25:1
   |
25 | #[dom_struct]
   | ^^^^^^^^^^^^^ did you mean `SourceBufferList`?
```

After:

```rust
error[E0412]: cannot find type `SourceBuffer` in this scope
  --> /Users/nox/src/servo/components/script/dom/mediasource.rs:28:39
   |
28 |     source_buffers: DOMRefCell<Vec<JS<SourceBuffer>>>,
   |                                       ^^^^^^^^^^^^ did you mean `SourceBufferList`?
```

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

bors-servo commented Sep 2, 2017

@bors-servo bors-servo merged commit 5dad4c8 into master Sep 2, 2017
3 of 4 checks passed
3 of 4 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
dependency-ci Dependencies checked
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the dom-struct-errors branch Sep 11, 2017
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

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