summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJim Meyering <jim@meyering.net>2002-11-05 20:28:36 +0000
committerJim Meyering <jim@meyering.net>2002-11-05 20:28:36 +0000
commitf3fc3d47268365532b022079de28528bfbf082a7 (patch)
treeed5013d5c175d74015b86ed33d5090176607fecc /src
parent3f2cc46da4afe2fac5aecd2009c8fa26e28d36e0 (diff)
downloadcoreutils-f3fc3d47268365532b022079de28528bfbf082a7.tar.xz
(dopass):
Use primitives from inttostr.h, not human.h, to print large numbers simply. (OUTPUT_BLOCK_SIZE): remove. (dopass): When printing progress, use floor for what has been done so far (since we should be conservative there), and ceiling for what needs to be done (since that's what other programs use).
Diffstat (limited to 'src')
-rw-r--r--src/shred.c27
1 files changed, 14 insertions, 13 deletions
diff --git a/src/shred.c b/src/shred.c
index 51dad9e71..1570212cd 100644
--- a/src/shred.c
+++ b/src/shred.c
@@ -107,6 +107,7 @@
#include "closeout.h"
#include "error.h"
#include "human.h"
+#include "inttostr.h"
#include "quotearg.h" /* For quotearg_colon */
#include "quote.h" /* For quotearg_colon */
char *xstrdup PARAMS ((char const *));
@@ -132,10 +133,6 @@ char *xstrdup PARAMS ((char const *));
verbose output line. */
#define VERBOSE_UPDATE 5
-/* If positive, the units to use when printing sizes;
- if negative, the human-readable base. */
-#define OUTPUT_BLOCK_SIZE (-1024)
-
struct Options
{
int force; /* -f flag: chmod files if necessary */
@@ -878,11 +875,9 @@ dopass (int fd, char const *qname, off_t *sizep, int type,
else
{
int errnum = errno;
- char buf[LONGEST_HUMAN_READABLE + 1];
+ char buf[INT_BUFSIZE_BOUND (uintmax_t)];
error (0, errnum, _("%s: error writing at offset %s"),
- qname,
- human_readable ((uintmax_t) (offset + soff),
- buf, 1, 1));
+ qname, umaxtostr ((uintmax_t) offset + soff, buf));
/*
* I sometimes use shred on bad media, before throwing it
* out. Thus, I don't want it to give up on bad blocks.
@@ -923,9 +918,11 @@ dopass (int fd, char const *qname, off_t *sizep, int type,
{
char offset_buf[LONGEST_HUMAN_READABLE + 1];
char size_buf[LONGEST_HUMAN_READABLE + 1];
+ int human_progress_opts = (human_autoscale | human_SI
+ | human_base_1024 | human_B);
char const *human_offset
- = human_readable ((uintmax_t) offset, offset_buf, 1,
- OUTPUT_BLOCK_SIZE);
+ = human_readable (offset, offset_buf,
+ human_floor | human_progress_opts, 1, 1);
if (offset == size
|| !STREQ (previous_human_offset, human_offset))
@@ -941,10 +938,14 @@ dopass (int fd, char const *qname, off_t *sizep, int type,
: (off <= TYPE_MAXIMUM (uintmax_t) / 100
? off * 100 / size
: off / (size / 100)));
+ char const *human_size
+ = human_readable (size, size_buf,
+ human_ceiling | human_progress_opts,
+ 1, 1);
+ if (offset == size)
+ human_offset = human_size;
error (0, 0, _("%s: pass %lu/%lu (%s)...%s/%s %d%%"),
- qname, k, n, pass_string, human_offset,
- human_readable ((uintmax_t) size, size_buf, 1,
- OUTPUT_BLOCK_SIZE),
+ qname, k, n, pass_string, human_offset, human_size,
percent);
}