date: maybe_set_utc only once

It's not clear at all why we do it twice. git archaeology did not help.

function                                             old     new   delta
date_main                                            836     839      +3
maybe_set_utc                                         16       -     -16

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/coreutils/date.c b/coreutils/date.c
index 5b15ce778481b1938262540f56502f3a4b1feef2..87dc3bbd0cf6c5ef07164ed449bd224d3f2ac153 100644 (file)
--- a/coreutils/date.c
+++ b/coreutils/date.c
@@ -36,8 +36,8 @@
 //config:# defaults to "no": stat's nanosecond field is a bit non-portable
 //config:config FEATURE_DATE_NANO
 //config:      bool "Support %[num]N nanosecond format specifier"
-//config:      default n
-//config:      depends on DATE  # syscall(__NR_clock_gettime)
+//config:      default n  # syscall(__NR_clock_gettime)
+//config:      depends on DATE
 //config:      select PLATFORM_LINUX
 //config:      help
 //config:      Support %[num]N format specifier. Adds ~250 bytes of code.
@@ -203,6 +203,7 @@ int date_main(int argc UNUSED_PARAM, char **argv)
                        IF_FEATURE_DATE_ISOFMT(, &isofmt_arg, &fmt_str2dt)
        );
        argv += optind;
+
        maybe_set_utc(opt);
 
        if (ENABLE_FEATURE_DATE_ISOFMT && (opt & OPT_TIMESPEC)) {
@@ -301,8 +302,6 @@ int date_main(int argc UNUSED_PARAM, char **argv)
                        tm_time.tm_isdst = -1;
                ts.tv_sec = validate_tm_time(date_str, &tm_time);
 
-               maybe_set_utc(opt);
-
                /* if setting time, set it */
                if ((opt & OPT_SET) && stime(&ts.tv_sec) < 0) {
                        bb_perror_msg("can't set date");