wrong option name in error message for 'brotli -j -k' #282

New Issue

claunia · 2026-01-29T20:41:21Z

claunia commented

2026-01-29 20:41:21 +00:00

Originally created by @jo-he on GitHub (Jul 5, 2019).

Here, -n should be changed to -k:

$ grep -n -- '--keep' c/tools/brotli.c
266:            fprintf(stderr, "argument --rm / -j or --keep / -n already set\n");
391:          fprintf(stderr, "argument --rm / -j or --keep / -n already set\n");
404:          fprintf(stderr, "argument --rm / -j or --keep / -n already set\n");
549:"  -k, --keep

Originally created by @jo-he on GitHub (Jul 5, 2019). Here, `-n` should be changed to `-k`: ``` $ grep -n -- '--keep' c/tools/brotli.c 266: fprintf(stderr, "argument --rm / -j or --keep / -n already set\n"); 391: fprintf(stderr, "argument --rm / -j or --keep / -n already set\n"); 404: fprintf(stderr, "argument --rm / -j or --keep / -n already set\n"); 549:" -k, --keep ```

claunia closed this issue

2026-01-29 20:41:21 +00:00

claunia commented

2026-01-29 20:41:22 +00:00

@eustas commented on GitHub (Jul 16, 2019):

Thanks.
Will land the fix soon.

@eustas commented on GitHub (Jul 16, 2019): Thanks. Will land the fix soon.

claunia commented

2026-01-29 20:41:22 +00:00

@eustas commented on GitHub (Jul 17, 2019):

Fixed

@eustas commented on GitHub (Jul 17, 2019): Fixed

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#282