bsdgrep(1): Don't exit before processing every file

[kevans]

Summary

Given an empty pattern (i.e. grep "" A B), bsdgrep(1)
would previously exit() with the appropriate error code upon
encountering an empty file. Likely intended as an optimization,
but this behavior is technically incorrect since an empty pattern
should match every line.

PR: 220924

[emaste]

usr.bin/grep/util.c

264 ↗

(On Diff #31091)

Doesn't this imply that an empty pattern matches once in an empty file?

[kevans]

usr.bin/grep/util.c

264 ↗

(On Diff #31091)

...right, still not correct behavior. Now that I reread this block, it should also do the wrong thing with printf "\nA" | grep "" and omit the second line.

I'll amend the test for both the problem you mentioned and this one.

[kevans]

usr.bin/grep/util.c

264 ↗

(On Diff #31091)

Nope, misread again. Drats

[kevans]

Empty files tell no tales nor do they match an empty string