Bug in isArray implementation

[jkodumal]

The body of isArray is as follows:

    for _, c := range buf {
        switch c {
        case ' ':
        case '\n':
        case '\t':
            continue
        case '[':
            return true
        default:
            break
        }
    }

This seems innocuous enough, but the break is defined to be a break on the switch statement, not the for loop. This means that calls to isArray will always return true if there's a [ anywhere in the JSON document.

[evanphx]

Ah! Oops. Yeah, looks like a definite bug.

[jkodumal]

I'll open a pull request. I've also noticed that paths containing array indices aren't completely supported either-- will submit a PR for that as well.

[klauspost]

Is this closed by #4?

[evanphx]

Yup!