Nope, you're right on that one -- just slap the return in the ifdef
for netware and commit, I guess.. ;-)
-- Jon
On Mon, Jan 07, 2002 at 05:48:06PM -0800, Brian Pane wrote:
> Jon Travis wrote:
>
> >Ok, here's a small patch which gets things working correctly for me.
> >Not sure if this patch is correct, though.
> >
>
> Thanks, I just tested this on Solaris and Linux, and it produced
> the expected results on both. The patch logic looks okay, with one
> possible exception: if NETWARE is defined, do you really want to fall
> into the "if (tm->tm_isdst)" check if the "if (daylightOnOff)" check
> fails? Or should it be:
>
> #ifdef NETWARE
> /* Need to adjust the global variable each time otherwise
> the web server would have to be restarted when daylight
> savings changes.
> */
> if (daylightOnOff) {
> return server_gmt_offset + daylightOffset;
> }
> #else
> if(tm->tm_isdst) {
> return server_gmt_offset + 3600;
> }
> #endif
> return server_gmt_offset;
> #endif
>
>
