hyperledger-labs / fabric-smart-client / 34840579748

91%

Build Type

push

github

Committed by web-flow

Commit Message

lint: enable sqlclosecheck and fix the two flagged query result sets (#1820)

Both flagged sites (BindingStore.HaveSameBinding, SignerInfoStore.FilterExistingSigners)
deferred rows.Close() through utils.IgnoreErrorFunc, an indirection the linter can't see
through. Defer a closure that calls rows.Close() directly instead, which both sqlclosecheck
and errcheck recognize, and drop the now-unneeded utils import.

Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>

Coverage Stats

2 of 2 new or added lines in 2 files covered. (100.0%)

11 existing lines in 5 files now uncovered.

27243 of 31934 relevant lines covered (85.31%)

2358.3 hits per line