Now checking if a field is not anonymous before

skipping it when walking over values in an struct, to make sure that we
don't skip validly accessible (exported) embedded values in an
unexported field.

This new checking logic is the new recommended strategy for reflective
struct walking as of the Go 1.6 release.

Related/References:

- Go 1.6 reflect release note: https://golang.org/doc/go1.6#reflect
- Issue: https://golang.org/issue/12367
    (golang/go#12367)
- CL: https://golang.org/cl/14085
    (https://go-review.googlesource.com/#/c/14085/)
- Commit: golang/go@afe9837

Author: Rican7

query/encode.go

@@ -138,7 +138,7 @@ func reflectValue(values url.Values, val reflect.Value, scope string) error {
 	typ := val.Type()
 	for i := 0; i < typ.NumField(); i++ {
 		sf := typ.Field(i)
-		if sf.PkgPath != "" { // unexported
+		if sf.PkgPath != "" && !sf.Anonymous { // unexported
 			continue
 		}