From: Jim Meyering <jim@meyering.net>
Date: Sat, 23 Jan 1999 16:48:36 +0000 (+0000)
Subject: (pfstatus): Use __format__ and __printf__.
X-Git-Tag: TEXTUTILS-1_22h~128
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9550ca6b715c0bb20ee646adbbec3cf0a7053828;p=platform%2Fupstream%2Fcoreutils.git

(pfstatus): Use __format__ and __printf__.
Remove `if __GNUC__ >= 2' guard.
---

diff --git a/src/shred.c b/src/shred.c
index d91620e..fb0aec2 100644
--- a/src/shred.c
+++ b/src/shred.c
@@ -1,7 +1,8 @@
 /* TODO:
    x use getopt_long
    x use error, not pferror
-   - bracket strings with _(...) for gettext
+   - don't use pfstatus -- or maybe leave it in and see who complains
+   x bracket strings with _(...) for gettext
    - use consistent non-capitalizatin in error messages
  */
 
@@ -506,10 +507,11 @@ irand_mod (struct irand_state *r, word32 n)
 
 /*
  * Like perror() but fancier.  (And fmt is not allowed to be NULL)
+ * This apparent use of features specific to GNU C is actually portable;
+ * see the definitions in error.h.
  */
-#if __GNUC__ >= 2
-static void pfstatus (char const *,...) __attribute__ ((format (printf, 1, 2)));
-#endif
+static void pfstatus (char const *,...)
+     __attribute__ ((__format__ (__printf__, 1, 2)));
 
 /*
  * Maintain a status line on stdout.  This is done by using CR and
@@ -1194,8 +1196,7 @@ wipename (char *oldname, struct Options const *flags)
 		sync ();	/* Force directory out */
 	      if (origname)
 		{
-		  pfstatus (_("%s: renamed to `%s'"),
-			    origname, newname);
+		  pfstatus (_("%s: renamed to `%s'"), origname, newname);
 		  if (flags->verbose > 1)
 		    flushstatus ();
 		}