[bug] wrong condition of `current.matched` in src/util/location.js#27

[puxxustc]

This line https://github.com/vuejs/vue-router/blob/dev/src/util/location.js#L27

    } else if (current.matched) {

current.matched is an Array, so this condition should be

    } else if (current.matched.length === 0) {

[fnlctrl]

It's perfectly fine to put an array inside if, to check for its existence.

[puxxustc]

current.matched can be []

[fnlctrl]

It's guaranteed to be not emtpy. Please provide an actual reproduction if you believe there's a bug.

[posva]

@fnlctrl There's probably something wrong with that check, it may be unnecessary because in the flow types it's always an array, it cannot be absent. So we're probably missing a little improvement either in the types or in the codition there

[puxxustc]

Here is a stacktrace recorded by sentry:

vue@2.1.10, vue-loader@11.0.0, vue-router@2.2.1, vuex@2.1.2, vuex-router-sync@4.1.2, webpack@2.2.1

[fnlctrl]

Again, please provide a reproduction, thanks!