]> sjero.net Git - wget/blobdiff - src/retr.c
[svn] Big progress bar update.
[wget] / src / retr.c
index 75ae3252d1911a1ea97c8dff19701fcefdcac1f7..9b8ddc214ad74f429ccd45cca6fdbfe1478bf38d 100644 (file)
@@ -36,6 +36,7 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include "wget.h"
 #include "utils.h"
 #include "retr.h"
+#include "progress.h"
 #include "url.h"
 #include "recur.h"
 #include "ftp.h"
@@ -51,11 +52,6 @@ extern int errno;
 int global_download_count;
 
 \f
-/* Flags for show_progress().  */
-enum spflags { SP_NONE, SP_INIT, SP_FINISH };
-
-static int show_progress PARAMS ((long, long, enum spflags));
-
 #define MIN(i, j) ((i) <= (j) ? (i) : (j))
 
 /* Reads the contents of file descriptor FD, until it is closed, or a
@@ -85,23 +81,28 @@ get_contents (int fd, FILE *fp, long *len, long restval, long expected,
 {
   int res = 0;
   static char c[8192];
+  void *progress = NULL;
 
   *len = restval;
   if (opt.verbose)
-    show_progress (restval, expected, SP_INIT);
+    progress = progress_create (restval, expected);
+
   if (rbuf && RBUF_FD (rbuf) == fd)
     {
+      int need_flush = 0;
       while ((res = rbuf_flush (rbuf, c, sizeof (c))) != 0)
        {
          if (fwrite (c, sizeof (char), res, fp) < res)
            return -2;
          if (opt.verbose)
-           {
-             if (show_progress (res, expected, SP_NONE))
-               fflush (fp);
-           }
+           progress_update (progress, res);
          *len += res;
+         need_flush = 1;
        }
+      if (need_flush)
+       fflush (fp);
+      if (ferror (fp))
+       return -2;
     }
   /* Read from fd while there is available data.
 
@@ -124,13 +125,15 @@ get_contents (int fd, FILE *fp, long *len, long restval, long expected,
 #endif /* HAVE_SSL */
       if (res > 0)
        {
-         if (fwrite (c, sizeof (char), res, fp) < res)
+         fwrite (c, sizeof (char), res, fp);
+         /* Always flush the contents of the network packet.  This
+            should not be adverse to performance, as the network
+            packets typically won't be too tiny anyway.  */
+         fflush (fp);
+         if (ferror (fp))
            return -2;
          if (opt.verbose)
-           {
-             if (show_progress (res, expected, SP_NONE))
-               fflush (fp);
-           }
+           progress_update (progress, res);
          *len += res;
        }
       else
@@ -139,132 +142,14 @@ get_contents (int fd, FILE *fp, long *len, long restval, long expected,
   if (res < -1)
     res = -1;
   if (opt.verbose)
-    show_progress (0, expected, SP_FINISH);
+    progress_finish (progress);
   return res;
 }
-
-static void
-print_percentage (long bytes, long expected)
-{
-  int percentage = (int)(100.0 * bytes / expected);
-  logprintf (LOG_VERBOSE, "%3d%%", percentage);
-}
-
-/* Show the dotted progress report of file loading.  Called with
-   length and a flag to tell it whether to reset or not.  It keeps the
-   offset information in static local variables.
-
-   Return value: 1 or 0, designating whether any dots have been drawn.
-
-   If the init argument is set, the routine will initialize.
-
-   If the res is non-zero, res/line_bytes lines are skipped
-   (meaning the appropriate number ok kilobytes), and the number of
-   "dots" fitting on the first line are drawn as ','.  */
-static int
-show_progress (long res, long expected, enum spflags flags)
-{
-  static struct wget_timer *timer;
-  static long line_bytes;
-  static long offs, initial_skip;
-  static int ndot, nrow;
-  static long last_timer_value, time_offset;
-  int any_output = 0;
-
-  if (flags == SP_FINISH)
-    {
-      int dot = ndot;
-      char *tmpstr = (char *)alloca (2 * opt.dots_in_line + 1);
-      char *tmpp = tmpstr;
-      time_offset = wtimer_elapsed (timer) - last_timer_value;
-      for (; dot < opt.dots_in_line; dot++)
-       {
-         if (!(dot % opt.dot_spacing))
-           *tmpp++ = ' ';
-         *tmpp++ = ' ';
-       }
-      *tmpp = '\0';
-      logputs (LOG_VERBOSE, tmpstr);
-      if (expected)
-       print_percentage (nrow * line_bytes + ndot * opt.dot_bytes + offs,
-                         expected);
-      logprintf (LOG_VERBOSE, " @%s",
-                rate (ndot * opt.dot_bytes
-                      + offs - (initial_skip % line_bytes),
-                      time_offset, 1));
-      logputs (LOG_VERBOSE, "\n\n");
-      return 0;
-    }
-
-  /* Temporarily disable flushing.  */
-  log_set_flush (0);
-
-  /* init set means initialization.  If res is set, it also means that
-     the retrieval is *not* done from the beginning.  The part that
-     was already retrieved is not shown again.  */
-  if (flags == SP_INIT)
-    {
-      /* Generic initialization of static variables.  */
-      offs = 0L;
-      ndot = nrow = 0;
-      line_bytes = (long)opt.dots_in_line * opt.dot_bytes;
-      if (!timer)
-       timer = wtimer_allocate ();
-      wtimer_reset (timer);
-      last_timer_value = 0;
-      time_offset = 0;
-      initial_skip = res;
-      if (res)
-       {
-         if (res >= line_bytes)
-           {
-             nrow = res / line_bytes;
-             res %= line_bytes;
-             logprintf (LOG_VERBOSE,
-                        _("\n          [ skipping %dK ]"),
-                        (int) ((nrow * line_bytes) / 1024));
-             ndot = 0;
-           }
-       }
-      logprintf (LOG_VERBOSE, "\n%5ldK", nrow * line_bytes / 1024);
-    }
-  /* Offset gets incremented by current value.  */
-  offs += res;
-  /* While offset is >= opt.dot_bytes, print dots, taking care to
-     precede every 50th dot with a status message.  */
-  for (; offs >= opt.dot_bytes; offs -= opt.dot_bytes)
-    {
-      if (!(ndot % opt.dot_spacing))
-       logputs (LOG_VERBOSE, " ");
-      any_output = 1;
-      logputs (LOG_VERBOSE, flags == SP_INIT ? "," : ".");
-      ++ndot;
-      if (ndot == opt.dots_in_line)
-       {
-         time_offset = wtimer_elapsed (timer) - last_timer_value;
-         last_timer_value += time_offset;
-
-         ndot = 0;
-         ++nrow;
-         if (expected)
-           print_percentage (nrow * line_bytes, expected);
-         logprintf (LOG_VERBOSE, " @%s",
-                    rate (line_bytes - (initial_skip % line_bytes),
-                          time_offset, 1));
-         initial_skip = 0;
-         logprintf (LOG_VERBOSE, "\n%5ldK", nrow * line_bytes / 1024);
-       }
-    }
-
-  /* Reenable flushing.  */
-  log_set_flush (1);
-
-  return any_output;
-}
 \f
-/* Print out the appropriate download rate.  Appropriate means that if
-   rate is > 1024 bytes per second, kilobytes are used, and if rate >
-   1024 * 1024 bps, megabytes are used.
+/* Return a printed representation of the download rate, as
+   appropriate for the speed.  Appropriate means that if rate is
+   greater than 1K/s, kilobytes are used, and if rate is greater than
+   1MB/s, megabytes are used.
 
    If PAD is non-zero, strings will be padded to the width of 7
    characters (xxxx.xx).  */
@@ -274,6 +159,9 @@ rate (long bytes, long msecs, int pad)
   static char res[15];
   double dlrate;
 
+  assert (msecs >= 0);
+  assert (bytes >= 0);
+
   if (msecs == 0)
     /* If elapsed time is 0, it means we're under the granularity of
        the timer.  This often happens on systems that use time() for
@@ -284,9 +172,9 @@ rate (long bytes, long msecs, int pad)
   if (dlrate < 1024.0)
     sprintf (res, pad ? "%7.2f B/s" : "%.2f B/s", dlrate);
   else if (dlrate < 1024.0 * 1024.0)
-    sprintf (res, pad ? "%7.2f KB/s" : "%.2f KB/s", dlrate / 1024.0);
+    sprintf (res, pad ? "%7.2f K/s" : "%.2f K/s", dlrate / 1024.0);
   else if (dlrate < 1024.0 * 1024.0 * 1024.0)
-    sprintf (res, pad ? "%7.2f MB/s" : "%.2f MB/s", dlrate / (1024.0 * 1024.0));
+    sprintf (res, pad ? "%7.2f M/s" : "%.2f M/s", dlrate / (1024.0 * 1024.0));
   else
     /* Maybe someone will need this one day.  More realistically, it
        will get tickled by buggy timers. */