Skip to content

Commit f287a21

Browse files
committed
tee: remove collapsible_if
1 parent 5daf0a5 commit f287a21

1 file changed

Lines changed: 14 additions & 24 deletions

File tree

src/uu/tee/src/tee.rs

Lines changed: 14 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -246,18 +246,13 @@ impl Write for MultiWriter {
246246
let mode = self.output_error_mode.clone();
247247
let mut errors = 0;
248248
self.writers.retain_mut(|writer| {
249-
let result = writer.write_all(buf);
250-
match result {
251-
Err(f) => {
252-
if let Err(e) = process_error(mode.as_ref(), f, writer, &mut errors) {
253-
if aborted.is_none() {
254-
aborted = Some(e);
255-
}
256-
}
257-
false
258-
}
259-
_ => true,
260-
}
249+
writer
250+
.write_all(buf)
251+
.map_err(|f| {
252+
let _ = process_error(mode.as_ref(), f, writer, &mut errors)
253+
.map_err(|e| aborted.get_or_insert(e));
254+
})
255+
.is_ok()
261256
});
262257
self.ignored_errors += errors;
263258
if let Some(e) = aborted {
@@ -277,18 +272,13 @@ impl Write for MultiWriter {
277272
let mode = self.output_error_mode.clone();
278273
let mut errors = 0;
279274
self.writers.retain_mut(|writer| {
280-
let result = writer.flush();
281-
match result {
282-
Err(f) => {
283-
if let Err(e) = process_error(mode.as_ref(), f, writer, &mut errors) {
284-
if aborted.is_none() {
285-
aborted = Some(e);
286-
}
287-
}
288-
false
289-
}
290-
_ => true,
291-
}
275+
writer
276+
.flush()
277+
.map_err(|f| {
278+
let _ = process_error(mode.as_ref(), f, writer, &mut errors)
279+
.map_err(|e| aborted.get_or_insert(e));
280+
})
281+
.is_ok()
292282
});
293283
self.ignored_errors += errors;
294284
if let Some(e) = aborted {

0 commit comments

Comments
 (0)