[svn] Default the progress implementation to "bar".

Published in <sxsu1vgfqx7.fsf@florida.arsdigita.de>.
This commit is contained in:
hniksic 2001-11-27 02:29:45 -08:00
parent 69c78db55b
commit baf8750197
3 changed files with 19 additions and 7 deletions

7
NEWS
View File

@ -7,8 +7,11 @@ Please send GNU Wget bug reports to <bug-wget@gnu.org>.
* Changes in Wget 1.8. * Changes in Wget 1.8.
** A new progress indicator is now available. Try it with ** A new progress indicator is now available and used by default.
--progress=bar or using `progress = bar' in `.wgetrc'. You can choose the progress bar type with `--progress=TYPE'. Two
types are available, "bar" (the new default), and "dot" (the old
dotted indicator). You can permanently revert to the old progress
indicator by putting `progress = dot' in your `.wgetrc'.
** "Recursive retrieval" has been revamped: ** "Recursive retrieval" has been revamped:

View File

@ -1,3 +1,7 @@
2001-11-27 Hrvoje Niksic <hniksic@arsdigita.com>
* progress.c: Change the default progress implementation to "bar".
2001-11-27 Hrvoje Niksic <hniksic@arsdigita.com> 2001-11-27 Hrvoje Niksic <hniksic@arsdigita.com>
* progress.c (bar_create): Print two newlines. * progress.c (bar_create): Print two newlines.

View File

@ -62,13 +62,18 @@ static struct progress_implementation implementations[] = {
}; };
static struct progress_implementation *current_impl; static struct progress_implementation *current_impl;
/* Default progress implementation should be something that works /* Progress implementation used by default. Can be overriden in
wgetrc or by the fallback one. */
#define DEFAULT_PROGRESS_IMPLEMENTATION "bar"
/* Fallnback progress implementation should be something that works
under all display types. If you put something other than "dot" under all display types. If you put something other than "dot"
here, remember that bar_set_params tries to switch to this if we're here, remember that bar_set_params tries to switch to this if we're
not running on a TTY. So changing this to "bar" could cause not running on a TTY. So changing this to "bar" could cause
infloop. */ infloop. */
#define DEFAULT_PROGRESS_IMPLEMENTATION "dot" #define FALLBACK_PROGRESS_IMPLEMENTATION "dot"
/* Return non-zero if NAME names a valid progress bar implementation. /* Return non-zero if NAME names a valid progress bar implementation.
The characters after the first : will be ignored. */ The characters after the first : will be ignored. */
@ -659,9 +664,9 @@ bar_set_params (const char *params)
/* We're not printing to a TTY, so revert to the fallback /* We're not printing to a TTY, so revert to the fallback
display. #### We're recursively calling display. #### We're recursively calling
set_progress_implementation here, which is slightly kludgy. set_progress_implementation here, which is slightly kludgy.
It would be nicer if that function could resolve this problem It would be nicer if we provided that function a return value
itself. */ indicating a failure of some sort. */
set_progress_implementation (NULL); set_progress_implementation (FALLBACK_PROGRESS_IMPLEMENTATION);
return; return;
} }