Not sure about removing the comment from the bug report as someone might need 
it as a clue, surely it would have been better to just addd your comment and 
leave mine intact.

Jani, are you going to post a real patch? I'm not a C programmer (life's too 
short etc) - I'd probably get it wrong again.

John.

-----Original Message-----
From: Jani Taskinen [mailto:[EMAIL PROTECTED]
Sent: 24 January 2005 04:44
To: Carter, John
Cc: internals@lists.php.net
Subject: Re: [PHP-DEV] Patch for bug 29334 Daylight Savings Win32



     This "fix" is not very ideal considering this note about tm_isdst property:

       "Positive value if daylight saving time is in effect; 0 if daylight
       saving time is not in effect; negative value if status of daylight
       saving time is unknown. If the TZ environment variable is set, the C
       run-time library assumes rules appropriate to the United States for
       implementing the calculation of daylight-saving time (DST)."

     In case the status of dst is unknown, you'd be removing 1 or more
     hours from the time..

     --Jani

     p.s. I removed this patch from the bug report.


On Fri, 14 Jan 2005, Carter, John wrote:

> Hi,
>
> This bug's been open a while (since July). I placed a small patch in the 
> comments in October (repeated below); can someone review and commit please?
>
> John.
>
> patch to win32 sendmail.c:
>
> --- old.c     2004-09-08 21:33:17.640625000 +0100
> +++ new.c     2004-09-08 21:39:06.890625000 +0100
> @@ -684,6 +684,7 @@
>       }
>       zoneh /= (60 * 60);
>       zonem = (abs(_timezone) / 60) - (zoneh * 60);
> +     zoneh += tm->tm_isdst;
>
>
>       if(!xheaders || !strstr(headers_lc, "date:")){
>
>
> John Carter
> Developer, Integro project
> Royal London
> (44) 1625 605735
> [EMAIL PROTECTED]
>
>
>
> The contents of this e-mail are confidential and for the addressee only.
> If you are not the addressee you should not print, copy or distribute
> this e-mail or use it or its contents or copy or publish its contents
> to any other person.  If you have received this e-mail in error please
> notify us and delete it from your system.
>
> The Royal London Mutual Insurance Society Limited for life and pension
> products. Registered in England No. 99064. Registered Office:
> 55 Gracechurch Street, London, EC3V 0RL. Royal London Asset Management Limited
> for investment management services. Registered in England No. 2244297.
> Registered Office: 55 Gracechurch Street, London, EC3V 0UF.
> Royal London Savings Limited for Individual Savings Accounts.
> Registered in England No. 3642633. Registered Office: 55 Gracechurch Street,
> London, EC3V 0RL. Royal London Unit Trust Managers Limited
> for unit trusts. Registered in England No. 2372439.
> Registered Office: 55 Gracechurch Street, London, EC3V 0UF.
> All these companies are authorised and regulated by the Financial Services 
> Authority.
>
>

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to