diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2023-04-13 00:10:10 +0200 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2023-04-13 03:02:11 +0200 |
| commit | 8b72877babb20be9bb46c4437f5e1870390c29cc (patch) | |
| tree | a688bf5a44babe327e4e7d8b897a84716886981c | |
| parent | 7c1f975cddccff1e9705e750eed8a54131d7a392 (diff) | |
| download | busybox-8b72877babb20be9bb46c4437f5e1870390c29cc.tar.gz | |
shuf: another tweak to COMMON_PREFIX_HACK code
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
| -rw-r--r-- | coreutils/shuf.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/coreutils/shuf.c b/coreutils/shuf.c index 107899a10..466969745 100644 --- a/coreutils/shuf.c +++ b/coreutils/shuf.c @@ -67,7 +67,7 @@ static void shuffle_lines(char **lines, unsigned numlines, unsigned outlines) } /* We can handle insanity like this: - * shuf -i 3333333333333333333333333333333333333333333333333333333333333123456789001-3333333333333333333333333333333333333333333333333333333333333123456789019 + * shuf -i 333333333333333333333333333333001-333333333333333333333333333333019 * but do we want to have +200 bytes of code (~40% code growth)? */ #define COMMON_PREFIX_HACK 0 @@ -128,16 +128,12 @@ int shuf_main(int argc, char **argv) if (padding_width > 5 && padding_width == strlen(b)) { /* How long is it? */ pfx = a; - while (isdigit(*a) && *a == *b) { + while (isdigit(*a) && *a == *b + && a[1] /* "111111-111111" case: avoid xatoull("") */ + ) { a++; b++; } - if (*a == '\0') { - /* "123456-123456", and we 'ate' all of them */ - /* prevent trying to xatoull("") */ - a--; - b--; - } pfx_len = a - pfx; /* can end up being 0 */ padding_width -= pfx_len; } else { |