diff options
| author | Grégoire Duchêne <gduchene@awhk.org> | 2022-12-03 21:37:43 +0000 |
|---|---|---|
| committer | Grégoire Duchêne <gduchene@awhk.org> | 2022-12-03 21:40:19 +0000 |
| commit | 16edafb4f96ec7accf2d797d10064364b5efdd5a (patch) | |
| tree | 95624aeaede2dbfef46eb55a09c328e749f1b486 /flag_test.go | |
| parent | e9163a8c80b9d444d4eac37199991e3f6e70094f (diff) | |
Remove ‘value’ argument from Flag{,Slice}Varv0.4.0
It’s basically redundant since a pointer to the value is already
provided; callers can just set a default value there.
Diffstat (limited to 'flag_test.go')
| -rw-r--r-- | flag_test.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/flag_test.go b/flag_test.go index 1f36b63..91c6caa 100644 --- a/flag_test.go +++ b/flag_test.go @@ -25,8 +25,8 @@ func TestFlagVar(s *testing.T) { t := core.T{T: s} fs := flag.NewFlagSet("", flag.PanicOnError) - var fl int - core.FlagVar(fs, &fl, "test", 42, "", strconv.Atoi) + fl := 42 + core.FlagVar(fs, &fl, "test", "", strconv.Atoi) t.AssertEqual(42, fl) t.AssertErrorIs(nil, fs.Parse([]string{"-test=84"})) t.AssertEqual(84, fl) @@ -46,8 +46,8 @@ func TestFlagSliceVar(s *testing.T) { t := core.T{T: s} fs := flag.NewFlagSet("", flag.PanicOnError) - var fl []int - core.FlagSliceVar(fs, &fl, "test", []int{42}, "", strconv.Atoi, ",") + fl := []int{42} + core.FlagSliceVar(fs, &fl, "test", "", strconv.Atoi, ",") t.AssertEqual([]int{42}, fl) t.AssertErrorIs(nil, fs.Parse([]string{"-test=1", "-test=2", "-test=42,84"})) t.AssertEqual([]int{1, 2, 42, 84}, fl) |