Remove error return value from the unlock function.

We defer the call to it so we can't really check exit status, and ignoring that causes lint failures.
open-telemetry · Aug 21, 2024 · bfd6d51 · bfd6d51
1 parent f3ccc6f
commit bfd6d51
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pkg/stanza/fileconsumer/internal/reader/reader_other.go b/pkg/stanza/fileconsumer/internal/reader/reader_other.go
@@ -9,6 +9,6 @@ func (r *Reader) tryLockFile() bool {
 	return true
 }
 
-func (r *Reader) unlockFile() error {
-	return nil
+func (r *Reader) unlockFile() {
+	return
 }
diff --git a/pkg/stanza/fileconsumer/internal/reader/reader_unix.go b/pkg/stanza/fileconsumer/internal/reader/reader_unix.go
@@ -23,6 +23,6 @@ func (r *Reader) tryLockFile() bool {
 	return true
 }
 
-func (r *Reader) unlockFile() error {
-	return unix.Flock(int(r.file.Fd()), unix.LOCK_UN)
+func (r *Reader) unlockFile() {
+	unix.Flock(int(r.file.Fd()), unix.LOCK_UN)
 }