bin/71618: [PATCH] cleanup of the usr.sbin/timed code
Dan Lukes
dan at obluda.cz
Sun Sep 12 18:40:23 PDT 2004
The following reply was made to PR bin/71618; it has been noted by GNATS.
From: Dan Lukes <dan at obluda.cz>
To: Giorgos Keramidas <keramida at freebsd.org>
Cc: bug-followup at freebsd.org
Subject: Re: bin/71618: [PATCH] cleanup of the usr.sbin/timed code
Date: Mon, 13 Sep 2004 03:37:11 +0200 (CEST)
On Sun, 12 Sep 2004, Giorgos Keramidas wrote:
>> - fprintf(fd, "%s %d %-6u (%ld,%ld) %-15s %s\n",
>> + fprintf(fd, "%s %d %-6u (%d,%d) %-15s %s\n",
>
> There's no guarantee that an int32_t value will be defined with an `int'.
> This might result in unexpected truncation of the printed value. A better
True. In the fact, I miss the existence of 'j' modifier and intmax_t
type.
> of work, can you try the following (and see if you can come up with fixes
> for the warnings this shows)?
>
> # cd /usr/src/usr.sbin/timed/
> # make NO_WERROR=1 WARNS?=3
I'm compiling with -Wall -O3.
It seems to be more than any WARNS=X
If there are some warning not corrected by attached patches, then it
mean I'm don't know how the good way to correct it.
> Thank you for your work so far :-)
I have a lot of spare time now only ... ;-)
Dan
More information about the freebsd-bugs
mailing list