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

Lots of warnings: the type of this value must be known in this context #19637

Closed
metux opened this issue Dec 23, 2017 · 2 comments
Closed

Lots of warnings: the type of this value must be known in this context #19637

metux opened this issue Dec 23, 2017 · 2 comments

Comments

@metux
Copy link

@metux metux commented Dec 23, 2017

Hi folks,

I'm getting a lot of warnings like this:

warning[E0619]: the type of this value must be known in this context
--> /home/nekrad/src/moz/servo/target/release/build/script-917cc22c0e8f3269/out/Bindings/XMLHttpRequestEventTargetBinding.rs:1088:74
|
1088 | EventTargetBinding::GetConstructorObject(cx, global, >interface_proto.handle_mut());
| > ^^^^^^^^^^
|
= note: this will be made into a hard error in a future version of the compiler

warning[E0619]: the type of this value must be known in this context
--> /home/nekrad/src/moz/servo/target/release/build/script-917cc22c0e8f3269/out/Bindings/XMLHttpRequestEventTargetBinding.rs:1102:51
|
1102 | interface.handle_mut());
| ^^^^^^^^^^
|
= note: this will be made into a hard error in a future version of the compiler

What does that mean ?

I'm using a pretty fresh rustc (updated few days ago) - did anything
vital change in the compiler/language ?

--mtx

@SimonSapin
Copy link
Member

@SimonSapin SimonSapin commented Dec 23, 2017

This is rust-lang/rust#46906. In general, using a version other than the one specified in the rust-toolchain file is unsupported. If you use rustup or build with ./mach build the correct version is downloaded automatically.

At the moment, upgrading is blocked on #19519.

@metux
Copy link
Author

@metux metux commented Dec 23, 2017

Okay. I had upgraded rust, hoping that #[cfg()] on if statements would be implemented now ...

Nevertheless, this issue should be fixed sooner or later.

bors-servo added a commit that referenced this issue Jan 5, 2018
[Do not merge] Upgrade to rustc 1.24.0-nightly (0a3761e63 2018-01-03)

Do not merge (yet), as this might bring #19519 back.

Fixes #19635
Fixes #19637

<!-- 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/19683)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jan 9, 2018
[Do not merge] Upgrade to rustc 1.24.0-nightly (0a3761e63 2018-01-03)

Do not merge (yet), as this might bring #19519 back.

Fixes #19635
Fixes #19637

<!-- 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/19683)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jan 9, 2018
[Do not merge] Upgrade to rustc 1.24.0-nightly (0a3761e63 2018-01-03)

Do not merge (yet), as this might bring #19519 back.

Fixes #19635
Fixes #19637

<!-- 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/19683)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jan 9, 2018
[Do not merge] Upgrade to rustc 1.24.0-nightly (0a3761e63 2018-01-03)

Do not merge (yet), as this might bring #19519 back.

Fixes #19635
Fixes #19637

<!-- 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/19683)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jan 10, 2018
Upgrade to rustc 1.24.0-nightly (0a3761e63 2018-01-03)

Fixes #19635
Fixes #19637
Fixes #19735
Fixes #19519

<!-- 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/19683)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.