Merge pull request #54 from xztaityozx/lint/resolve-#46

Lint/resolve #46

go.mod

@@ -1,6 +1,6 @@
 module github.com/xztaityozx/sel
 
-go 1.21
+go 1.23
 
 require (
 	github.com/spf13/cobra v1.8.0

internal/iterator/iterator.go

@@ -1,6 +1,7 @@
 package iterator
 
 import (
+  "errors"
 	"fmt"
 	"regexp"
 	"strings"
@@ -96,7 +97,7 @@ func (i *Iterator) Reset(s string) {
 // ElementAt は指定したインデックスの値を返す。1-indexed
 func (i *Iterator) ElementAt(idx int) (string, error) {
 	if idx == 0 {
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	}
 
 	if idx < 0 {
@@ -116,7 +117,7 @@ func (i *Iterator) ElementAt(idx int) (string, error) {
 			return s, nil
 		}
 
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	} else {
 		if i.head >= idx {
 			return i.buf[idx], nil
@@ -136,7 +137,7 @@ func (i *Iterator) ElementAt(idx int) (string, error) {
 			}
 		}
 
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	}
 }
 
@@ -262,7 +263,7 @@ type RegexpIterator struct {
 
 func (r *RegexpIterator) ElementAt(idx int) (string, error) {
 	if idx == 0 {
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	}
 
 	if idx > 0 {
@@ -284,7 +285,7 @@ func (r *RegexpIterator) ElementAt(idx int) (string, error) {
 			}
 		}
 
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	} else {
 		// 負のインデックス指定されたとき
 		// rightmostなIndexの検索ができないので残りの文字列をすべて分割してしまう
@@ -329,7 +330,7 @@ func (r *RegexpIterator) ElementAt(idx int) (string, error) {
 			return s, nil
 		}
 
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	}
 }
 

internal/iterator/presplit.go

@@ -1,7 +1,7 @@
 package iterator
 
 import (
-	"fmt"
+  "errors"
 	"regexp"
 	"strings"
 )
@@ -18,12 +18,12 @@ type PreSplitIterator struct {
 
 func (p *PreSplitIterator) ElementAt(idx int) (string, error) {
 	if p.l < idx {
-		return "", fmt.Errorf(IndexOutOfRange)
+		return "", errors.New(IndexOutOfRange)
 	}
 
 	if idx < 0 {
 		if -p.l > idx {
-			return "", fmt.Errorf(IndexOutOfRange)
+			return "", errors.New(IndexOutOfRange)
 		}
 		return p.a[p.l+idx], nil
 	}

internal/option/option.go

@@ -42,7 +42,7 @@ type InputFiles struct {
 
 // Enumerate /path/to/input/files
 func (ifs InputFiles) Enumerate() ([]string, error) {
-	if ifs.Files == nil || len(ifs.Files) == 0 {
+	if len(ifs.Files) == 0 {
 		return nil, fmt.Errorf("there are no files")
 	}