script: -f means "flush", not "fsync"

function                                             old     new   delta
packed_usage                                       31765   31768      +3
script_main                                         1102    1082     -20

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2017-08-07 02:21:34 +02:00
parent 269b36a49a
commit 4dea1edd08

View File

@ -27,7 +27,8 @@
//usage: "\n" //usage: "\n"
//usage: "\n -a Append output" //usage: "\n -a Append output"
//usage: "\n -c PROG Run PROG, not shell" //usage: "\n -c PROG Run PROG, not shell"
//usage: "\n -f Flush output after each write" /* Accepted but has no effect (we never buffer output) */
/*//usage: "\n -f Flush output after each write"*/
//usage: "\n -q Quiet" //usage: "\n -q Quiet"
//usage: "\n -t[FILE] Send timing to stderr or FILE" //usage: "\n -t[FILE] Send timing to stderr or FILE"
@ -178,9 +179,10 @@ int script_main(int argc UNUSED_PARAM, char **argv)
} }
full_write(STDOUT_FILENO, buf, count); full_write(STDOUT_FILENO, buf, count);
full_write(outfd, buf, count); full_write(outfd, buf, count);
if (opt & OPT_f) { // If we'd be using (buffered) FILE i/o, we'd need this:
fsync(outfd); //if (opt & OPT_f) {
} // fflush(outfd);
//}
} }
} }
if (pfd[1].revents) { if (pfd[1].revents) {