Re: [RFC] Prepend timestamp in msgbuf


Hi,

On Mon, Oct 17, 2011 at 3:38 PM, Garrett Cooper <yanegomi@gmail.com> wrote:
> On Mon, Oct 17, 2011 at 12:27 PM, Arnaud Lacombe <lacombar@gmail.com> wro=
te:
>> Hi,
>>
>> On Mon, Oct 17, 2011 at 2:01 PM, Alexander Best <arundel@freebsd.org> wr=
ote:
>>> On Fri Oct 14 11, Arnaud Lacombe wrote:
>>> [...]
>>>
>>> cc -c -O -pipe -march=3Dcore2 -std=3Dc99 -g -Wall -Wredundant-decls -Wn=
ested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Win=
line -Wcast-qual -Wundef -Wno-pointer-sign -Wmissing-include-dirs -nostdinc=
 =A0-I. -I/usr/git-freebsd-head/sys -I/usr/git-freebsd-head/sys/contrib/alt=
q -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common =
-finline-limit=3D8000 --param inline-unit-growth=3D100 --param large-functi=
on-growth=3D1000 =A0-fno-omit-frame-pointer -mno-sse -mcmodel=3Dkernel -mno=
-red-zone -mno-mmx -msoft-float -fno-asynchronous-unwind-tables -ffreestand=
ing -fformat-extensions -fdiagnostics-show-option -fstack-protector -Werror=
 =A0/usr/git-freebsd-head/sys/kern/subr_msgbuf.c
>>> cc1: warnings being treated as errors
>>> /usr/git-freebsd-head/sys/kern/subr_msgbuf.c: In function 'msgbuf_do_ad=
dchar':
>>> /usr/git-freebsd-head/sys/kern/subr_msgbuf.c:171: warning: format '%d' =
expects type 'int', but argument 4 has type 'time_t' [-Wformat]
>>> *** Error code 1
>>>
>>> Stop in /usr/obj/usr/git-freebsd-head/sys/ARUNDEL.
>>> *** Error code 1
>>>
>>> Stop in /usr/git-freebsd-head.
>>> *** Error code 1
>>>
>>> Stop in /usr/git-freebsd-head.
>>>
>> FreeBSD has no time_t PRI... macros in any <machine/_inttypes.h>,
>> eventually cast it to `long'.
>>
>> Btw, I appreciate the very clear message of yours, no "Hi", no
>> signature, no idea what "ARUNDEL" is, and especially no details on
>> which architecture you are attempting to build, which should be, I
>> assume, LP64 ;-)
>
> =A0 =A0time_t maps to int32_t on i386 and int64_t on amd64 (at least), so
> you should be able to use "%zd" in the format string as the type is
> variable width depending on the architecture.
>
make sense.

Thanks,
 - Arnaud
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"