wc: fix word undercount with invalid byte sequences

[dhr412]

Closes #10322

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)

[codspeed-hq]

CodSpeed Performance Report

Merging this PR will not alter performance

_{Comparing dhr412:wc-invalid-bytes (8b6936f) with main (00f77cc)}

Summary

✅ 142 untouched benchmarks
⏩ 180 skipped benchmarks¹

Footnotes

1. 180 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[sylvestre]

some jobs are failing

[dhr412]

They're failing because the test_utf8 test expects a count of 2119, but after the commit it counts invalid byte sequences as words, increasing the total to 2178.

Should I go ahead and update the test?

[ChrisDryden]

The general guide is if the gnu implementation conflicts with the test then the test is wrong and needs to be updated. If gnu is giving the same value as the test then there is something wrong with the implementation

[ChrisDryden]

Validated it locally, this is tricky because there was a difference between different versions of GNU. Yes, the 9.9 version of GNU outputs 2178, so it should be okay to change this test

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)

[ChrisDryden]

Can you clean up the linting with clippy?

error: the borrowed expression implements the required traits
   --> tests/by-util/test_wc.rs:834:18
    |
834 |         .pipe_in(&[b'a', b' ', 0xff, b' ', b'b', b'\n'])
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `[b'a', b' ', 0xff, b' ', b'b', b'\n']`
    |

Looks all good to go just failing some of the CI tests

[github-actions]

GNU testsuite comparison:

Note: The gnu test tests/basenc/bounded-memory is now being skipped but was previously passing.