Upgrade to golangci-lint 2.10.0 and fix issues

Author: theory

Makefile

@@ -47,7 +47,7 @@ brew-lint-depends:
 
 .PHONY: debian-lint-depends # Install linting tools on Debian
 debian-lint-depends:
-	curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sudo sh -s -- -b /usr/bin v2.5.0
+	curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sudo sh -s -- -b /usr/bin v2.10.1
 
 .PHONY: install-generators # Install Go code generators
 install-generators:

README.md

@@ -35,7 +35,7 @@ A brief overview of [RFC 9535 JSONPath] syntax:
 
 ## Dependencies
 
-This package has no runtime dependencies, only testing dependencies.
+This package has only test dependencies.
 
 ## Copyright
 

parser/lex.go

@@ -213,7 +213,7 @@ func (lex *lexer) skipBlankSpace() rune {
 
 // isBlankSpace returns true if r is blank space.
 func isBlankSpace(r rune) bool {
-	return blanks&(1<<uint64(r)) != 0
+	return r >= 0 && blanks&(1<<r) != 0
 }
 
 // peekPastBlankSpace returns the next non-blank space rune from the current

parser/lex_test.go

@@ -1,3 +1,4 @@
+//nolint:revive
 package parser
 
 import (

path_example_test.go

@@ -190,16 +190,17 @@ func ExampleParser() {
 	parser := jsonpath.NewParser()
 
 	// Parse a list of paths.
-	paths := []*jsonpath.Path{}
-	for _, path := range []string{
+	strings := []string{
 		"$.store.book[*].author",
 		"$..author",
 		"$.store..color",
 		"$..book[2].author",
 		"$..book[2].publisher",
 		"$..book[?@.isbn].title",
 		"$..book[?@.price<10].title",
-	} {
+	}
+	paths := make([]*jsonpath.Path, 0, len(strings))
+	for _, path := range strings {
 		p, err := parser.Parse(path)
 		if err != nil {
 			log.Fatal(err)

spec/function.go

@@ -540,9 +540,9 @@ func (fe *FuncExpr) String() string {
 // [FuncExprArg] in fe (as passed to [Function]) and passing them to fe's
 // [FuncExtension].
 func (fe *FuncExpr) evaluate(current, root any) PathValue {
-	res := []PathValue{}
-	for _, a := range fe.args {
-		res = append(res, a.evaluate(current, root))
+	res := make([]PathValue, len(fe.args))
+	for i, a := range fe.args {
+		res[i] = a.evaluate(current, root)
 	}
 
 	return fe.fn.Evaluate(res)

spec/query.go

@@ -52,7 +52,7 @@ func (q *PathQuery) Select(current, root any) []any {
 		res[0] = root
 	}
 	for _, seg := range q.segments {
-		segRes := []any{}
+		segRes := make([]any, 0, len(res))
 		for _, v := range res {
 			segRes = append(segRes, seg.Select(v, root)...)
 		}
@@ -73,7 +73,7 @@ func (q *PathQuery) SelectLocated(current, root any, parent NormalizedPath) []*L
 		res[0] = newLocatedNode(parent, current)
 	}
 	for _, seg := range q.segments {
-		segRes := []*LocatedNode{}
+		segRes := make([]*LocatedNode, 0, len(res))
 		for _, v := range res {
 			segRes = append(segRes, seg.SelectLocated(v.Node, root, v.Path)...)
 		}