Support F (output filename) command

[dspinellis]

This also requires storing input_name as PathBuf to avoid lossy conversions when outputting an OS-specific byte sequence representing the file name.

[Copilot]

Pull request overview

Adds support for GNU sed’s F command (print current input file name), including correct handling of non-ASCII file names by storing input_name as a PathBuf and emitting OS-encoded bytes for the filename.

Changes:

• Implement F command execution in the sed processor.
• Store ProcessingContext.input_name as PathBuf (and adjust initialization / error formatting accordingly).
• Add fixtures and tests for F with regular files, non-ASCII file names, stdin, and --posix rejection.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/fixtures/sed/output/filename_αρχείο1	Expected output for F with a non-ASCII input filename (interleaved with normal printing).
tests/fixtures/sed/output/filename_file	Expected output for F under -n (filename printed per line).
tests/fixtures/sed/input/αρχείο1	Non-ASCII input fixture used to validate filename handling.
tests/by-util/test_sed.rs	New test coverage for F (file, non-ASCII path, stdin, and --posix behavior).
src/sed/processor.rs	Runtime implementation of F and PathBuf propagation for input_name.
src/sed/mod.rs	Initialize input_name as PathBuf for stdin (-).
src/sed/error_handling.rs	Quote PathBuf input names in runtime error messages.
src/sed/compiler.rs	Allow F only when not in --posix mode.
src/sed/command.rs	Update ProcessingContext.input_name type to PathBuf.

Suppressed comments (1)

src/sed/processor.rs:744

• This builds a Vec from the path bytes and then write_bytes immediately clones again internally, causing an extra allocation/copy per input line. You can avoid the first allocation by writing the path bytes directly and then writing a newline separately.

                    let mut bytes = context.input_name.as_os_str().as_encoded_bytes().to_vec();
                    bytes.push(b'\n');
                    output.write_bytes(&bytes)?;

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot wasn't able to review any files in this pull request.

[codspeed-hq]

Merging this PR will degrade performance by 6.17%

❌ 1 regressed benchmark
✅ 10 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
❌	number_fix	1.4 s	1.5 s	-6.17%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

---

_{Comparing dspinellis:main (d4cced3) with main (d837ddf)}