diff options
author | Felipe Contreras <felipe.contreras@gmail.com> | 2014-01-23 18:54:35 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-01-23 19:36:57 -0500 |
commit | 9fd4305448a4639deade433893c5233a324df3a2 (patch) | |
tree | 6f083a5831209c68cfa9356dff77d46ddea1a525 /lib/cmdline.c | |
parent | ae2924a2bdc5255745e68f2b9206404ddadfc5bf (diff) |
lib/cmdline.c: fix style issues
WARNING: space prohibited between function name and open parenthesis '('
+int get_option (char **str, int *pint)
WARNING: space prohibited between function name and open parenthesis '('
+ *pint = simple_strtol (cur, str, 0);
ERROR: trailing whitespace
+ $
WARNING: please, no spaces at the start of a line
+ $
WARNING: space prohibited between function name and open parenthesis '('
+ res = get_option ((char **)&str, ints + i);
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/cmdline.c')
-rw-r--r-- | lib/cmdline.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/lib/cmdline.c b/lib/cmdline.c index eb6791188cf5..54663335226f 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c | |||
@@ -49,13 +49,13 @@ static int get_range(char **str, int *pint) | |||
49 | * 3 - hyphen found to denote a range | 49 | * 3 - hyphen found to denote a range |
50 | */ | 50 | */ |
51 | 51 | ||
52 | int get_option (char **str, int *pint) | 52 | int get_option(char **str, int *pint) |
53 | { | 53 | { |
54 | char *cur = *str; | 54 | char *cur = *str; |
55 | 55 | ||
56 | if (!cur || !(*cur)) | 56 | if (!cur || !(*cur)) |
57 | return 0; | 57 | return 0; |
58 | *pint = simple_strtol (cur, str, 0); | 58 | *pint = simple_strtol(cur, str, 0); |
59 | if (cur == *str) | 59 | if (cur == *str) |
60 | return 0; | 60 | return 0; |
61 | if (**str == ',') { | 61 | if (**str == ',') { |
@@ -84,13 +84,13 @@ int get_option (char **str, int *pint) | |||
84 | * the parse to end (typically a null terminator, if @str is | 84 | * the parse to end (typically a null terminator, if @str is |
85 | * completely parseable). | 85 | * completely parseable). |
86 | */ | 86 | */ |
87 | 87 | ||
88 | char *get_options(const char *str, int nints, int *ints) | 88 | char *get_options(const char *str, int nints, int *ints) |
89 | { | 89 | { |
90 | int res, i = 1; | 90 | int res, i = 1; |
91 | 91 | ||
92 | while (i < nints) { | 92 | while (i < nints) { |
93 | res = get_option ((char **)&str, ints + i); | 93 | res = get_option((char **)&str, ints + i); |
94 | if (res == 0) | 94 | if (res == 0) |
95 | break; | 95 | break; |
96 | if (res == 3) { | 96 | if (res == 3) { |
@@ -153,7 +153,6 @@ unsigned long long memparse(const char *ptr, char **retptr) | |||
153 | return ret; | 153 | return ret; |
154 | } | 154 | } |
155 | 155 | ||
156 | |||
157 | EXPORT_SYMBOL(memparse); | 156 | EXPORT_SYMBOL(memparse); |
158 | EXPORT_SYMBOL(get_option); | 157 | EXPORT_SYMBOL(get_option); |
159 | EXPORT_SYMBOL(get_options); | 158 | EXPORT_SYMBOL(get_options); |