Re: [Patch 1/3] tabled: make time2str reentrant

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, 03 Jan 2010 03:27:59 -0500
Jeff Garzik <jeff@xxxxxxxxxx> wrote:

> > -	sprintf(datestr, "Date: %s", time2str(timestr, time(NULL)));
> > +	sprintf(datestr, "Date: %s", time2str(timestr, 64, time(NULL)));
> 
> applied 1-3, and then added sizeof() to the above time2str calls...  The 
> hardcoded sizes I used for the strings is ugly enough...  let's not add 
> additional "naked numbers" on top of that.

I thought wrapping was worse. Sure...

-- Pete
--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Fedora Clound]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux