Thus spake Juli Mallett <[EMAIL PROTECTED]>:
> * De: Craig Rodrigues <[EMAIL PROTECTED]> [ Data: 2002-12-29 ]
>       [ Subjecte: Re: sparc64 tinderbox failure ]
> > I'm not sure if your patch will solve the problem.
> > The offending code is here:
> >     240         if (lflag) { 
> >     241                 char buf[32];
> >     242                 snprintf(buf, sizeof(buf), "%ld-blocks", blocksize);
> >     243                 printf("%-13s %*s %*s\n",
> >     244                     "Device:",
> >     245                     hlen, buf,
> >     246                     hlen, "Used:");
> >     247         }
> > 
> > 
> > Doesn't the printf() statement in question have the wrong number of
> > arguments?
> 
> No, it's using variable field-length specifiers.

Right.  The complaint is that hlen is 64 bits and the printf()
expects the field length specifier to be an int.  The same goes
for getbsize(&hlen, ...), so I'm not sure why the compiler didn't
complain about a type mismatch.  I guess it just coerced the
pointer to an int *.

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to