[ryanduguid]

Summary

ZipIterable::len() returned files.remaining, which counts down during a pass, so once archive.files() had been iterated, archive.len() reported 0 (or the number of entries not yet visited). Its doc says "Number of entries in the central directory". It now returns the central directory's entry count, the same value ZipIterableFiles starts from and resets to. ZipIterableFiles::len() still reports what is left in the pass.

Evidence

• Before: the new archive_len_is_unchanged_by_iteration test fails on master: left: 1, right: 2.
• After: cargo test --test iterable_zip: 15 passed.

Tests pass with default features, --all-features and --no-default-features on stable (rustc 1.93.1, Linux). cargo fmt --check --all is clean, and cargo clippy --all-targets is clean with default and all features.

Risk

Only ZipIterable::len() and is_empty() change, and only after a pass has started. ZipIterable is not in a release yet.

Unverified

• MSRV and nightly builds, and Windows and macOS runs: left to CI.

• cargo clippy --all-targets --no-default-features fails on master at src/extra_fields/extra_field.rs:369, which this PR doesn't touch.

• The PR title must conform to Conventional Commits and start with one of the types specified by the Angular convention.

---

^{Need help on this PR? Tag @codesmith-bot with what you need.}