Some cleanups and tweaks for wc.1

Hi tech@, Some cleanups and tweaks for wc.1: - Removed unnecessary string.h include - Changed 'format_and_print' argument type to int64_t and casting inside the function - Declaring 'print_counts', 'format_and_print', and 'cnt' as static - Remove unnecessary cast for NULL, and (void) casts

Re: static in ze kernel

On Friday 29 April 2016 14:09:23, Mark Kettenis wrote: > I think we should simply continue the current practice of not using > static in the kernel. I mean, what is the benefit of abandoning > that practice if you disable the optimizations that compiler would > make anyway? You get a clear

vmctl: move some validations in start_vm()

Hi everyone, The following patch moves all the validations of the start command in start_vm() as suggested in the comment. Index: main.c === RCS file: /cvs/src/usr.sbin/vmctl/main.c,v retrieving revision 1.16 diff -u -p -r1.16

Re: httpd: fix/style: unbalanced va_start and va_end macros

On Wed, Apr 27, 2016 at 02:43:27PM +0200, Hiltjo Posthuma wrote: > Hi, > > The following patch for httpd fixes unbalanced va_start() and va_end() macros. > This is in style with the rest of httpd. Also POSIX says: > > "Each invocation of the va_start() and va_copy() macros shall be matched by a

Re: httpd: patch for portability asprintf use

On Fri, May 06, 2016 at 06:48:38PM +0200, Reyk Floeter wrote: > > > On 06.05.2016, at 18:36, Theo de Raadt wrote: > > > >> If OpenBSD's behavior of asprintf is non-standard and everyone else is > >> doing it differently, we would probably have to apply the patch. But

ftp/www.openbsd.org will be down for an upgrade today.

There will be an extended downtime of the main ftp and www sites for an upgrade today starting in approximately one hour's time from now. The mirror sites should be unaffected - so use a mirror if you discover the main site is unavailable today. Thanks -Bob

Re: More dbm(3) removal from man pages

On Sun, May 08, 2016 at 05:42:33PM +0200, Christian Weisgerber wrote: > jmc@ pointed out these places where dbm(3) is still mentioned. > > For pwd_mkdb.8, I suggest to remove the whole paragraph. It's been > there since the very first version of the man page in the CSRG tree, > 25 years ago. At

More dbm(3) removal from man pages

jmc@ pointed out these places where dbm(3) is still mentioned. For pwd_mkdb.8, I suggest to remove the whole paragraph. It's been there since the very first version of the man page in the CSRG tree, 25 years ago. At some point it was renamed from Compatibility to Standards, although it doesn't

Re: libc/sparc64 ABI cleanup

> Date: Sat, 7 May 2016 22:58:23 -0700 > From: Philip Guenther > > Currently, the soft-float bits in libc on sparc64 export not just the > sparc ABI symbols _Q_* and _Qp_*, but also the internal implementation > bits, __fpu_* > > AFAICT, the __fpu_* names are not required

Re: xclock patch

> This patch forces xclock to read XErrorDB before pledge(). Further > calls to any of the X error handler will used the in-memory copy (see > libX11/src/ErrDes.c:147). So basically, it primes the in-memory cache. Then the syscall codepaths are avoided later on. > I'm not yet 100% sure if

Re: xclock patch

On Sat, May 07, 2016 at 02:01:34PM -0600, Theo de Raadt wrote: > > > So I don't understand why pledge is being added here. > > > > Because you suggested it back in november... > > Of course I suggested it: I believe I said it would be nice if > the simplest of X programs were investigated. > >

[patch] remove custom getline from vi

Hello tech@, I'm currently working on removing the binc family tree (including {GET,ADD}_SPACE_{RET,GOTO}) from vi. During this quest I found this gem which I reckon can be done better with the libc native function. This has the benefit of extra checks from getline and it echos all system

libc/sparc64 ABI cleanup

Currently, the soft-float bits in libc on sparc64 export not just the sparc ABI symbols _Q_* and _Qp_*, but also the internal implementation bits, __fpu_* AFAICT, the __fpu_* names are not required by our tool chain; in contrast, calls to the _Q* names are generated by gcc. Diff below cleans