Page MenuHomeFreeBSD
Differential D48196

tee: minor cleanup
AcceptedPublic
Actions

Authored by kevans on Tue, Dec 24, 6:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 4, 4:23 PM
Unknown Object (File)
Tue, Dec 31, 5:46 PM
Unknown Object (File)
Thu, Dec 26, 5:18 AM
Unknown Object (File)
Thu, Dec 26, 3:36 AM
Unknown Object (File)
Wed, Dec 25, 8:59 AM
Unknown Object (File)
Wed, Dec 25, 6:44 AM
Subscribers
imp

Details

Reviewers
des
emaste
asomers
ngie
Group Reviewers
tests
Summary

Pull the open flags out of the loop into a local var. They won't be
changing, so this is marginally more readable.

Adds some extra brackets around the loop in preparation for a future
change that may try to fallback to opening the path as a socket if we
get an EOPNOTSUPP.

No functional change.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 61322
Build 58206: arc lint + arc unit

Event Timeline

kevans created this revision.Tue, Dec 24, 6:11 PM
Herald added a subscriber: imp. · View Herald TranscriptTue, Dec 24, 6:11 PM
kevans requested review of this revision.Tue, Dec 24, 6:11 PM
kevans added a parent revision: D48195: tee: add some basic tests.
Harbormaster completed remote builds in B61321: Diff 148375.Tue, Dec 24, 6:11 PM
kevans updated this revision to Diff 148378.Tue, Dec 24, 10:06 PM

omode -> oflags

Harbormaster completed remote builds in B61322: Diff 148378.Tue, Dec 24, 10:06 PM
asomers accepted this revision.Tue, Dec 24, 10:12 PM
This revision is now accepted and ready to land.Tue, Dec 24, 10:12 PM
kevans added a child revision: D48197: tee: try opening as a UNIX socket if open(2) fails.Tue, Dec 24, 10:30 PM
emaste accepted this revision.Wed, Dec 25, 2:43 AM

I think the added braces are an improvement even absent the subsequent change.

ngie accepted this revision.Wed, Dec 25, 6:42 PM
des accepted this revision.Thu, Dec 26, 1:35 PM
des added inline comments.
usr.bin/tee/tee.c
64

I would merge the two int lines and the two char * lines.

102

I'd prefer braces around this line.

kevans marked 2 inline comments as done.Fri, Dec 27, 4:10 AM
kevans added inline comments.
usr.bin/tee/tee.c
64

Also re-sorted

Revision Contents
Changeset List

PathSize
usr.bin/
tee/
14 lines

Diff 148378

View Options

usr.bin/tee/tee.c

Loading...