errors unmarshalling inline structs are swallowed in Strict mode

[quentinmit]

While trying to implement a custom marshaler/unmarshaler for durations: https://play.golang.org/p/TQnD5pV-381

I discovered that any errors that occur when unmarshaling inline structs get swallowed. I think this is happening because of this block of code in decode.go:

https://github.com/goccy/go-yaml/blob/v1.9.2/decode.go#L984

			newFieldValue, err := d.createDecodedNewValue(ctx, fieldValue.Type(), mapNode)
			if d.disallowUnknownField {
				var ufe *unknownFieldError
				if xerrors.As(err, &ufe) {
					err = nil
				}

				if err = d.deleteStructKeys(fieldValue.Type(), unknownFields); err != nil {
					return errors.Wrapf(err, "cannot delete struct keys")
				}
			}

			if err != nil {
				if foundErr != nil {
					continue
				}

The call to deleteStructKeys overwrites the err variable from createDecodedNewValue before it has been checked. Perhaps this is as simple as adding a missing colon.

[quentinmit]

That was indeed the fix, so I sent a PR.

[quentinmit]

I believe this is now fixed with those two PRs.