recompute ovecSlice

The first two entries are always PCRE2_UNSET, then there's 2*(capture_top-1). At least that's how I read https://www.pcre.org/current/doc/html/pcre2callout.html

.woodpecker.yml

@@ -1,13 +0,0 @@
-matrix:
-  platform:
-    - linux/amd64
-    - linux/arm64
-    - linux/riscv64
-
-steps:
-  test:
-    image: gitea.elara.ws/elara6331/golang:latest
-    commands:
-      - go test
-    when:
-      - event: push

README.md

@@ -1,7 +1,6 @@
 # pcre
 
 [![Go Reference](https://pkg.go.dev/badge/go.elara.ws/pcre.svg)](https://pkg.go.dev/go.elara.ws/pcre)
-[![status-badge](https://ci.elara.ws/api/badges/49/status.svg)](https://ci.elara.ws/49)
 
 This package provides a CGo-free port of the PCRE2 regular expression library. The [lib](lib) directory contains source code automatically translated from PCRE2's C source. This package wraps that code and provides an interface as close as possible to Go's stdlib [regexp](https://pkg.go.dev/regexp) package
 

pcre.go

@@ -8,7 +8,6 @@
 package pcre
 
 import (
-	"math"
 	"os"
 	"runtime"
 	"strconv"
@@ -20,8 +19,6 @@ import (
 	"modernc.org/libc"
 )
 
-const Unset = math.MaxUint
-
 // Version returns the version of pcre2 embedded in this library.
 func Version() string { return lib.DPACKAGE_VERSION }
 
@@ -211,12 +208,8 @@ func (r *Regexp) FindSubmatch(b []byte) [][]byte {
 
 	out := make([][]byte, 0, len(match)/2)
 	for i := 0; i < len(match); i += 2 {
-		if match[i] == Unset {
-			out = append(out, nil)
-		} else {
 		out = append(out, b[match[i]:match[i+1]])
 	}
-	}
 	return out
 }
 
@@ -260,12 +253,8 @@ func (r *Regexp) FindAllSubmatch(b []byte, n int) [][][]byte {
 		outMatch := make([][]byte, 0, len(match)/2)
 
 		for i := 0; i < len(match); i += 2 {
-			if match[i] == Unset {
-				outMatch = append(outMatch, nil)
-			} else {
 			outMatch = append(outMatch, b[match[i]:match[i+1]])
 		}
-		}
 
 		out[index] = outMatch
 	}
@@ -603,7 +592,7 @@ func (r *Regexp) SetCallout(fn func(cb *CalloutBlock) int32) error {
 		calloutStrBytes := unsafe.Slice((*byte)(unsafe.Pointer(ccb.Fcallout_string)), ccb.Fcallout_string_length)
 		cb.CalloutString = string(calloutStrBytes)
 		
-		ovecSlice := unsafe.Slice((*lib.Tsize_t)(unsafe.Pointer(ccb.Foffset_vector)), (ccb.Fcapture_top*2)-1)
+		ovecSlice := unsafe.Slice((*lib.Tsize_t)(unsafe.Pointer(ccb.Foffset_vector)), 2+(ccb.Fcapture_top-1)*2)
 		if len(ovecSlice) > 2 {
 			ovecSlice = ovecSlice[2:]
 			for i := 0; i < len(ovecSlice); i += 2 {

pcre_test.go

@@ -4,7 +4,6 @@ import (
 	"strings"
 	"sync"
 	"testing"
-	"reflect"
 
 	"go.elara.ws/pcre"
 )
@@ -290,11 +289,3 @@ func TestCallout(t *testing.T) {
 		t.Error("expected regular expression to match the string")
 	}
 }
-
-func TestVarnish(t *testing.T) {
-	regex := pcre.MustCompile(`varnish(?: \(Varnish\/([\d.]{1,250})\))?`)
-	matches := regex.FindStringSubmatch("1.1 varnish")
-	if !reflect.DeepEqual(matches, []string{"varnish", ""}) {
-		t.Errorf(`Expected ["varnish" ""], got %q`, matches)
-	}
-}

types.go

@@ -21,7 +21,7 @@ const (
 	FirstLine          = CompileOption(lib.DPCRE2_FIRSTLINE)
 	Literal            = CompileOption(lib.DPCRE2_LITERAL)
 	MatchInvalidUTF    = CompileOption(lib.DPCRE2_MATCH_INVALID_UTF)
-	MatchUnsetBackref = CompileOption(lib.DPCRE2_MATCH_UNSET_BACKREF)
+	MactchUnsetBackref = CompileOption(lib.DPCRE2_MATCH_UNSET_BACKREF)
 	Multiline          = CompileOption(lib.DPCRE2_MULTILINE)
 	NeverBackslashC    = CompileOption(lib.DPCRE2_NEVER_BACKSLASH_C)
 	NeverUCP           = CompileOption(lib.DPCRE2_NEVER_UCP)