On Tue, Jul 19, 2011 at 13:22, Kelly Anderson
<ke...@silka.with-linux.com> wrote:
> The following commit introduces a bogus error message on boot since
> localtime may be a negative number, i.e. GMT-7.

> The following patch just uses errno directly rather than using it indirectly
> through hwclock_apply_localtime_delta():

Ah, tm->tm_gmtoff can be negative.

> +       errno = 0;
>          min = hwclock_apply_localtime_delta();

We usually prefer to return the errno instead of mangling it at the
caller's side, and have the values that can be negative passed as
arguments. Mind switching that? Or let me know.

Thanks,
Kay
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to