Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/external/bsd/tmux/dist/compat




To: source-changes-d%NetBSD.org@localhost

Subject: Re: CVS commit: src/external/bsd/tmux/dist/compat

From: Joerg Sonnenberger <joerg%britannica.bec.de@localhost>

Date: Mon, 30 May 2011 20:46:21 +0200


On Mon, May 30, 2011 at 06:10:36PM +0000, Christos Zoulas wrote:
> In article <20110530162059.1A6D2175D0%cvs.netbsd.org@localhost>,
> Joerg Sonnenberger <source-changes-d%NetBSD.org@localhost> wrote:
> >-=-=-=-=-=-
> >
> 
> >+#ifdef __clang__
> >+            char            buf[128];
> >+#else
> >             char            buf[CMSG_SPACE(sizeof(int))];
> >+#endif
> >     } cmsgbuf;
> 
> Wouldn't it be better insted to re-define CMSG_SPACE for clang like:
> #ifdef __clang__
> #define CMSG_SPACE(x) ((x) + 128)
> #else
> ...
> #endif

No. The real fix would be to teach the kernel compat32 handling for cmsg
and make it a proper constant again. 128 is simply large enough for all
platforms we have.

Joerg


Follow-Ups:

Re: CVS commit: src/external/bsd/tmux/dist/compat
From: Christos Zoulas


References:

Re: CVS commit: src/external/bsd/tmux/dist/compat
From: Christos Zoulas




Prev by Date: Re: CVS commit: src/external/bsd/tmux/dist/compat

Next by Date: Re: CVS commit: src/sys/kern

Previous by Thread: Re: CVS commit: src/external/bsd/tmux/dist/compat

Next by Thread: Re: CVS commit: src/external/bsd/tmux/dist/compat

Indexes:

reverse Date

reverse Thread

Old Index



Home | Main Index | Thread Index | Old Index