From 36d1c4540af3ef058d68092e41af05aa4a8c5eca Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Thu, 8 Mar 2018 18:48:26 +0100 Subject: all: fix gometalinter warnings Fix typos, non-canonical code, remove dead code, etc. --- pkg/email/parser_test.go | 3 +++ pkg/email/patch_test.go | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) (limited to 'pkg/email') diff --git a/pkg/email/parser_test.go b/pkg/email/parser_test.go index 69e237806..a99ab17f6 100644 --- a/pkg/email/parser_test.go +++ b/pkg/email/parser_test.go @@ -91,6 +91,9 @@ func TestAddAddrContextEmptyName(t *testing.T) { t.Fatalf("want: %q, got %q", want, email1) } email2, context1, err := RemoveAddrContext(email1) + if err != nil { + t.Fatal(err) + } if email != email2 { t.Fatalf("want: %q, got %q", email, email2) } diff --git a/pkg/email/patch_test.go b/pkg/email/patch_test.go index b10211ff0..c8cae4621 100644 --- a/pkg/email/patch_test.go +++ b/pkg/email/patch_test.go @@ -157,7 +157,7 @@ The handling of the might_cancel queueing is not properly protected, so parallel operations on the file descriptor can race with each other and lead to list corruptions or use after free. -Protect the context for these operations with a seperate lock. +Protect the context for these operations with a separate lock. Reported-by: Foo Bar Signed-off-by: Foo Bar -- cgit mrf-deployment