Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-04 Thread Martin Husemann
On Mon, Jun 03, 2013 at 08:00:58PM +, David Holland wrote: > > #define__DECONST(t, a) const_cast(a) > > and I suppose one could also do something like this: > > #define __DECONST(t, v) \ >(sizeof((v) - (const t)0) ? (t)(unsigned long)(v) : 0) Sounds good to me. I seriously wonder w

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-03 Thread David Holland
On Sun, Jun 02, 2013 at 12:40:12AM +, Christos Zoulas wrote: > > > > Log Message: > > > > apply __UNCONST() if it exists. > > > > > > Reminds me: should/can __UNCONST() be revisited w.r.t. the thread below? > > > > > > http://lists.freebsd.org/pipermail/freebsd-arch/2013-May/014371.htm

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-03 Thread Alan Barrett
On Mon, 03 Jun 2013, Christos Zoulas wrote: | > Well, there is an advantage that the FreeBSD one has over ours. It can be | > used in c++ with -Wold-style-cast, if defined as: | > | > #define __DECONST(t, a) const_cast(a) | | That, and why is it cast to an unsigned long and not uintptr_t? So th

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-02 Thread Christos Zoulas
On Jun 3, 5:34am, jruoho...@iki.fi (Jukka Ruohonen) wrote: -- Subject: Re: CVS commit: xsrc/external/mit/xauth/dist | > Well, there is an advantage that the FreeBSD one has over ours. It can be | > used in c++ with -Wold-style-cast, if defined as: | > | > #define __DECONST(t, a

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-02 Thread Jukka Ruohonen
On Sun, Jun 02, 2013 at 12:40:12AM +, Christos Zoulas wrote: > > > Reminds me: should/can __UNCONST() be revisited w.r.t. the thread below? > > > > > > http://lists.freebsd.org/pipermail/freebsd-arch/2013-May/014371.html > > > >How/why? Yes, it's unsafe. That's the *point*. > > Well, there

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-01 Thread Christos Zoulas
In article <20130601232927.ga19...@netbsd.org>, David Holland wrote: >On Fri, May 31, 2013 at 12:26:53PM +0300, Jukka Ruohonen wrote: > > > Module Name: xsrc > > > Committed By: mrg > > > Date: Fri May 31 06:46:39 UTC 2013 > > > > > > Modified Files: > > > xsrc/external/mi

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-06-01 Thread David Holland
On Fri, May 31, 2013 at 12:26:53PM +0300, Jukka Ruohonen wrote: > > Module Name: xsrc > > Committed By: mrg > > Date: Fri May 31 06:46:39 UTC 2013 > > > > Modified Files: > >xsrc/external/mit/xauth/dist: process.c > > > > Log Message: > > apply __UNCONST() if

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-05-31 Thread Christos Zoulas
In article <20130531092653.GA24785@marx.bitnet>, Jukka Ruohonen wrote: >On Fri, May 31, 2013 at 06:46:39AM +, matthew green wrote: >> Module Name: xsrc >> Committed By:mrg >> Date:Fri May 31 06:46:39 UTC 2013 >> >> Modified Files: >> xsrc/external/mit/xauth/dist:

Re: CVS commit: xsrc/external/mit/xauth/dist

2013-05-31 Thread Jukka Ruohonen
On Fri, May 31, 2013 at 06:46:39AM +, matthew green wrote: > Module Name: xsrc > Committed By: mrg > Date: Fri May 31 06:46:39 UTC 2013 > > Modified Files: > xsrc/external/mit/xauth/dist: process.c > > Log Message: > apply __UNCONST() if it exists. Reminds me: should/can __UNC