Make libnv headers includable from C++
ClosedPublic
Actions

Authored by rstone on Feb 17 2015, 5:42 PM.

Tags

None

Referenced Files

	F105778648: D1868.diff
	Fri, Dec 20, 1:54 PM

	Unknown Object (File)
	Thu, Dec 12, 4:06 PM

	Unknown Object (File)
	Wed, Dec 4, 5:44 PM

	Unknown Object (File)
	Wed, Dec 4, 4:38 AM

	Unknown Object (File)
	Tue, Dec 3, 11:48 AM

	Unknown Object (File)
	Fri, Nov 29, 3:04 AM

	Unknown Object (File)
	Thu, Nov 21, 4:14 PM

	Unknown Object (File)
	Thu, Nov 21, 1:40 PM

View All 71 Files

Subscribers

freebsd-net-list

pjd

Details

Reviewers

jfv

Commits

rS279421: Make libnv headers includable from C++

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

rstone updated this revision to Diff 3811.Feb 17 2015, 5:42 PM

rstone retitled this revision from to Make libnv headers includable from C++.

rstone edited the test plan for this revision. (Show Details)

rstone added a reviewer: jfv.

rstone added subscribers: pjd, Unknown Object (MLST).

jfv accepted this revision.Feb 18 2015, 7:59 PM

jfv edited edge metadata.

This revision is now accepted and ready to land.Feb 18 2015, 7:59 PM

Looks good to me, apart from the small nit above.

lib/libnv/nv.h
66	Could you please add an empty line after this one for consistency? Thanks.

rstone closed this revision.Feb 20 2015, 1:09 AM

Revision Contents
Changeset List

Path

Size

lib/

libnv/

dnv.h

4 lines

nv.h

3 lines

Diff 3811

View Options

lib/libnv/dnv.h