Looks Good.
On Thu, Mar 31, 2011 at 4:31 PM, Ben Pfaff <[email protected]> wrote: > PRINTF_FORMAT is more portable than raw __attribute__. We use it > elsewhere, I don't know why it wasn't used here. > --- > lib/vlog.h | 7 ++++--- > 1 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/lib/vlog.h b/lib/vlog.h > index 7411f76..00d324f 100644 > --- a/lib/vlog.h > +++ b/lib/vlog.h > @@ -21,6 +21,7 @@ > #include <stdarg.h> > #include <stdbool.h> > #include <time.h> > +#include "compiler.h" > #include "util.h" > > #ifdef __cplusplus > @@ -152,13 +153,13 @@ int vlog_reopen_log_file(void); > void vlog_init(void); > void vlog_exit(void); > void vlog(const struct vlog_module *, enum vlog_level, const char *format, > ...) > - __attribute__((format(printf, 3, 4))); > + PRINTF_FORMAT (3, 4); > void vlog_valist(const struct vlog_module *, enum vlog_level, > const char *, va_list) > - __attribute__((format(printf, 3, 0))); > + PRINTF_FORMAT (3, 0); > void vlog_rate_limit(const struct vlog_module *, enum vlog_level, > struct vlog_rate_limit *, const char *, ...) > - __attribute__((format(printf, 4, 5))); > + PRINTF_FORMAT (4, 5); > > /* Creates and initializes a global instance of a module named MODULE, and > * defines a static variable named THIS_MODULE that points to it, for use with > -- > 1.7.1 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
