===================================================================
RCS file: /cvsroot/curl/curl/lib/progress.c,v
retrieving revision 1.81
diff --unified -r1.81 progress.c
--- lib/progress.c	11 Mar 2007 09:11:30 -0000	1.81
+++ lib/progress.c	17 Mar 2007 19:14:54 -0000
@@ -281,72 +281,135 @@
      (data->progress.timespent>0?data->progress.timespent:1));
 
   if(data->progress.lastshow == Curl_tvlong(now))
-    return 0; /* never update this more than once a second if the end isn't
-                 reached */
-  data->progress.lastshow = now.tv_sec;
-
-  /* Let's do the "current speed" thing, which should use the fastest
-     of the dl/ul speeds. Store the fasted speed at entry 'nowindex'. */
-  data->progress.speeder[ nowindex ] =
-    data->progress.downloaded>data->progress.uploaded?
-    data->progress.downloaded:data->progress.uploaded;
-
-  /* remember the exact time for this moment */
-  data->progress.speeder_time [ nowindex ] = now;
-
-  /* advance our speeder_c counter, which is increased every time we get
-     here and we expect it to never wrap as 2^32 is a lot of seconds! */
-  data->progress.speeder_c++;
-
-  /* figure out how many index entries of data we have stored in our speeder
-     array. With N_ENTRIES filled in, we have about N_ENTRIES-1 seconds of
-     transfer. Imagine, after one second we have filled in two entries,
-     after two seconds we've filled in three entries etc. */
-  countindex = ((data->progress.speeder_c>=CURR_TIME)?
-                CURR_TIME:data->progress.speeder_c) - 1;
-
-  /* first of all, we don't do this if there's no counted seconds yet */
-  if(countindex) {
-    long span_ms;
-
-    /* Get the index position to compare with the 'nowindex' position.
-       Get the oldest entry possible. While we have less than CURR_TIME
-       entries, the first entry will remain the oldest. */
-    checkindex = (data->progress.speeder_c>=CURR_TIME)?
-      data->progress.speeder_c%CURR_TIME:0;
-
-    /* Figure out the exact time for the time span */
-    span_ms = Curl_tvdiff(now,
-                          data->progress.speeder_time[checkindex]);
-    if(0 == span_ms)
-      span_ms=1; /* at least one millisecond MUST have passed */
-
-    /* Calculate the average speed the last 'span_ms' milliseconds */
+    ; /* never update this more than once a second if the end isn't
+          reached */
+  else {
+    data->progress.lastshow = now.tv_sec;
+  
+    /* Let's do the "current speed" thing, which should use the fastest
+       of the dl/ul speeds. Store the fasted speed at entry 'nowindex'. */
+    data->progress.speeder[ nowindex ] =
+      data->progress.downloaded>data->progress.uploaded?
+      data->progress.downloaded:data->progress.uploaded;
+  
+    /* remember the exact time for this moment */
+    data->progress.speeder_time [ nowindex ] = now;
+  
+    /* advance our speeder_c counter, which is increased every time we get
+       here and we expect it to never wrap as 2^32 is a lot of seconds! */
+    data->progress.speeder_c++;
+  
+    /* figure out how many index entries of data we have stored in our speeder
+       array. With N_ENTRIES filled in, we have about N_ENTRIES-1 seconds of
+       transfer. Imagine, after one second we have filled in two entries,
+       after two seconds we've filled in three entries etc. */
+    countindex = ((data->progress.speeder_c>=CURR_TIME)?
+                  CURR_TIME:data->progress.speeder_c) - 1;
+  
+    /* first of all, we don't do this if there's no counted seconds yet */
+    if(countindex) {
+      long span_ms;
+  
+      /* Get the index position to compare with the 'nowindex' position.
+         Get the oldest entry possible. While we have less than CURR_TIME
+         entries, the first entry will remain the oldest. */
+      checkindex = (data->progress.speeder_c>=CURR_TIME)?
+        data->progress.speeder_c%CURR_TIME:0;
+  
+      /* Figure out the exact time for the time span */
+      span_ms = Curl_tvdiff(now,
+                            data->progress.speeder_time[checkindex]);
+      if(0 == span_ms)
+        span_ms=1; /* at least one millisecond MUST have passed */
+  
+      /* Calculate the average speed the last 'span_ms' milliseconds */
+      {
+        curl_off_t amount = data->progress.speeder[nowindex]-
+          data->progress.speeder[checkindex];
+  
+        if(amount > 4294967 /* 0xffffffff/1000 */)
+          /* the 'amount' value is bigger than would fit in 32 bits if
+             multiplied with 1000, so we use the double math for this */
+          data->progress.current_speed = (curl_off_t)
+            ((double)amount/((double)span_ms/1000.0));
+        else
+          /* the 'amount' value is small enough to fit within 32 bits even
+             when multiplied with 1000 */
+          data->progress.current_speed = amount*1000/span_ms;
+      }
+    }
+    else
+      /* the first second we use the main average */
+      data->progress.current_speed =
+        (data->progress.ulspeed>data->progress.dlspeed)?
+        data->progress.ulspeed:data->progress.dlspeed;
+  
+    if(!(data->progress & PGRS_HIDE) && !data->set.fprogress)
     {
-      curl_off_t amount = data->progress.speeder[nowindex]-
-        data->progress.speeder[checkindex];
-
-      if(amount > 4294967 /* 0xffffffff/1000 */)
-        /* the 'amount' value is bigger than would fit in 32 bits if
-           multiplied with 1000, so we use the double math for this */
-        data->progress.current_speed = (curl_off_t)
-          ((double)amount/((double)span_ms/1000.0));
-      else
-        /* the 'amount' value is small enough to fit within 32 bits even
-           when multiplied with 1000 */
-        data->progress.current_speed = amount*1000/span_ms;
+      /* Figure out the estimated time of arrival for the upload */
+      if((data->progress.flags & PGRS_UL_SIZE_KNOWN) &&
+         (data->progress.ulspeed>0) &&
+         (data->progress.size_ul > 100) ) {
+        ulestimate = (long)(data->progress.size_ul / data->progress.ulspeed);
+        ulpercen = (int)(100*(data->progress.uploaded/100) /
+                          (data->progress.size_ul/100) );
+      }
+    
+      /* ... and the download */
+      if((data->progress.flags & PGRS_DL_SIZE_KNOWN) &&
+         (data->progress.dlspeed>0) &&
+         (data->progress.size_dl>100)) {
+        dlestimate = (long)(data->progress.size_dl / data->progress.dlspeed);
+        dlpercen = (int)(100*(data->progress.downloaded/100) /
+                          (data->progress.size_dl/100));
+      }
+    
+      /* Now figure out which of them that is slower and use for the for
+         total estimate! */
+      total_estimate = ulestimate>dlestimate?ulestimate:dlestimate;
+    
+      /* create the three time strings */
+      time2str(time_left, total_estimate > 0?(total_estimate - timespent):0);
+      time2str(time_total, total_estimate);
+      time2str(time_spent, timespent);
+    
+      /* Get the total amount of data expected to get transfered */
+      total_expected_transfer =
+        (data->progress.flags & PGRS_UL_SIZE_KNOWN?
+         data->progress.size_ul:data->progress.uploaded)+
+        (data->progress.flags & PGRS_DL_SIZE_KNOWN?
+         data->progress.size_dl:data->progress.downloaded);
+    
+      /* We have transfered this much so far */
+      total_transfer = data->progress.downloaded + data->progress.uploaded;
+    
+      /* Get the percentage of data transfered so far */
+      if(total_expected_transfer > 100)
+        total_percen=(int)(100*(total_transfer/100) /
+                           (total_expected_transfer/100) );
+    
+      fprintf(data->set.err,
+              "\r%3d %s  %3d %s  %3d %s  %s  %s %s %s %s %s",
+              total_percen,  /* 3 letters */                /* total % */
+              max5data(total_expected_transfer, max5[2]),   /* total size */
+              dlpercen,      /* 3 letters */                /* rcvd % */
+              max5data(data->progress.downloaded, max5[0]), /* rcvd size */
+              ulpercen,      /* 3 letters */                /* xfer % */
+              max5data(data->progress.uploaded, max5[1]),   /* xfer size */
+              max5data(data->progress.dlspeed, max5[3]),    /* avrg dl speed */
+              max5data(data->progress.ulspeed, max5[4]),    /* avrg ul speed */
+              time_total,    /* 8 letters */                /* total time */
+              time_spent,    /* 8 letters */                /* time spent */
+              time_left,     /* 8 letters */                /* time left */
+              max5data(data->progress.current_speed, max5[5]) /* current speed */
+              );
+    
+      /* we flush the output stream to make it appear as soon as possible */
+      fflush(data->set.err);
     }
   }
-  else
-    /* the first second we use the main average */
-    data->progress.current_speed =
-      (data->progress.ulspeed>data->progress.dlspeed)?
-      data->progress.ulspeed:data->progress.dlspeed;
-
-  if(data->progress.flags & PGRS_HIDE)
-    return 0;
 
-  else if(data->set.fprogress) {
+  if(data->set.fprogress && !(data->progress & PGRS_HIDE)) {
     /* There's a callback set, so we call that instead of writing
        anything ourselves. This really is the way to go. */
     result= data->set.fprogress(data->set.progress_client,
@@ -359,66 +422,5 @@
     return result;
   }
 
-  /* Figure out the estimated time of arrival for the upload */
-  if((data->progress.flags & PGRS_UL_SIZE_KNOWN) &&
-     (data->progress.ulspeed>0) &&
-     (data->progress.size_ul > 100) ) {
-    ulestimate = (long)(data->progress.size_ul / data->progress.ulspeed);
-    ulpercen = (int)(100*(data->progress.uploaded/100) /
-                      (data->progress.size_ul/100) );
-  }
-
-  /* ... and the download */
-  if((data->progress.flags & PGRS_DL_SIZE_KNOWN) &&
-     (data->progress.dlspeed>0) &&
-     (data->progress.size_dl>100)) {
-    dlestimate = (long)(data->progress.size_dl / data->progress.dlspeed);
-    dlpercen = (int)(100*(data->progress.downloaded/100) /
-                      (data->progress.size_dl/100));
-  }
-
-  /* Now figure out which of them that is slower and use for the for
-     total estimate! */
-  total_estimate = ulestimate>dlestimate?ulestimate:dlestimate;
-
-  /* create the three time strings */
-  time2str(time_left, total_estimate > 0?(total_estimate - timespent):0);
-  time2str(time_total, total_estimate);
-  time2str(time_spent, timespent);
-
-  /* Get the total amount of data expected to get transfered */
-  total_expected_transfer =
-    (data->progress.flags & PGRS_UL_SIZE_KNOWN?
-     data->progress.size_ul:data->progress.uploaded)+
-    (data->progress.flags & PGRS_DL_SIZE_KNOWN?
-     data->progress.size_dl:data->progress.downloaded);
-
-  /* We have transfered this much so far */
-  total_transfer = data->progress.downloaded + data->progress.uploaded;
-
-  /* Get the percentage of data transfered so far */
-  if(total_expected_transfer > 100)
-    total_percen=(int)(100*(total_transfer/100) /
-                       (total_expected_transfer/100) );
-
-  fprintf(data->set.err,
-          "\r%3d %s  %3d %s  %3d %s  %s  %s %s %s %s %s",
-          total_percen,  /* 3 letters */                /* total % */
-          max5data(total_expected_transfer, max5[2]),   /* total size */
-          dlpercen,      /* 3 letters */                /* rcvd % */
-          max5data(data->progress.downloaded, max5[0]), /* rcvd size */
-          ulpercen,      /* 3 letters */                /* xfer % */
-          max5data(data->progress.uploaded, max5[1]),   /* xfer size */
-          max5data(data->progress.dlspeed, max5[3]),    /* avrg dl speed */
-          max5data(data->progress.ulspeed, max5[4]),    /* avrg ul speed */
-          time_total,    /* 8 letters */                /* total time */
-          time_spent,    /* 8 letters */                /* time spent */
-          time_left,     /* 8 letters */                /* time left */
-          max5data(data->progress.current_speed, max5[5]) /* current speed */
-          );
-
-  /* we flush the output stream to make it appear as soon as possible */
-  fflush(data->set.err);
-
   return 0;
 }
