echo: don't output "\n" if "\c" is encountered

cakebaker · cakebaker · commit c29bcb219f6d · 2023-10-03T14:37:15.000+02:00
diff --git a/src/uu/echo/src/echo.rs b/src/uu/echo/src/echo.rs
@@ -175,7 +175,7 @@ fn execute(no_newline: bool, escaped: bool, free: &[String]) -> io::Result<()> {
         }
         if escaped {
             if print_escaped(input, &mut output)?.is_break() {
-                break;
+                return Ok(());
             }
         } else {
             write!(output, "{input}")?;
diff --git a/tests/by-util/test_echo.rs b/tests/by-util/test_echo.rs
@@ -122,7 +122,7 @@ fn test_escape_no_further_output() {
     new_ucmd!()
         .args(&["-e", "a\\cb", "c"])
         .succeeds()
-        .stdout_only("a\n");
+        .stdout_only("a");
 }
 
 #[test]

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ fn execute(no_newline: bool, escaped: bool, free: &[String]) -> io::Result<()> {`
`175`	`175`	`}`
`176`	`176`	`if escaped {`
`177`	`177`	`if print_escaped(input, &mut output)?.is_break() {`
`178`		`- break;`
	`178`	`+ return Ok(());`
`179`	`179`	`}`
`180`	`180`	`} else {`
`181`	`181`	`write!(output, "{input}")?;`
Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ fn test_escape_no_further_output() {`
`122`	`122`	`new_ucmd!()`
`123`	`123`	`.args(&["-e", "a\\cb", "c"])`
`124`	`124`	`.succeeds()`
`125`		`- .stdout_only("a\n");`
	`125`	`+ .stdout_only("a");`
`126`	`126`	`}`
`127`	`127`
`128`	`128`	`#[test]`