diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-06-22 09:43:20 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-06-22 09:43:20 -0500 |
commit | 0fb5d634a007f3d0424f95569ac3f83d500d054c (patch) | |
tree | 55550a10bb3a7220282a8905c0eae3962f92ec21 /components/util/thread.rs | |
parent | 87d991ebd24886051ad1131bdbe3b9019cb1c4b3 (diff) | |
parent | b103e8baa7145a125f5eb02debe0947ecc24776d (diff) | |
download | servo-0fb5d634a007f3d0424f95569ac3f83d500d054c.tar.gz servo-0fb5d634a007f3d0424f95569ac3f83d500d054c.zip |
Auto merge of #11816 - servo:stable-style, r=nox
Make the style crate (almost) build with a stable compiler
<!-- Please describe your changes on the following line: -->
The bulk of this is adding cargo features to make derived implementations of `heapsize` and `serde` traits optional.
"Almost" because `std::intrinsics::discriminant_value` is currently unstable and doesn’t have any stable replacement that I know of. For now, this PR conditionally replaces it with `unimplemented!()`.
r? @nox
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes <s>fix</s> are part of #11815 (github issue number if applicable).
<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not <s>require tests</s> *have tests yet* because that requires https://github.com/servo/servo/issues/11806, but I still want to land this before it bitrots. (Tests should check that the build succeeds with a stable compiler.)
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11816)
<!-- Reviewable:end -->
Diffstat (limited to 'components/util/thread.rs')
-rw-r--r-- | components/util/thread.rs | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/components/util/thread.rs b/components/util/thread.rs index 87d8954dcf9..571d3529d9f 100644 --- a/components/util/thread.rs +++ b/components/util/thread.rs @@ -2,13 +2,13 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -use backtrace::Backtrace; -use ipc_channel::ipc::IpcSender; -use panicking; -use serde::Serialize; -use std::any::Any; +#[cfg(feature = "servo")] use backtrace::Backtrace; +#[cfg(feature = "servo")] use ipc_channel::ipc::IpcSender; +#[cfg(feature = "servo")] use panicking; +#[cfg(feature = "servo")] use serde::Serialize; +#[cfg(feature = "servo")] use std::any::Any; +#[cfg(feature = "servo")] use thread_state; use std::thread; -use thread_state; pub fn spawn_named<F>(name: String, f: F) where F: FnOnce() + Send + 'static @@ -17,6 +17,7 @@ pub fn spawn_named<F>(name: String, f: F) } /// Arrange to send a particular message to a channel if the thread fails. +#[cfg(feature = "servo")] pub fn spawn_named_with_send_on_panic<F, Id>(name: String, state: thread_state::ThreadState, f: F, |