mikros/serde
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Always consider empty output to be unsuccessful

Browse Source
This commit is contained in:
David Tolnay 2023-07-19 16:13:02 -07:00
parent d3da41927a
commit 6c0e838a7c
No known key found for this signature in database
GPG Key ID: F9BA143B95FF6D82
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
precompiled/serde_derive/src/lib_precompiled.rs
View File

@ -6,7 +6,7 @@ use crate::bytecode::Bytecode;
use proc_macro::{Delimiter, Group, Ident, Literal, Punct, Spacing, Span, TokenStream, TokenTree};
use std::io::{Read, Write};
use std::iter::FromIterator;
use std::process::{Command, Stdio};
use std::process::{Command, ExitStatus, Stdio};
use std::str::FromStr;
#[proc_macro_derive(Serialize, attributes(serde))]
@ -48,11 +48,8 @@ fn derive(select: u8, input: TokenStream) -> TokenStream {
buf.clear();
stdout.read_to_end(&mut buf).unwrap();
let success = match child.wait() {
Ok(exit_status) => exit_status.success(),
Err(_) => !buf.is_empty(),
};
if !success {
let success = child.wait().as_ref().map_or(true, ExitStatus::success);
if !success || buf.is_empty() {
panic!();
}