diff options
author | Jesper Juhl <jj@chaosbits.net> | 2011-06-15 05:53:13 -0400 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2011-06-16 14:40:03 -0400 |
commit | f0f3ca8d967462dafb815412b14ca3339b9817a6 (patch) | |
tree | 2bbea0758c9e5519a81a985b75278054efd08551 /scripts/docproc.c | |
parent | 1c5454eed85af71df9c01ab923e0c1b841b2e99b (diff) |
docproc: cleanup brace placement
The placement of the opening brace "{" after 'if' statements in
scripts/docproc.c is inconsistent. Most are placed on the same line as the 'if'
statement itself as per CodingStyle, but a few are not. This patch cleans up
the inconsistency. We save a few source lines and the file then uses the same
style throughout, which is nice.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'scripts/docproc.c')
-rw-r--r-- | scripts/docproc.c | 17 |
1 files changed, 5 insertions, 12 deletions
diff --git a/scripts/docproc.c b/scripts/docproc.c index 98dec87974d0..4cfdc1797eb8 100644 --- a/scripts/docproc.c +++ b/scripts/docproc.c | |||
@@ -205,8 +205,7 @@ static void find_export_symbols(char * filename) | |||
205 | PATH_MAX - strlen(real_filename)); | 205 | PATH_MAX - strlen(real_filename)); |
206 | sym = add_new_file(filename); | 206 | sym = add_new_file(filename); |
207 | fp = fopen(real_filename, "r"); | 207 | fp = fopen(real_filename, "r"); |
208 | if (fp == NULL) | 208 | if (fp == NULL) { |
209 | { | ||
210 | fprintf(stderr, "docproc: "); | 209 | fprintf(stderr, "docproc: "); |
211 | perror(real_filename); | 210 | perror(real_filename); |
212 | exit(1); | 211 | exit(1); |
@@ -487,8 +486,7 @@ static void parse_file(FILE *infile) | |||
487 | default: | 486 | default: |
488 | defaultline(line); | 487 | defaultline(line); |
489 | } | 488 | } |
490 | } | 489 | } else { |
491 | else { | ||
492 | defaultline(line); | 490 | defaultline(line); |
493 | } | 491 | } |
494 | } | 492 | } |
@@ -519,8 +517,7 @@ int main(int argc, char *argv[]) | |||
519 | exit(2); | 517 | exit(2); |
520 | } | 518 | } |
521 | 519 | ||
522 | if (strcmp("doc", argv[1]) == 0) | 520 | if (strcmp("doc", argv[1]) == 0) { |
523 | { | ||
524 | /* Need to do this in two passes. | 521 | /* Need to do this in two passes. |
525 | * First pass is used to collect all symbols exported | 522 | * First pass is used to collect all symbols exported |
526 | * in the various files; | 523 | * in the various files; |
@@ -556,9 +553,7 @@ int main(int argc, char *argv[]) | |||
556 | fprintf(stderr, "Warning: didn't use docs for %s\n", | 553 | fprintf(stderr, "Warning: didn't use docs for %s\n", |
557 | all_list[i]); | 554 | all_list[i]); |
558 | } | 555 | } |
559 | } | 556 | } else if (strcmp("depend", argv[1]) == 0) { |
560 | else if (strcmp("depend", argv[1]) == 0) | ||
561 | { | ||
562 | /* Create first part of dependency chain | 557 | /* Create first part of dependency chain |
563 | * file.tmpl */ | 558 | * file.tmpl */ |
564 | printf("%s\t", argv[2]); | 559 | printf("%s\t", argv[2]); |
@@ -571,9 +566,7 @@ int main(int argc, char *argv[]) | |||
571 | findall = adddep; | 566 | findall = adddep; |
572 | parse_file(infile); | 567 | parse_file(infile); |
573 | printf("\n"); | 568 | printf("\n"); |
574 | } | 569 | } else { |
575 | else | ||
576 | { | ||
577 | fprintf(stderr, "Unknown option: %s\n", argv[1]); | 570 | fprintf(stderr, "Unknown option: %s\n", argv[1]); |
578 | exit(1); | 571 | exit(1); |
579 | } | 572 | } |