Dereference default values before stringing (#40)

Author: dlo9

flagset_test.go

@@ -5,6 +5,8 @@ import (
 	"flag"
 	"fmt"
 	"os"
+	"reflect"
+	"strconv"
 	"testing"
 	"time"
 
@@ -462,6 +464,32 @@ func TestDefaultsViaTag(t *testing.T) {
 `, buf.String())
 }
 
+func TestSimpleTypeDefaultsViaTag(t *testing.T) {
+	type Config struct {
+		Count int16 `default:"42"`
+	}
+
+	flagsfiller.RegisterSimpleType(func(s string, tag reflect.StructTag) (int16, error) {
+		i, err := strconv.ParseInt(s, 10, 16)
+		return int16(i), err
+	})
+
+	var config Config
+
+	filler := flagsfiller.New()
+
+	var flagset flag.FlagSet
+	err := filler.Fill(&flagset, &config)
+	require.NoError(t, err)
+
+	buf := grabUsage(flagset)
+
+	assert.Equal(t, `
+  -count value
+    	 (default 42)
+`, buf.String())
+}
+
 func TestBadDefaultsViaTag(t *testing.T) {
 	type BadBoolConfig struct {
 		Enabled bool `default:"wrong"`

simple.go

@@ -32,7 +32,7 @@ func (v *simpleType[T]) String() string {
 	if v.val == nil {
 		return fmt.Sprint(nil)
 	}
-	return fmt.Sprint(v.val)
+	return fmt.Sprint(*v.val)
 }
 
 func (v *simpleType[T]) StrConverter(s string) (T, error) {