From 455793dfd9c657a76d7a143e0e0eb395d779379b Mon Sep 17 00:00:00 2001 From: Sander van Harmelen Date: Sat, 20 May 2017 12:43:28 +0200 Subject: Improve performance when using gometalinter (#566) * Improve performance when using gometalinter Before this change when I opened a big project that had 6000+ warnings/errors it took ages to get the actual warnings/errors and it caused my CPU to be busy for quite some time. The call to gometalinter alone took about 24 seconds, but after that vim was struggling as well. After this change the gometalinter call just takes 2 seconds and nothing noticable happens with the CPU and/or vim. * Removed obsolete test This logic is no longer done by the `ale` plugin, but by `gometalinter` itself. --- test/handler/test_gometalinter_handler.vader | 21 --------------------- 1 file changed, 21 deletions(-) (limited to 'test/handler') diff --git a/test/handler/test_gometalinter_handler.vader b/test/handler/test_gometalinter_handler.vader index 52a4fc96..603ba22d 100644 --- a/test/handler/test_gometalinter_handler.vader +++ b/test/handler/test_gometalinter_handler.vader @@ -51,24 +51,3 @@ Execute (The gometalinter handler should handle relative paths correctly): \ 'baz.go:12:3:warning: expected ''package'', found ''IDENT'' gibberish (staticcheck)', \ 'baz.go:37:5:error: expected ''package'', found ''IDENT'' gibberish (golint)', \ ]) - - -Execute (The gometalinter handler should filter out errors from other files): - silent file! /some/path/sql.go - - AssertEqual - \ [], - \ ale_linters#go#gometalinter#Handler(bufnr(''), [ - \ '/some/path/interface_implementation_test.go:417::warning: cyclomatic complexity 24 of function testGetUserHeaders() is high (> 10) (gocyclo)', - \ '/some/path/sql_helpers.go:38::warning: cyclomatic complexity 11 of function CreateTestUserMetadataDB() is high (> 10) (gocyclo)', - \ '/some/path/sql_alpha.go:560:7:warning: ineffectual assignment to err (ineffassign)', - \ '/some/path/sql_alpha.go:589:7:warning: ineffectual assignment to err (ineffassign)', - \ '/some/path/sql_test.go:124:9:warning: should not use basic type untyped string as key in context.WithValue (golint)', - \ '/some/path/interface_implementation_test.go:640::warning: declaration of "cfg" shadows declaration at sql_test.go:21 (vetshadow)', - \ '/some/path/sql_helpers.go:55::warning: declaration of "err" shadows declaration at sql_helpers.go:48 (vetshadow)', - \ '/some/path/sql_helpers.go:91::warning: declaration of "err" shadows declaration at sql_helpers.go:48 (vetshadow)', - \ '/some/path/sql_helpers.go:108::warning: declaration of "err" shadows declaration at sql_helpers.go:48 (vetshadow)', - \ '/some/path/user_metadata_db.go:149::warning: declaration of "err" shadows declaration at user_metadata_db.go:140 (vetshadow)', - \ '/some/path/user_metadata_db.go:188::warning: declaration of "err" shadows declaration at user_metadata_db.go:179 (vetshadow)', - \ '/some/path/queries_alpha.go:62::warning: Potential hardcoded credentials,HIGH,LOW (gas)', - \ ]) -- cgit v1.2.3