Paths

Table of Contentst

Differential D59588

chflags: Alias nocache to noucache for consistency
ClosedPublic
Actions

Authored by ziaee on Fri, Sep 11, 12:14 PM.

Tags

None

Referenced Files

	F175363480: D59588.id186584.diff
	Sat, Oct 10, 7:30 AM

	F175346089: D59588.id186577.diff
	Sat, Oct 10, 4:26 AM

	F175342272: D59588.diff
	Sat, Oct 10, 3:35 AM

	Unknown Object (File)
	Thu, Oct 8, 12:22 PM

	Unknown Object (File)
	Thu, Oct 8, 3:20 AM

	Unknown Object (File)
	Wed, Oct 7, 4:37 AM

	Unknown Object (File)
	Sun, Oct 4, 7:04 PM

	Unknown Object (File)
	Sat, Oct 3, 1:56 PM

View All 62 Files

Subscribers

Details

Reviewers

fuz
rmacklem

Commits

rG49b4137da6ea: chflags: Alias nocache to noucache for consistency

Summary

Requested by: fuz

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ziaee created this revision.Fri, Sep 11, 12:14 PM

Herald added a subscriber: imp. · View Herald TranscriptFri, Sep 11, 12:14 PM

ziaee requested review of this revision.Fri, Sep 11, 12:14 PM

Harbormaster completed remote builds in B76771: Diff 186457.Fri, Sep 11, 12:14 PM

fuz added inline comments.Fri, Sep 11, 1:10 PM

lib/libc/gen/strtofflags.c
83–88	Try this, should build fine and avoids the duplicated `nousystem` line.

rmacklem accepted this revision.Fri, Sep 11, 1:39 PM

rmacklem added inline comments.

lib/libc/gen/strtofflags.c
83–88	You can do so, but I did it the way I did it to maintain a consistent style. style(9) says that style should only be changed for major re-writes. But choose whichever way you prefer.

This revision is now accepted and ready to land.Fri, Sep 11, 1:39 PM

Accepted subject to the changes I have proposed.

lib/libc/gen/strtofflags.c
83–88	It's not really a style change. More of a simplification of lines that'll be touched anyway.

so like this?

This revision now requires review to proceed.Sun, Sep 13, 10:14 AM

Harbormaster completed remote builds in B76826: Diff 186577.Sun, Sep 13, 10:14 AM

Looks good!
Approved for src.

This revision is now accepted and ready to land.Sun, Sep 13, 11:17 AM

Closed by commit rG49b4137da6ea: chflags: Alias nocache to noucache for consistency (authored by ziaee). · Explain WhySun, Sep 13, 11:43 AM

This revision was automatically updated to reflect the committed changes.

ziaee added a commit: rG49b4137da6ea: chflags: Alias nocache to noucache for consistency.

Revision Contents
Changeset List

Path

Size

bin/

chflags/

4 lines

lib/

libc/

gen/

5 lines

Diff 186584

bin/chflags/chflags.1

Loading...

lib/libc/gen/strtofflags.c

Loading...