feat(oiiotool): flexible offset notation with commas

[lgritz]

Fixes #5207

Sometimes the WxH+X+Y and +X+Y notation (borrowed from X11!) can be a little awkward. This patch extends the accepted syntax to allow commas to separate the offset components, for example also accepting WxY,X,Y and X,Y.

This affects the commands

--create
--crop
--cut
--fit
--fullsize
--origin
--originoffset
--paste
--pattern
--printstats
--resize

[Copilot]

Pull request overview

Extends oiiotool's geometry parsing to accept a comma-separated alternative (WxH,X,Y and X,Y) in addition to the X11-style WxH+X+Y / +X+Y form. This resolves an awkward case where signed-prefix syntax breaks when offset components are produced by expressions whose sign isn't known in advance (issue #5207).

Changes:

• Update scan_offset and scan_res_offset to try the legacy +/- syntax first and fall back to comma-separated parsing.
• Add unit tests for the comma-separated forms in unit_test_adjust_geometry, and add CLI runtime tests in the oiiotool-attribs testsuite.
• Document the new comma-syntax form across all affected commands in oiiotool.rst.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/oiiotool/oiiotool.cpp	Extends scan_offset/scan_res_offset to accept comma-separated origin/offset; adds unit tests.
src/doc/oiiotool.rst	Documents new comma notation for --create, --crop, --cut, --fit, --fullsize, --origin, --originoffset, --paste, --pattern, --printstats, --resize.
testsuite/oiiotool-attribs/run.py	Adds runtime tests covering both X11 and comma origin/originoffset syntax.
testsuite/oiiotool-attribs/ref/out.txt	Updates expected output to match new tests.
testsuite/oiiotool-attribs/ref/out-jpeg9d.txt	Updates expected output for jpeg9d variant to match new tests.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.