Use line buffering for stdout. fixes #179

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 6befa12aa478afdf3c2bc8302e5fe4de22ee8acb..bb12f718e73aa0d1f25950050f11a69d685b540a 100644 (file)
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -17,6 +17,7 @@ UNRELEASED
 - Macos build fix (#174, thanks @tycho)
 - Fix xmt stats in Netdata output (#172, thanks @vlvkobal)
 - Only increase num_alive if response is not a duplicate (#151, thanks @brownowski)
+- Use line buffering for stdout (#179, thanks @bg6cq)
 
 fping 4.2 (2019-02-19)
 ======================

diff --git a/src/fping.c b/src/fping.c
index a335d977f5076625faa9d8bcf77bc9f4f9d54b58..214a3ed959981005d3037bf57dea68f1b1421800 100644 (file)
--- a/src/fping.c
+++ b/src/fping.c
@@ -1031,6 +1031,7 @@ int main(int argc, char** argv)
 
     signal(SIGINT, finish);
     signal(SIGQUIT, sigstatus);
+    setlinebuf(stdout);
 
     gettimeofday(&start_time, NULL);
     current_time = start_time;
@@ -1430,8 +1431,6 @@ void print_per_system_stats(void)
     HOST_ENTRY* h;
     int resp;
 
-    fflush(stdout);
-
     if (verbose_flag || per_recv_flag)
         fprintf(stderr, "\n");
 
@@ -1511,8 +1510,6 @@ void print_netdata(void)
     int i, avg;
     HOST_ENTRY* h;
 
-    fflush(stdout);
-
     for (i = 0; i < num_hosts; i++) {
         h = table[i];
 
@@ -1594,8 +1591,6 @@ void print_per_system_splits(void)
     HOST_ENTRY* h;
     struct tm* curr_tm;
 
-    fflush(stdout);
-
     if (verbose_flag || per_recv_flag)
         fprintf(stderr, "\n");
 
@@ -1659,7 +1654,6 @@ void print_per_system_splits(void)
 
 void print_global_stats(void)
 {
-    fflush(stdout);
     fprintf(stderr, "\n");
     fprintf(stderr, " %7d targets\n", num_hosts);
     fprintf(stderr, " %7d alive\n", num_alive);
@@ -2299,7 +2293,6 @@ int wait_for_reply(long wait_time)
         remove_job(h);
     }
 
-    fflush(stdout);
     return num_jobs;
 }