aboutsummaryrefslogtreecommitdiffhomepage
path: root/helper
diff options
context:
space:
mode:
authorOphestra <cat@gensokyo.uk>2025-10-14 01:48:56 +0900
committerOphestra <cat@gensokyo.uk>2025-10-14 01:48:56 +0900
commit048c1957f136347826990513d198e58d049446cd (patch)
tree14953fd43820640ec8595a626934528203f15b3b /helper
parent790d77075e40ee3162d7925e983f01747b5c2c89 (diff)
helper/args: variadic check function
This package turns out to be much less widely used than anticipated, and might be facing removal. This change makes test cases cleaner. Signed-off-by: Ophestra <cat@gensokyo.uk>
Diffstat (limited to 'helper')
-rw-r--r--helper/args.go6
-rw-r--r--helper/args_test.go4
-rw-r--r--helper/helper_test.go2
3 files changed, 6 insertions, 6 deletions
diff --git a/helper/args.go b/helper/args.go
index 88b5e691..80cd3024 100644
--- a/helper/args.go
+++ b/helper/args.go
@@ -35,7 +35,7 @@ func (a argsWt) String() string {
}
// NewCheckedArgs returns a checked null-terminated argument writer for a copy of args.
-func NewCheckedArgs(args []string) (wt io.WriterTo, err error) {
+func NewCheckedArgs(args ...string) (wt io.WriterTo, err error) {
a := make(argsWt, len(args))
for i, arg := range args {
a[i], err = syscall.ByteSliceFromString(arg)
@@ -49,8 +49,8 @@ func NewCheckedArgs(args []string) (wt io.WriterTo, err error) {
// MustNewCheckedArgs returns a checked null-terminated argument writer for a copy of args.
// If s contains a NUL byte this function panics instead of returning an error.
-func MustNewCheckedArgs(args []string) io.WriterTo {
- a, err := NewCheckedArgs(args)
+func MustNewCheckedArgs(args ...string) io.WriterTo {
+ a, err := NewCheckedArgs(args...)
if err != nil {
panic(err.Error())
}
diff --git a/helper/args_test.go b/helper/args_test.go
index 3aff8cf5..d6225c3e 100644
--- a/helper/args_test.go
+++ b/helper/args_test.go
@@ -22,7 +22,7 @@ func TestNewCheckedArgs(t *testing.T) {
t.Parallel()
args := []string{"\x00"}
- if _, err := helper.NewCheckedArgs(args); !errors.Is(err, syscall.EINVAL) {
+ if _, err := helper.NewCheckedArgs(args...); !errors.Is(err, syscall.EINVAL) {
t.Errorf("NewCheckedArgs: error = %v, wantErr %v", err, syscall.EINVAL)
}
@@ -36,6 +36,6 @@ func TestNewCheckedArgs(t *testing.T) {
t.Errorf("MustNewCheckedArgs: panic = %v, wantPanic %v", r, wantPanic)
}
}()
- helper.MustNewCheckedArgs(badPayload)
+ helper.MustNewCheckedArgs(badPayload...)
})
}
diff --git a/helper/helper_test.go b/helper/helper_test.go
index 45f7bc96..4cc36d34 100644
--- a/helper/helper_test.go
+++ b/helper/helper_test.go
@@ -27,7 +27,7 @@ var (
}
wantPayload = strings.Join(wantArgs, "\x00") + "\x00"
- argsWt = helper.MustNewCheckedArgs(wantArgs)
+ argsWt = helper.MustNewCheckedArgs(wantArgs...)
)
func argF(argsFd, statFd int) []string {