cde0cde151
In the stabilization attempt of `#[unix_sigpipe = "sig_dfl"]`, a concern was raised related to using a language attribute for the feature: Long term, we want `fn lang_start()` to be definable by any crate, not just libstd. Having a special language attribute in that case becomes awkward. So as a first step towards towards the next stabilization attempt, this PR changes the `#[unix_sigpipe = "..."]` attribute to a compiler flag `-Zon-broken-pipe=...` to remove that concern, since now the language is not "contaminated" by this feature. Another point was also raised, namely that the ui should not leak **how** it does things, but rather what the **end effect** is. The new flag uses the proposed naming. This is of course something that can be iterated on further before stabilization.
34 lines
955 B
Rust
34 lines
955 B
Rust
#![feature(rustc_private)]
|
|
extern crate libc;
|
|
|
|
/// So tests don't have to bring libc in scope themselves
|
|
pub enum SignalHandler {
|
|
Ignore,
|
|
Default,
|
|
}
|
|
|
|
/// Helper to assert that [`libc::SIGPIPE`] has the expected signal handler.
|
|
pub fn assert_sigpipe_handler(expected_handler: SignalHandler) {
|
|
#[cfg(unix)]
|
|
#[cfg(not(any(
|
|
target_os = "emscripten",
|
|
target_os = "fuchsia",
|
|
target_os = "horizon",
|
|
target_os = "android",
|
|
)))]
|
|
{
|
|
let actual = unsafe {
|
|
let mut actual: libc::sigaction = std::mem::zeroed();
|
|
libc::sigaction(libc::SIGPIPE, std::ptr::null(), &mut actual);
|
|
actual.sa_sigaction
|
|
};
|
|
|
|
let expected = match expected_handler {
|
|
SignalHandler::Ignore => libc::SIG_IGN,
|
|
SignalHandler::Default => libc::SIG_DFL,
|
|
};
|
|
|
|
assert_eq!(actual, expected, "actual and expected SIGPIPE disposition differs");
|
|
}
|
|
}
|