This patch makes code more readable and removes empty fprintf() formats.

Index: io.c
===================================================================
RCS file: /cvs/src/bin/ed/io.c,v
retrieving revision 1.19
diff -u -p -r1.19 io.c
--- io.c        22 Mar 2016 17:58:28 -0000      1.19
+++ io.c        25 Nov 2016 21:49:56 -0000
@@ -63,7 +63,8 @@ read_file(char *fn, int n)
                seterrmsg("cannot close input file");
                return ERR;
        }
-       fprintf(stderr, !scripted ? "%d\n" : "", size);
+       if (!scripted)
+               fprintf(stderr, "%d\n", size);
        return current_addr - n;
 }

@@ -164,7 +165,8 @@ write_file(char *fn, char *mode, int n,
                seterrmsg("cannot close output file");
                return ERR;
        }
-       fprintf(stderr, !scripted ? "%d\n" : "", size);
+       if (!scripted)
+               fprintf(stderr, "%d\n", size);
        return n ? m - n + 1 : 0;
 }

Index: main.c
===================================================================
RCS file: /cvs/src/bin/ed/main.c,v
retrieving revision 1.58
diff -u -p -r1.58 main.c
--- main.c      16 Aug 2016 20:04:46 -0000      1.58
+++ main.c      25 Nov 2016 21:49:57 -0000
@@ -218,9 +218,10 @@ top:
                                fputs("?\n", stderr);
                                seterrmsg("warning: file modified");
                                if (!interactive) {
-                                       fprintf(stderr, garrulous ?
-                                           "script, line %d: %s\n" :
-                                           "", lineno, errmsg);
+                                       if (garrulous)
+                                               fprintf(stderr,
+                                                   "script, line %d: %s\n",
+                                                   lineno, errmsg);
                                        quit(2);
                                }
                                clearerr(stdin);
@@ -252,28 +253,31 @@ top:
                        fputs("?\n", stderr);           /* give warning */
                        seterrmsg("warning: file modified");
                        if (!interactive) {
-                               fprintf(stderr, garrulous ?
-                                   "script, line %d: %s\n" :
-                                   "", lineno, errmsg);
+                               if (garrulous)
+                                       fprintf(stderr,
+                                           "script, line %d: %s\n",
+                                           lineno, errmsg);
                                quit(2);
                        }
                        break;
                case FATAL:
-                       if (!interactive)
-                               fprintf(stderr, garrulous ?
-                                   "script, line %d: %s\n" : "",
-                                   lineno, errmsg);
-                       else
-                               fprintf(stderr, garrulous ? "%s\n" : "",
-                                   errmsg);
+                       if (garrulous) {
+                               if (!interactive)
+                                       fprintf(stderr,
+                                           "script, line %d: %s\n",
+                                           lineno, errmsg);
+                               else
+                                       fprintf(stderr, "%s\n", errmsg);
+                       }
                        quit(3);
                        break;
                default:
                        fputs("?\n", stderr);
                        if (!interactive) {
-                               fprintf(stderr, garrulous ?
-                                   "script, line %d: %s\n" : "",
-                                   lineno, errmsg);
+                               if (garrulous)
+                                       fprintf(stderr,
+                                           "script, line %d: %s\n",
+                                           lineno, errmsg);
                                quit(2);
                        }
                        break;

Reply via email to