Return early from more cache tag operations

Started an MR. Turns out we do pass empty arrays to these during tests so they need the same treatment as isValid(). Have run out of time - we still need to do the same for the trait methods.

Added the early returns.

Geez those counts went way down!

Reviewing the change the early return seems straight forward and makes sense.

Just a small nitpick, we're returning early twice with basically the same conditions.

Agreed with the nitpick and removed that change from the MR, leaving RTBC since it's just reverting a change.

RTBC++

Committed and pushed 08bf243d3f to 11.x and 9bc3874fce to 10.3.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.