No matter how trivial the change, tidy always finds a way to complain...

rust-lang-ci · May 18, 2021 · e2edee4 · e2edee4
1 parent 6f7dea7
commit e2edee4
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/test/codegen/try_identity.rs b/src/test/codegen/try_identity.rs
@@ -9,8 +9,8 @@ type R = Result<u64, i32>;
 
 // This was written to the `?` from `try_trait`, but `try_trait_v2` uses a different structure,
 // so the relevant desugar is copied inline in order to keep the test testing the same thing.
-// FIXME(#85133): while this might be useful for `r#try!`, it would be nice to have a MIR optimization
-// that picks up the `?` desugaring, as `SimplifyArmIdentity` does not.  See #85133
+// FIXME(#85133): while this might be useful for `r#try!`, it would be nice to have a MIR
+// optimization that picks up the `?` desugaring, as `SimplifyArmIdentity` does not.
 #[no_mangle]
 pub fn try_identity(x: R) -> R {
 // CHECK: start:

diff --git a/src/test/mir-opt/simplify-arm.rs b/src/test/mir-opt/simplify-arm.rs
@@ -30,8 +30,8 @@ fn from_error<T, E>(e: E) -> Result<T, E> {
 
 // This was written to the `?` from `try_trait`, but `try_trait_v2` uses a different structure,
 // so the relevant desugar is copied inline in order to keep the test testing the same thing.
-// FIXME(#85133): while this might be useful for `r#try!`, it would be nice to have a MIR optimization
-// that picks up the `?` desugaring, as `SimplifyArmIdentity` does not.  See #85133
+// FIXME(#85133): while this might be useful for `r#try!`, it would be nice to have a MIR
+// optimization that picks up the `?` desugaring, as `SimplifyArmIdentity` does not.
 fn id_try(r: Result<u8, i32>) -> Result<u8, i32> {
     let x = match into_result(r) {
         Err(e) => return from_error(From::from(e)),

diff --git a/src/test/mir-opt/simplify_try.rs b/src/test/mir-opt/simplify_try.rs
@@ -15,8 +15,8 @@ fn from_error<T, E>(e: E) -> Result<T, E> {
 
 // This was written to the `?` from `try_trait`, but `try_trait_v2` uses a different structure,
 // so the relevant desugar is copied inline in order to keep the test testing the same thing.
-// FIXME(#85133): while this might be useful for `r#try!`, it would be nice to have a MIR optimization
-// that picks up the `?` desugaring, as `SimplifyArmIdentity` does not.  See #85133
+// FIXME(#85133): while this might be useful for `r#try!`, it would be nice to have a MIR
+// optimization that picks up the `?` desugaring, as `SimplifyArmIdentity` does not.
 fn try_identity(x: Result<u32, i32>) -> Result<u32, i32> {
     let y = match into_result(x) {
         Err(e) => return from_error(From::from(e)),