Skip to content

Match on ast/hir::ExprKind::Err #3591

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Dec 28, 2018
Merged

Match on ast/hir::ExprKind::Err #3591

merged 1 commit into from
Dec 28, 2018

Conversation

flip1995
Copy link
Member

@mati865
Copy link
Member

mati865 commented Dec 28, 2018

Travis error:

+++cargo +nightly fmt --all -- --check
Diff in /home/travis/build/rust-lang/rust-clippy/clippy_lints/src/utils/author.rs at line 506:
             },
             ExprKind::Err => {
                 println!("Err = {}", current);
-            }
+            },
         }
     }
 
Diff in /home/travis/build/rust-lang/rust-clippy/clippy_lints/src/utils/inspector.rs at line 349:
         },
         hir::ExprKind::Err => {
             println!("{}Err", ind);
-        }
+        },
     }
 }

@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 28, 2018
@phansch
Copy link
Member

phansch commented Dec 28, 2018

@bors r+

@bors
Copy link
Contributor

bors commented Dec 28, 2018

📌 Commit 909bfd3 has been approved by phansch

@bors
Copy link
Contributor

bors commented Dec 28, 2018

⌛ Testing commit 909bfd3 with merge 32b3bc2...

bors added a commit that referenced this pull request Dec 28, 2018
@bors
Copy link
Contributor

bors commented Dec 28, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: phansch
Pushing 32b3bc2 to master...

@bors bors merged commit 909bfd3 into rust-lang:master Dec 28, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants