This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: time(time_t*) problem


On Thu, Jul 25, 2002 at 09:01:42AM -0700, David MacMahon wrote:
>On Wed, Jul 24, 2002 at 01:33:34AM -0400, Christopher Faylor wrote:
>>On Tue, Jul 23, 2002 at 10:34:02PM -0700, David MacMahon wrote:
>>>(sorry, I didn't do a Changelog >entry).
>> 
>> Why?
>
>I was a little bit rushed (OK, lazy) at the time.  Below is a better
>submission.  Do you prefer these to be inline (as below) or as
>attachments (or both)?

Inline is fine as long as there are no whitespace problems.  This was
perfect.  The ChangeLog was perfect, too.

Thanks for the patch and for the rationalization of why it was
necessary.  Until I read your message, I thought that cygwin was working
correctly.  Now I can see that it obviously wasn't.

I also love patches that remove code and make cygwin faster, too.  :-)

The patch has been committed.

cgf

>2002-07-24  David MacMahon  <davidm@smartsc.com>
>
>	* times.cc (to_time_t): Always round time_t down to nearest second.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]