Skip to content

diff: add -N and --unidirectional-new-file - #316

Open
sylvestre wants to merge 3 commits into
mainfrom
diff-new-file
Open

sylvestre wants to merge 3 commits into
mainfrom
diff-new-file

Conversation

@sylvestre

Copy link
Copy Markdown
Collaborator

Stack created with GitHub Stacks CLI • Give Feedback 💬

@sylvestre
sylvestre added this pull request to stack #319 October 24, 2026 09:22
@github-actions

Copy link
Copy Markdown

GNU diffutils testsuite comparison:

Test results comparison:
  Current:   TOTAL: 33 / PASSED: 9 / FAILED: 20 / SKIPPED: 4
  Reference: TOTAL: 33 / PASSED: 8 / FAILED: 21 / SKIPPED: 4

Changes from main branch:
  TOTAL: +0
  PASSED: +1
  FAILED: -1

Test improvements (1):
  + new-file

@codspeed

codspeed Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

1 warning

⚠️ 24 benchmarks are dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

✅ 26 untouched benchmarks


Comparing diff-new-file (6145ab7) with main (6b3d629)

Open in CodSpeed

jayhemnani9910 and others added 3 commits October 24, 2026 14:50
The normal format printed a single added or deleted line as a range,
"1a2,2" and "2,2d1", where GNU diff and POSIX write "1a2" and "2d1",
and -e printed "2,2d" for "2d". Changed lines already had this
(see #10); do the same for added and deleted ones.
@sylvestre
sylvestre removed this pull request from stack #319 October 24, 2026 13:01
@sylvestre
sylvestre changed the base branch from diff-normal-ranges to main October 24, 2026 13:02
@sylvestre
sylvestre added this pull request to stack #328 October 24, 2026 13:02
@sylvestre
sylvestre marked this pull request as ready for review October 24, 2026 13:14
Copilot AI balanced review requested due to automatic review settings October 24, 2026 13:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants