In `@dvd/dvd_split.ixx`:
- Around line 139-143: The existing check around iso_path unconditionally logs
and returns when the .iso exists, ignoring options.overwrite; update the logic
in dvd_split.ixx to mirror generate_extra_xbox behavior by respecting
options.overwrite: when std::filesystem::exists(iso_path) evaluate
options.overwrite and either remove/replace the file (or allow continuing) if
options.overwrite is true, otherwise log the warning referencing
options.image_name and return; ensure you reference iso_path, options.overwrite
and options.image_name so the overwrite behavior is consistent with
generate_extra_xbox.
- Around line 157-171: The loop variable psn is being incremented twice (once by
the for loop ++psn and again by the psn += 1 at the end), causing every other
sector to be skipped; remove the redundant psn += 1 statement so the
for(uint32_t psn = -DVD_LBA_START;; ++psn) loop alone controls iteration,
leaving the body (including sdram_fs.read, RecordingFrame_to_DataFrame,
scrambler.descramble and iso_fs.write) unchanged.
- Line 163: The call uses a C-style cast of the std::vector object (rf) to
RecordingFrame&, which reads garbage; instead take a reference to the struct
stored inside the vector's data (e.g., reinterpret_cast a reference or pointer
to the first byte of rf) and pass that to RecordingFrame_to_DataFrame, and
ensure rf.size() >= sizeof(RecordingFrame) before casting. Update the call site
(RecordingFrame_to_DataFrame and rf) to use a memory-based cast (e.g.,
reinterpret_cast to RecordingFrame& from &rf[0] or from rf.data()) and add a
size check to avoid out-of-bounds access.

In `@dvd/dvd_split.ixx`:
- Line 6: Remove the unused `#include` <numeric> from dvd_split.ixx; locate the
include directive (the line containing "<numeric>") and delete it so the file no
longer pulls in the numeric header since no std::accumulate/std::iota/etc. are
used in the compilation unit.

In `@dvd/dvd_split.ixx`:
- Line 156: Remove the unused local variable bytes_read declared in extract_iso
(it's never assigned or read); either delete the bytes_read declaration from the
extract_iso function or, if the intention was to track read bytes, replace
usages where read counts are ignored with assignments to bytes_read and then use
it for validating reads or loop termination—locate the declaration of bytes_read
in extract_iso and remove or repurpose it accordingly.