From 79c783eaf80332633a178b8851b35dc4bbad77f2 Mon Sep 17 00:00:00 2001 From: Matt Jaffee Date: Wed, 12 Sep 2018 17:31:57 -0500 Subject: [PATCH] add nolint directive there isn't much piont in making snapshot take a WriterTo, because the only thing that's going to be written to a fragment is a roaring bitmap. making it WriterTo just makes it slightly more annoying to jump into the Bitmap.WriterTo implementation. since it's a private function, it should be straightforward to change this in the future if the need arises. --- fragment.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fragment.go b/fragment.go index 6d92ce37f..0d2bc27f8 100644 --- a/fragment.go +++ b/fragment.go @@ -1496,7 +1496,7 @@ func (f *fragment) snapshot() error { // snapshot writes the fragment f with bm as the data. It is unprotected, and // f.mu must be locked when calling it. -func snapshot(f *fragment, bm *roaring.Bitmap) error { +func snapshot(f *fragment, bm *roaring.Bitmap) error { // nolint: interfacer f.Logger.Printf("fragment: snapshotting %s/%s/%s/%d", f.index, f.field, f.view, f.shard) completeMessage := fmt.Sprintf("fragment: snapshot complete %s/%s/%s/%d", f.index, f.field, f.view, f.shard)