Re: Add format attribute for buffer_printf functions

2019-04-11 Thread Fabian Groffen
On 11-04-2019 19:43:46 -0700, Kevin J. McCarthy wrote: [snip] > > The git codebase also tries to > >be extremely portable, but it makes a fair amount of use of > >__attribute__ including this one. For other compilers it does: > > > >#ifndef __GNUC__ > >#ifndef __attribute__ > >#define __attribute

Re: Add format attribute for buffer_printf functions

2019-04-11 Thread Kevin J. McCarthy
On Thu, Apr 11, 2019 at 10:08:56PM -0400, Aaron Schrab wrote: There are already a few uses of __attribute__ in the mutt code; mostly in `intl/` but also one in `monitor.c`. Well, the intl isn't really "mutt code", and the __attribute__ is only for a _LIBC defined function free_mem() in there.

Re: Add format attribute for buffer_printf functions

2019-04-11 Thread Aaron Schrab
At 07:17 -0700 11 Apr 2019, "Kevin J. McCarthy" wrote: On Wed, Apr 10, 2019 at 07:44:45PM -0400, aaron+m...@schrab.com wrote: Let compilers know that `mutt_buffer_printf()` and `mutt_buffer_add_printf()` expect the second argument to be a `printf`-style format string with the arguments for the