Page MenuHomeFreeBSD
Differential D42010

split: Further option parsing improvements.
ClosedPublic
Actions

Authored by des on Sep 28 2023, 1:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 25, 5:30 AM
Unknown Object (File)
Dec 3 2024, 7:31 PM
Unknown Object (File)
Nov 25 2024, 2:50 PM
Unknown Object (File)
Nov 25 2024, 2:50 PM
Unknown Object (File)
Nov 25 2024, 2:50 PM
Unknown Object (File)
Nov 25 2024, 2:49 PM
Unknown Object (File)
Nov 20 2024, 9:00 AM
Unknown Object (File)
Nov 19 2024, 6:43 AM
View All 49 Files
Subscribers
imp
kevans

Details

Reviewers
christos
kevans
Group Reviewers
Klara
Commits
rGdb95aeb922c9: split: Further option parsing improvements.
rGb3cef2d2f538: split: Further option parsing improvements.
rGd7399551b028: split: Further option parsing improvements.
Summary
  • Whenever possible, use strtonum() to parse numeric arguments.
  • Improve usefulness and consistency of error messages.
  • While here, fix some type and style issues.

Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Sep 28 2023, 1:18 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 28 2023, 1:18 PM
des requested review of this revision.Sep 28 2023, 1:18 PM
Harbormaster completed remote builds in B53754: Diff 127900.Sep 28 2023, 1:18 PM
des added a child revision: D42011: split: Improve GNU compatibility..Sep 28 2023, 1:18 PM
christos added a comment.Sep 28 2023, 1:26 PM

LGTM. It might also be better to have the style changes into a different patch.

christos accepted this revision.Sep 28 2023, 1:28 PM
This revision is now accepted and ready to land.Sep 28 2023, 1:28 PM
kevans accepted this revision.Sep 28 2023, 4:18 PM
kevans added a subscriber: kevans.
kevans added inline comments.
usr.bin/split/split.c
247–249

Indentation here and the same below on 258 could be fixed while you're in the area on style changes

des updated this revision to Diff 127962.Sep 28 2023, 4:21 PM

fix indentation

This revision now requires review to proceed.Sep 28 2023, 4:21 PM
Harbormaster completed remote builds in B53759: Diff 127962.Sep 28 2023, 4:22 PM
des marked an inline comment as done.Sep 28 2023, 4:24 PM
des added inline comments.
usr.bin/split/split.c
247–249

Done. Note that the error checking is insufficient (if your disk is full it will print “write: No error”) but I'll leave that alone for now.

des marked an inline comment as done.Sep 28 2023, 4:24 PM
kevans accepted this revision.Sep 28 2023, 4:25 PM
This revision is now accepted and ready to land.Sep 28 2023, 4:25 PM
Closed by commit rGd7399551b028: split: Further option parsing improvements. (authored by des). · Explain WhySep 28 2023, 4:38 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGd7399551b028: split: Further option parsing improvements..
des added a commit: rGb3cef2d2f538: split: Further option parsing improvements..Oct 5 2023, 12:03 AM
des added a commit: rGdb95aeb922c9: split: Further option parsing improvements..Oct 5 2023, 3:56 PM

Revision Contents
Changeset List

PathSize
usr.bin/
split/
84 lines

Diff 127964

View Options

usr.bin/split/split.c

Loading...