[hackers] [sbase] Nuke some fprintf() calls and replace with weprintf() || sin

From: <git_AT_suckless.org>
Date: Mon, 17 Nov 2014 17:33:28 +0100

commit 4dc2b783b85f8d90eb45684efc55696e9b070ea8
Author: sin <sin_AT_2f30.org>
Date: Mon Nov 17 16:27:12 2014 +0000

    Nuke some fprintf() calls and replace with weprintf()

diff --git a/chgrp.c b/chgrp.c
index 51e99b4..0a3c38d 100644
--- a/chgrp.c
+++ b/chgrp.c
_AT_@ -25,7 +25,7 @@ static void
 chgrp(const char *path)
 {
         if (chown(path, st.st_uid, gid) == -1) {
- fprintf(stderr, "chgrp: '%s': %s\n", path, strerror(errno));
+ weprintf("chown %s:", path);
                 failures++;
         }
         if (rflag)
_AT_@ -58,8 +58,7 @@ main(int argc, char *argv[])
 
         while (*++argv) {
                 if (stat(*argv, &st) == -1) {
- fprintf(stderr, "stat %s: %s\n", *argv,
- strerror(errno));
+ weprintf("stat %s:", *argv);
                         failures++;
                         continue;
                 }
diff --git a/renice.c b/renice.c
index feba155..06cbc46 100644
--- a/renice.c
+++ b/renice.c
_AT_@ -1,5 +1,4 @@
 /* See LICENSE file for copyright and license details. */
-#include <errno.h>
 #include <limits.h>
 #include <pwd.h>
 #include <stdio.h>
_AT_@ -55,7 +54,6 @@ main(int argc, char *argv[])
 
                         errno = 0;
                         do pwd = getpwnam(argv[i]); while (errno == EINTR);
-
                         if (pwd)
                                 who = pwd->pw_uid;
                         else if (errno != 0) {
_AT_@ -83,11 +81,11 @@ strtop(const char *s)
         errno = 0;
         n = strtol(s, &end, 10);
         if (*end != '\0') {
- fprintf(stderr, "%s: not an integer\n", s);
+ weprintf("%s: not an integer\n", s);
                 return -1;
         }
         if (errno != 0 || n <= 0 || n > INT_MAX) {
- fprintf(stderr, "%s: invalid value\n", s);
+ weprintf("%s: invalid value\n", s);
                 return -1;
         }
 
_AT_@ -100,15 +98,13 @@ renice(int which, int who, long adj)
         errno = 0;
         adj += getpriority(which, who);
         if (errno != 0) {
- fprintf(stderr, "can't get %d nice level: %s\n",
- who, strerror(errno));
+ weprintf("getpriority %d:", who);
                 return 0;
         }
 
         adj = MAX(PRIO_MIN, MIN(adj, PRIO_MAX));
         if (setpriority(which, who, (int)adj) == -1) {
- fprintf(stderr, "can't set %d nice level: %s\n",
- who, strerror(errno));
+ weprintf("setpriority %d:", who);
                 return 0;
         }
 
Received on Mon Nov 17 2014 - 17:33:28 CET

This archive was generated by hypermail 2.3.0 : Mon Nov 17 2014 - 17:36:16 CET