i3bar: fix usage description, make -b happen
Patch status: merged
Patch by Simon Elsbrock
To apply this patch, use:
curl http://cr.i3wm.org/patch/117/raw.patch | git am
b/i3bar/src/main.c
| 13 |
@@ -53,12 +53,12 @@ char *expand_path(char *path) {
|
| 14 |
} |
| 15 |
|
| 16 |
void print_usage(char *elf_name) {
|
| 17 |
- printf("Usage: %s [-b bar_id] [-s sock_path] [-h] [-v]\n", elf_name);
|
| 18 |
+ printf("Usage: %s -b bar_id [-s sock_path] [-h] [-v]\n", elf_name);
|
| 19 |
printf("\n");
|
| 20 |
- printf("--bar_id <bar_id>\tBar ID for which to get the configuration\n");
|
| 21 |
- printf("-s <sock_path>\tConnect to i3 via <sock_path>\n");
|
| 22 |
- printf("-h\t\tDisplay this help-message and exit\n");
|
| 23 |
- printf("-v\t\tDisplay version number and exit\n");
|
| 24 |
+ printf("-b, --bar_id <bar_id>\tBar ID for which to get the configuration\n");
|
| 25 |
+ printf("-s, --socket <sock_path>\tConnect to i3 via <sock_path>\n");
|
| 26 |
+ printf("-h, --help Display this help-message and exit\n");
|
| 27 |
+ printf("-v, --version Display version number and exit\n");
|
| 28 |
printf("\n");
|
| 29 |
printf(" PLEASE NOTE that i3bar will be automatically started by i3\n"
|
| 30 |
" as soon as there is a 'bar' configuration block in your\n" |
| 31 |
@@ -97,13 +97,13 @@ int main(int argc, char **argv) {
|
| 32 |
|
| 33 |
static struct option long_opt[] = {
|
| 34 |
{ "socket", required_argument, 0, 's' },
|
| 35 |
- { "bar_id", required_argument, 0, 0 },
|
| 36 |
+ { "bar_id", required_argument, 0, 'b' },
|
| 37 |
{ "help", no_argument, 0, 'h' },
|
| 38 |
{ "version", no_argument, 0, 'v' },
|
| 39 |
{ NULL, 0, 0, 0}
|
| 40 |
}; |
| 41 |
|
| 42 |
- while ((opt = getopt_long(argc, argv, "s:hv", long_opt, &option_index)) != -1) {
|
| 43 |
+ while ((opt = getopt_long(argc, argv, "b:s:hv", long_opt, &option_index)) != -1) {
|
| 44 |
switch (opt) {
|
| 45 |
case 's': |
| 46 |
socket_path = expand_path(optarg); |
| 47 |
@@ -112,11 +112,8 @@ int main(int argc, char **argv) {
|
| 48 |
printf("i3bar version " I3_VERSION " © 2010-2011 Axel Wagner and contributors\n");
|
| 49 |
exit(EXIT_SUCCESS); |
| 50 |
break; |
| 51 |
- case 0: |
| 52 |
- if (!strcmp(long_opt[option_index].name, "bar_id")) {
|
| 53 |
- FREE(config.bar_id); |
| 54 |
- config.bar_id = sstrdup(optarg); |
| 55 |
- } |
| 56 |
+ case 'b': |
| 57 |
+ config.bar_id = sstrdup(optarg); |
| 58 |
break; |
| 59 |
default: |
| 60 |
print_usage(argv[0]); |