Change code that uses [de]packetize() to use "test-tool pkt-line". That we had these two concurrently in the tests is mostly just a historical anomaly/inconsistency that we can fix. Even the CC'd author of [de]packetize() has recently been using the test-tool, in af22a63c399 (sideband: diagnose more sideband anomalies, 2020-10-28). "Mostly" because it turns out we were missing one feature in the test-tool, to packetize raw input without any \n or \0 munging. This series adds that capabability, migrates those users, and finally removes the now-obsolete [de]packetize() functions. Ævar Arnfjörð Bjarmason (5): serve tests: add missing "extra delim" test serve tests: use test_cmp in "protocol violations" test tests: replace [de]packetize() shell+perl test-tool pkt-line tests: replace remaining packetize() with "test-tool pkt-line" test-lib-functions.sh: remove unused [de]packetize() functions t/helper/test-pkt-line.c | 13 ++++++ t/t5410-receive-pack-alternates.sh | 42 +++++++++++++----- t/t5411/once-0010-report-status-v1.sh | 12 ++--- t/t5500-fetch-pack.sh | 15 ++++--- t/t5530-upload-pack-error.sh | 24 +++++----- t/t5562-http-backend-content-length.sh | 16 ++++--- t/t5570-git-daemon.sh | 22 ++++++---- t/t5704-protocol-violations.sh | 61 ++++++++++++++++++-------- t/test-lib-functions.sh | 42 ------------------ 9 files changed, 136 insertions(+), 111 deletions(-) -- 2.32.0.636.g43e71d69cff
