Originally sent to the wrong email address - sorry.<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Jonathan Leffler</b> <span dir="ltr"><<a href="mailto:jonathan.leffler@gmail.com">jonathan.leffler@gmail.com</a>></span><br>
Date: Mon, Dec 8, 2008 at 12:14 PM<br>Subject: Re: zdump loop problem<br>To: Robbin Kawabata <<a href="mailto:Robbin.Kawabata@sun.com">Robbin.Kawabata@sun.com</a>><br>Cc: <a href="mailto:tz@lecserver.nci.nih.gov">tz@lecserver.nci.nih.gov</a>, <a href="mailto:a.radke@arcor.de">a.radke@arcor.de</a><br>
<br><br><br><br><div class="gmail_quote"><div class="Ih2E3d">On Mon, Dec 8, 2008 at 11:50 AM, Robbin Kawabata <span dir="ltr"><<a href="mailto:Robbin.Kawabata@sun.com" target="_blank">Robbin.Kawabata@sun.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I ran ado's fix against all the timezones, and got good results.<br>
<br>
This is the diff:<br>
<br>
*** zdump_loop.c Tue Dec 2 11:17:01 2008<br>
--- zdump_fix_ado.c Tue Dec 2 12:49:05 2008<br>
***************<br>
*** 312,322 ****<br>
<div> (void) strncpy(buf, abbr(&tm), sizeof (buf) - 1);<br>
}<br>
for (;;) {<br>
! if (t >= cuthitime)<br>
break;<br>
newt = t + SECSPERHOUR * 12;<br>
- if (newt >= cuthitime)<br>
- break;<br>
</div><div> if (newt <= t)<br>
break;<br>
newtmp = localtime(&newt);<br>
</div>--- 312,320 ----<br>
<div> (void) strncpy(buf, abbr(&tm), sizeof (buf) - 1);<br>
}<br>
for (;;) {<br>
</div>! if (t >= cuthitime || t >= cuthitime - SECSPERHOUR * 12)<br>
<div> break;</div></blockquote></div><div><br>Isn't that code equivalent to just:<br><br>if (t >= cuthitime - SECSPERHOUR * 12)<br><br>I've not followed all the details of the thread, but AFAIUI, this code is protecting against overflow on positive quantities (rather than underflow on negative ones), and the double test is redundant unless cuthitime is within 12 * SECPERHOUR of the smallest (most negative) integer.<br>
<br> <br></div><div class="Ih2E3d"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div> newt = t + SECSPERHOUR * 12;<br>
if (newt <= t)<br>
break;<br>
newtmp = localtime(&newt);<br>
<br>
<br>
</div>However, isn't the check for "if (newt <= t) break" no longer needed?<br>
</blockquote></div><div><br>Probably...<br> <br></div><div class="Ih2E3d"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">----- Begin Forwarded Message -----<br>
<br>
Date: Mon, 01 Dec 2008 16:21:00 -0500<br>
<div>From: "Olson, Arthur David (NIH/NCI) [E]" <<a href="mailto:olsona@dc37a.nci.nih.gov" target="_blank">olsona@dc37a.nci.nih.gov</a>><br>
</div>Subject: Re: zdump loop problem<br>
<div>To: <a href="mailto:tz@lecserver.nci.nih.gov" target="_blank">tz@lecserver.nci.nih.gov</a><br>
Cc: Andreas Radke <<a href="mailto:a.radke@arcor.de" target="_blank">a.radke@arcor.de</a>><br>
<br>
</div><div>So...in the real world, does changing the zic.c lines reading...<br>
</div><div> if (t >= cuthitime)<br>
break;<br>
newt = t + SECSPERHOUR * 12;<br>
if (newt >= cuthitime)<br>
break;<br>
</div>...to read...<br>
<div> if (t >= cuthitime || t >= cuthitime -<br>
SECSPERHOUR * 12)<br>
break;<br>
newt = t + SECSPERHOUR * 12;<br>
</div><div>...result in working code? I'd particularly appreciate feedback from<br>
Andreas Radke or other ArchLinux users.<br>
<br>
</div>----- End Forwarded Message -----<br>
</blockquote></div></div><font color="#888888"><br><br></font></div><br><br clear="all"><br>-- <br>Jonathan Leffler <<a href="mailto:jonathan.leffler@gmail.com">jonathan.leffler@gmail.com</a>> #include <disclaimer.h><br>
Guardian of DBD::Informix - v2008.0513 - <a href="http://dbi.perl.org">http://dbi.perl.org</a><br>"Blessed are we who can laugh at ourselves, for we shall never cease to be amused."<br>