Hello!

They chose octal because they have -t (--radix) so other outputs may be chosen 
as needed. The usual default for implementations that do not have -t is 
decimal, reflected in busybox having this in the help for strings. If someone 
could write full -t support that would of course be ideal, but until then, 
decimal is the more common (and more useful) output.


Cheers,

Chris


________________________________
From: busybox <busybox-boun...@busybox.net> on behalf of Tito 
<farmat...@tiscali.it>
Sent: October 11, 2016 3:38 PM
To: busybox@busybox.net
Subject: Re: Subject: [PATCH] strings: Correct output to be decimal per the 
help, instead of octal

On 10/11/2016 04:00 PM, Chris Renshaw wrote:
>>From fe9333ce1d69e292e14144a61ff88a23ffed0053 Mon Sep 17 00:00:00 2001
> From: Chris Renshaw <osm0...@outlook.com>
> Date: Thu, 22 Sep 2016 16:17:05 -0300
> Subject: [PATCH] strings: Correct output to be decimal per the help, instead 
> of octal
>  Most smaller/incomplete implementations of strings output decimal by default,
>  so best to correct that behavior rather than just change the help to say 
> octal
>  (it would be great to get full POSIX --radix at some point, to give the
>  output in any desired format, but this will do for now)
>
> ---
>  miscutils/strings.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/miscutils/strings.c b/miscutils/strings.c
> index 9f50182..cc4ad9a 100644
> --- a/miscutils/strings.c
> +++ b/miscutils/strings.c
> @@ -68,7 +68,7 @@ int strings_main(int argc UNUSED_PARAM, char **argv)
>                                                        printf(fmt, *argv);
>                                                }
>                                                if (option_mask32 & 
> PRINT_OFFSET) {
> -                                                     printf("%7"OFF_FMT"o ", 
> offset - n);
> +                                                     printf("%7"OFF_FMT"d ", 
> offset - n);
>                                                }
>                                                fputs(string, stdout);
>                                        }
>


Hi,
man STRINGS(1) GNU Development Tools binutils-2.25 says:

"-o  Like -t o.  Some other versions of strings have -o act like -t d instead.
Since we can not be compatible with both ways, we simply chose one."

As we mimick GNU strings for linux maybe correcting the help would be the better
option.

Ciao,
Tito

_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
busybox Info Page<http://lists.busybox.net/mailman/listinfo/busybox>
lists.busybox.net
BusyBox combines tiny versions of many common UNIX utilities into a single 
small executable. It provides minimalist replacements for most of the utilities 
you usually ...



_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to