]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
hwclock: don't ifdef printf arguments
authorRuediger Meier <ruediger.meier@ga-group.nl>
Wed, 28 Jun 2017 16:37:44 +0000 (18:37 +0200)
committerRuediger Meier <ruediger.meier@ga-group.nl>
Thu, 29 Jun 2017 12:04:36 +0000 (14:04 +0200)
This may fails if printf() is macro, introduced in cc7cb070.

clang compiler warnings:

    CC       sys-utils/hwclock.o
  ../sys-utils/hwclock.c:1228:2: warning: embedding a directive within macro arguments has undefined behavior [-Wembedded-directive]
  #ifdef __linux__
   ^
  ../sys-utils/hwclock.c:1230:2: warning: embedding a directive within macro arguments has undefined behavior [-Wembedded-directive]
  #endif
   ^
  2 warnings generated.

CC: J William Piggott <elseifthen@gmx.com>
Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>
sys-utils/hwclock.c

index 2f2c03a6f89aa28fb7d9f3af26f961ae981514e8..a0a48dd130c1753aa310b92d9368e4454588fdb8 100644 (file)
@@ -1225,10 +1225,11 @@ usage(const struct hwclock_control *ctl)
        fputs(USAGE_OPTIONS, out);
        fputs(_(" -u, --utc            inform hwclock the RTC timescale is UTC\n"), out);
        fputs(_(" -l, --localtime      inform hwclock the RTC timescale is Local\n"), out);
-       fprintf(out, _(
 #ifdef __linux__
-               " -f, --rtc <file>     use an alternate file to %1$s\n"
+       printf(_(
+               " -f, --rtc <file>     use an alternate file to %1$s\n"), _PATH_RTC_DEV);
 #endif
+       printf(_(
                "     --directisa      use the ISA bus instead of %1$s access\n"), _PATH_RTC_DEV);
        fputs(_("     --date <time>    date/time input for --set and --predict\n"), out);
 #if defined(__linux__) && defined(__alpha__)