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 | |
| 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')
| -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); |
