cc'ing @bardbloom again.  Yes, in fact if we remove the shouldBeIgnored variable and move the containsAnyIgnoreFilename call into the if, the short-circuiting || operator will have the same effect.