use an interface{} instead of reflect.Value

Author: peterebden

group.go

@@ -74,8 +74,8 @@ func (g *Group) AddGroup(shortDescription string, longDescription string, data i
 }
 
 // AddOption adds a new option to this group.
-func (g *Group) AddOption(option *Option, value reflect.Value) {
-	option.value = value
+func (g *Group) AddOption(option *Option, data interface{}) {
+	option.value = reflect.ValueOf(data)
 	option.group = g
 	g.options = append(g.options, option)
 }

group_test.go

@@ -1,7 +1,6 @@
 package flags
 
 import (
-	"reflect"
 	"testing"
 )
 
@@ -262,7 +261,7 @@ func TestAddOptionNonOptional(t *testing.T) {
 	p := NewParser(&opts, Default)
 	p.AddOption(&Option{
 		LongName: "test",
-	}, reflect.ValueOf(&opts.Test))
+	}, &opts.Test)
 	_, err := p.ParseArgs([]string{"--test"})
 	if err != nil {
 		t.Errorf("unexpected error: %s", err)