Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 11 Apr 2013 11:05:28 -0700
From:      Juli Mallett <jmallett@FreeBSD.org>
To:        Gleb Smirnoff <glebius@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Bruce Evans <brde@optusnet.com.au>
Subject:   Re: svn commit: r249355 - head/lib/libkvm
Message-ID:  <CACVs6=9UL9Z8x8BVVM9mLkfF_-=F=%2BUdH-yG_35_-vgH_NuqYA@mail.gmail.com>
In-Reply-To: <20130411180013.GB76816@FreeBSD.org>
References:  <201304110730.r3B7Uo6d067302@svn.freebsd.org> <20130411175308.Q1435@besplex.bde.org> <20130411082457.GS76816@FreeBSD.org> <20130411184049.W1641@besplex.bde.org> <20130411091418.GW76816@FreeBSD.org> <20130411201652.F1911@besplex.bde.org> <20130411180013.GB76816@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Apr 11, 2013 at 11:00 AM, Gleb Smirnoff <glebius@freebsd.org> wrote:
>   Bruce,
>
> On Thu, Apr 11, 2013 at 09:07:25PM +1000, Bruce Evans wrote:
> B> Just routine avoidance of namespace pollution.  This is easy in such a
> B> simple header.
>
> Sorry, with all respect, but I can't call including sys/types.h
> a namespace pollution.
>
> Ok, even you force me to name it that way, still I would prefer
> namespace pollution instead of handmade copy pasted typedefs.

But Gleb, making such changes unilaterally is a bit of a leap.  The
project has mostly accepted Bruce's wisdom about trying to minimize
and reduce namespace pollution.  Now, this isn't a standard header so
it's quite a bit less of a concern, but it's not no concern.  If you
think that we should reverse our trend on including
namespace-polluting headers in system headers, we should discuss that
on arch@, and it shouldn't be something that's done without any
discussion or consideration.

Should we expect further changes of this nature (and of the proposed
nature removing __size_t and __ssize_t use) if you make changes to
other headers as part of your work?  Are you going to add
<sys/types.h> to every header currently using <sys/_types.h> in a
single go, or will you be doing that a little at a time when making
functional changes?

Thanks,
Juli.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACVs6=9UL9Z8x8BVVM9mLkfF_-=F=%2BUdH-yG_35_-vgH_NuqYA>