[Toybox] [PATCH] file: fix more '%' conversions than data arguments warning.

[Rob Landley]

On 09/17/2018 06:11 PM, enh wrote:
> Not sure how I messed this up, since both the toybox and Android build
> systems warn about this. (Android has -Werror, so at least it kept me
> from checking this in there.)

I fixed this on sunday, just forgot to push. (And I saw this email in the web
archive wednesday-ish, and today I fished it out of gmail's spam filter.)

> -    xprintf("BMP image, %d x %d, %d bpp, %s\n", w, h, bpp);
> +    xprintf("BMP image, %d x %d, %d bpp\n", w, h, bpp);

Same fix I did. I tried to figure out what argument to supply, but the ubuntu
"file" treats bmp as "data", so...

Speaking of which, I still want a bunch of tiny test files for this command. I
made a test bmp with "the gimp", and I can whip up a bunch of hello world ELF
files easily enough, but there's a lot of file types here...

Also, "tests/file.test" reading from "tests/files/file" is way too confusing,
and I'm open to suggestions. Really tests/*.test is kinda redundant already, and
having "file" under tests... Hmmm...

Maybe:

  test/scripts/{readlink,sed,rm...}
  test/files/{utf8,blkid,bzcat...}

Alternately, moving the files out of the "files" subdirectory and putting them
in with tests gives the ".test" suffix a reason to exist. (So tests/bzcat.test
and tests/bzcat/ would both be for bzcat...)

Sigh. Aesthetic decision. No emprically right answer. :P

Rob