Date: Sun, 7 Jun 2015 08:54:25 +0000 (UTC) From: Andriy Gapon <avg@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r284107 - head/sys/cddl/compat/opensolaris/sys Message-ID: <201506070854.t578sPdR012092@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: avg Date: Sun Jun 7 08:54:25 2015 New Revision: 284107 URL: https://svnweb.freebsd.org/changeset/base/284107 Log: compat nvpair.h: make sure that the names are mangled only for kernel Currently there is no good reason to mangle the userland API. The change was introduced in eac1d566b46edef765754203bef22c75c1699966, r279437. Also see https://reviews.freebsd.org/D1881. I am still convinced that nv should not have introduced intentionally conflicting API. Discussed with: rstone X-MFC with: r279437 Sponsored by: ClusterHQ Modified: head/sys/cddl/compat/opensolaris/sys/nvpair.h Modified: head/sys/cddl/compat/opensolaris/sys/nvpair.h ============================================================================== --- head/sys/cddl/compat/opensolaris/sys/nvpair.h Sun Jun 7 06:30:25 2015 (r284106) +++ head/sys/cddl/compat/opensolaris/sys/nvpair.h Sun Jun 7 08:54:25 2015 (r284107) @@ -29,6 +29,8 @@ #ifndef _OPENSOLARIS_SYS_NVPAIR_H_ #define _OPENSOLARIS_SYS_NVPAIR_H_ +#ifdef _KERNEL + /* * Some of the symbols in the Illumos nvpair library conflict with symbols * provided by nv(9), so we use this preprocessor hack to avoid the conflict. @@ -254,6 +256,8 @@ #define nvpair_unpack illumos_nvpair_unpack #define nvpair_unpack_descriptor illumos_nvpair_unpack_descriptor +#endif /* _KERNEL */ + #include_next <sys/nvpair.h> #endif
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201506070854.t578sPdR012092>