Differential D10959

Update the comment for the rootpath
ClosedPublic
Actions

Authored by bapt on May 27 2017, 3:01 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 30, 7:02 PM

	Unknown Object (File)
	Thu, Apr 30, 9:57 AM

	Unknown Object (File)
	Wed, Apr 29, 10:35 AM

	Unknown Object (File)
	Wed, Apr 29, 10:30 AM

	Unknown Object (File)
	Wed, Apr 29, 7:15 AM

	Unknown Object (File)
	Wed, Apr 29, 7:09 AM

	Unknown Object (File)
	Tue, Apr 28, 2:02 PM

	Unknown Object (File)
	Tue, Apr 28, 10:15 AM

Subscribers

Details

Reviewers

Commits

rS318999: Update the comments concerning net_parse_rootpath to reflect what it is now

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 9531
Build 9980: arc lint + arc unit

Event Timeline

bapt created this revision.May 27 2017, 3:01 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 27 2017, 3:01 PM

rgrimes added inline comments.May 27 2017, 4:52 PM

sys/boot/common/dev_net.c
376	Better would be (as NFS scheme) -> as an NFS scheme.
380	Typo? ips -> ip
381	Missing end ). Actually in all of these comments I dont see that the parens add any value

Pet rgrimes comment + small rewording

Harbormaster completed remote builds in B9531: Diff 28912.May 27 2017, 6:36 PM

add a forgotten word

Harbormaster completed remote builds in B9532: Diff 28913.May 27 2017, 6:37 PM

bapt marked 3 inline comments as done.May 27 2017, 6:37 PM

bapt added inline comments.

sys/boot/common/dev_net.c
381	actually reworded a bit

I think this should do nicely.

This revision is now accepted and ready to land.May 27 2017, 6:40 PM

Closed by commit rS318999: Update the comments concerning net_parse_rootpath to reflect what it is now (authored by bapt). · Explain WhyMay 27 2017, 6:46 PM

This revision was automatically updated to reflect the committed changes.

bapt marked an inline comment as done.

Revision Contents
Changeset List

Path

Size

sys/

boot/

common/

16 lines

Diff 28912

sys/boot/common/dev_net.c

Loading...