Okay but what is the default string version: fmt.Sprintf("%v", value) ?
Something else?


2013/10/28 Benoît Amiaux <notifications@github.com>

> In spew/dump.go:
>
> > @@ -241,6 +242,58 @@ func (d *dumpState) dumpSlice(v reflect.Value) {
> >  	}
> >  }
> >
> > +type valuesSorter struct {
> > +	values []reflect.Value
> > +}
> > +
> > +func (s *valuesSorter) Len() int {
> > +	return len(s.values)
> > +}
> > +
> > +func (s *valuesSorter) Swap(i, j int) {
> > +	s.values[i], s.values[j] = s.values[j], s.values[i]
> > +}
> > +
> > +func (s *valuesSorter) Less(i, j int) bool {
> > +	switch s.values[i].Kind() {
>
> I still think we don't care about natural order, just stability.
> Maybe we can at least remove the "notimplemented" case and replace it with
> the default string version
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/masagroup/go-spew/pull/1/files#r7245134>
> .
>