Re: CVS commit: [agc-netpgp-standalone] src/crypto/external/bsd/netpgp/dist/src

2012-10-25 Thread Christos Zoulas
In article 20121025040317.3b16417...@cvs.netbsd.org, Alistair G. Crooks source-changes-d@NetBSD.org wrote: -=-=-=-=-=- Module Name: src Committed By: agc Date: Thu Oct 25 04:03:16 UTC 2012 Modified Files: src/crypto/external/bsd/netpgp/dist/src/libverify

Re: CVS commit: src/lib/libc/time

2012-10-25 Thread Alan Barrett
On Thu, 25 Oct 2012, Martin Husemann wrote: Modified Files: src/lib/libc/time: zic.c Log Message: Add a few casts to avoid (IMHO bogus) gcc warnings breaking the vax build. - if (dayoff min_time / SECSPERDAY) { + if (dayoff (long)(min_time / SECSPERDAY)) { time_t and

Re: CVS commit: [agc-netpgp-standalone] src/crypto/external/bsd/netpgp/dist/src

2012-10-25 Thread Alistair Crooks
On Thu, Oct 25, 2012 at 12:40:28PM +, Christos Zoulas wrote: In article 20121025040317.3b16417...@cvs.netbsd.org, Alistair G. Crooks source-changes-d@NetBSD.org wrote: -=-=-=-=-=- Module Name: src Committed By:agc Date:Thu Oct 25 04:03:16 UTC 2012 Modified

Re: CVS commit: src/lib/libc/time

2012-10-25 Thread Martin Husemann
On Thu, Oct 25, 2012 at 04:18:54PM +0200, Alan Barrett wrote: Does adding a /*CONSTCOND*/ comment work? If so, I'd prefer that to a cast. If you do need a cast, then I think you should cast dayoff to zic_t. I backed it out and set NOGCCERROR instead - couldn't figure any way with casts or

Re: CVS commit: [agc-netpgp-standalone] src/crypto/external/bsd/netpgp/dist/src

2012-10-25 Thread Christos Zoulas
In article 20121025143517.gb19...@nef.pbox.org, Alistair Crooks a...@pkgsrc.org wrote: On Thu, Oct 25, 2012 at 12:40:28PM +, Christos Zoulas wrote: In article 20121025040317.3b16417...@cvs.netbsd.org, Alistair G. Crooks source-changes-d@NetBSD.org wrote: -=-=-=-=-=- Module Name:

Re: CVS commit: [agc-netpgp-standalone] src/crypto/external/bsd/netpgp/dist/src

2012-10-25 Thread Alistair Crooks
On Thu, Oct 25, 2012 at 04:09:12PM +, Christos Zoulas wrote: But I'm not sure it should - the memory pointer or filename argument is overloaded when it is passed in; the size parameter being negative is used to signify that it's a filename, so ssize_t is the correct type. Perhaps there