chsmack: put usage string at head
authorJosé Bollo <jose.bollo@open.eurogiciel.org>
Tue, 10 Dec 2013 13:13:23 +0000 (14:13 +0100)
committerJosé Bollo <jose.bollo@open.eurogiciel.org>
Tue, 17 Dec 2013 09:56:23 +0000 (10:56 +0100)
Setting it at head is a reading improvement for
developpers that can quickly show the usage of the
program. It also makes the main function more readable.

Signed-off-by: José Bollo <jose.bollo@open.eurogiciel.org>
utils/chsmack.c

index c9d9ecf..b7d90b0 100644 (file)
 #include <string.h>
 #include <getopt.h>
 
+static const char usage[] =
+       "Usage: %s [options] <path>\n"
+       "options:\n"  
+       " -a --access        set/remove "XATTR_NAME_SMACK"\n"  
+       " -e --exec          set/remove "XATTR_NAME_SMACKEXEC"\n"  
+       " -m --mmap          set/remove "XATTR_NAME_SMACKMMAP"\n"  
+       " -t --transmute     set/remove "XATTR_NAME_SMACKTRANSMUTE"\n"
+;
+
 int main(int argc, char *argv[])
 {
        static struct option options[] = {
@@ -84,12 +93,7 @@ int main(int argc, char *argv[])
                                transmute_flag = 1;
                                break;
                        default:
-                               printf("Usage: %s [options] <path>\n", basename(argv[0]));
-                               printf("options:\n");
-                               printf(" [--access|-a] set "XATTR_NAME_SMACK"\n");
-                               printf(" [--exec|-e] set "XATTR_NAME_SMACKEXEC"\n");
-                               printf(" [--mmap|-m] set "XATTR_NAME_SMACKMMAP"\n");
-                               printf(" [--transmute|-t] set "XATTR_NAME_SMACKTRANSMUTE"\n");
+                               printf(usage, basename(argv[0]));
                                exit(1);
                }