GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — `sdiff` initial implementation by zxvfc · Pull Request #159 · uutils/diffutils

Conversation

@zxvfc

@zxvfc zxvfc commented Jan 9, 2026

Copy link
Copy Markdown

Initial implementation of sdiff

So far, just reuse side_diff implementation to just show a side by side diff (same as diff -y)

Currently supports arguments: from_file, to_file, expand_tabs, tabsize, width since it is already was implemented for the diff and used by side_diff. The rest and interactive mode is supposed to be added in following PRs.

Relates to: #16

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
zxvfc added 4 commits January 7, 2026 18:18
@zxvfc
Extract shared code to utils
96115ea
@zxvfc
Make side_diff use separate set of formatting params
4be21f1
@zxvfc
Add sdiff initiall implementation
ed9f154
@zxvfc
Add call of sdiff to main and basic integration test
7355680
@zxvfc zxvfc mentioned this pull request Jan 9, 2026
sdiff command is not implemented #16
Open
zxvfc added 3 commits January 10, 2026 11:27
@zxvfc
Fix usage side_diff in fuzzers
a5a1baf
@zxvfc
Fix codestyle
f3c4c78
@zxvfc
Fix failed test
17bc12c
@codecov

codecov Bot commented Jan 10, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.78495% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.41%. Comparing base (c38fe5f) to head (9983fb2).
⚠️ Report is 78 commits behind head on main.

Files with missing lines Patch % Lines
src/sdiff.rs 90.47% 27 Missing and 1 partial ⚠️
src/diff.rs 40.00% 9 Missing ⚠️
src/utils.rs 97.67% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #159      +/-   ##
==========================================
+ Coverage   86.25%   86.41%   +0.15%     
==========================================
  Files          13       14       +1     
  Lines        6222     6565     +343     
  Branches      519      530      +11     
==========================================
+ Hits         5367     5673     +306     
- Misses        854      890      +36     
- Partials        1        2       +1
Flag Coverage Δ
macos_latest 86.36% <89.78%> (+0.15%) ⬆️
ubuntu_latest 86.41% <89.78%> (+0.12%) ⬆️
windows_latest 19.81% <18.81%> (-0.35%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
zxvfc added 2 commits January 11, 2026 16:08
@zxvfc
Add more tests for new code
06e1d92
@zxvfc
More tests for sdiff params parsing
9983fb2
GunterSchmidt pushed a commit to GunterSchmidt/diffutils that referenced this pull request Mar 3, 2026
Gunter Schmidt
Merged sdiff logic from PR uutils#159.
e1f02bf 
This is now a good option parser and has basic sdiff compare functionality. This is a good base for further development.
This was referenced Mar 3, 2026
Feat sdiff initial with parser #187
Closed
Fix both files can return an error #188
Closed
GunterSchmidt pushed a commit to GunterSchmidt/diffutils that referenced this pull request Mar 4, 2026
Gunter Schmidt
code cleanup
f1f0dbc 
* ArgParser: removed default option, which does not exist in GNU diffutils
* fix test 'cannot_read_files' (same as PR uutils#188)
* merge test from PR uutils#159
This was referenced Mar 9, 2026
Feat sdiff inital with parser #193
Closed
Breaking: Adapt coreutils dev file structure #201
Closed
@marc-hb

marc-hb commented May 7, 2026

Copy link
Copy Markdown
Contributor

#188 adds commits on top of this and #193 apparently wants to supersede it completely... without any comment here - pretty confusing!
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@sylvestre

sylvestre commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

sorry but it needs to be rebased
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
1 similar comment
@sylvestre

sylvestre commented Nov 10, 2026

Copy link
Copy Markdown
Collaborator

sorry but it needs to be rebased
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@zxvfc @marc-hb @sylvestre
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.