Code review comment for lp:~niemeyer/goyaml/fix-map-init

Reviewers: mp+86396_code.launchpad.net,

Message:
Please take a look.

Description:

https://code.launchpad.net/~niemeyer/goyaml/fix-map-init/+merge/86396

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/5503044/

Affected files:
   M decode.go
   M decode_test.go

Index: decode.go
=== <email address hidden> >
<email address hidden>
=== modified file 'decode.go'
--- decode.go 2011-11-24 19:47:20 +0000
+++ decode.go 2011-12-20 13:31:40 +0000
@@ -213,9 +213,9 @@
  //
  // It's a slightly convoluted case to handle properly:
  //
-// - Nil pointers should be zeroed out, unless being set to nil
-// - We don't know at this point yet what's the value to SetYAML() with.
-// - We can't separate pointer deref/init and setter checking, because
+// - nil pointers should be initialized, unless being set to nil
+// - we don't know at this point yet what's the value to SetYAML() with.
+// - we can't separate pointer deref/init and setter checking, because
  // a setter may be found while going down a pointer chain.
  //
  // Thus, here is how it takes care of it:
@@ -417,6 +417,10 @@
   if out.Kind() != reflect.Map {
    return false
   }
+ if out.IsNil() {
+ out.Set(reflect.MakeMap(out.Type()))
+ }
+
   outt := out.Type()
   kt := outt.Key()
   et := outt.Elem()

Index: decode_test.go
=== <email address hidden> >
<email address hidden>
=== modified file 'decode_test.go'
--- decode_test.go 2011-11-24 19:47:20 +0000
+++ decode_test.go 2011-12-20 13:31:40 +0000
@@ -81,16 +81,9 @@

   // Structs and type conversions.
   {"hello: world", &struct{ Hello string }{"world"}},
- {"a: {b: c}", &struct {
- A struct {
- B string
- }
- }{struct{ B string }{"c"}}},
- {"a: {b: c}", &struct {
- A *struct {
- B string
- }
- }{&struct{ B string }{"c"}}},
+ {"a: {b: c}", &struct{ A struct{ B string } }{struct{ B string }{"c"}}},
+ {"a: {b: c}", &struct{ A *struct{ B string } }{&struct{ B string }{"c"}}},
+ {"a: {b: c}", &struct{ A map[string]string
}{map[string]string{"b": "c"}}},
   {"a: 1", &struct{ A int }{1}},
   {"a: [1, 2]", &struct{ A []int }{[]int{1, 2}}},
   {"a: 1", &struct{ B int }{0}},