diff options
| author | Cyrill Gorcunov <gorcunov@gmail.com> | 2011-12-05 01:39:04 +0400 |
|---|---|---|
| committer | Cyrill Gorcunov <gorcunov@gmail.com> | 2011-12-05 01:39:04 +0400 |
| commit | 3b8c2974a13a4c73b2a31a8e84d446ef56574897 (patch) | |
| tree | e25b5b2577bac95559decd5521de3761e3d24d91 /nasm.c | |
| parent | 0e754dcafea4d8d458928afb0cc786517244b3f8 (diff) | |
| download | nasm-3b8c2974a13a4c73b2a31a8e84d446ef56574897.tar.gz nasm-3b8c2974a13a4c73b2a31a8e84d446ef56574897.tar.bz2 nasm-3b8c2974a13a4c73b2a31a8e84d446ef56574897.zip | |
Use proper bracing on setting warnings in comman line parsing
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
Diffstat (limited to 'nasm.c')
| -rw-r--r-- | nasm.c | 30 |
1 files changed, 16 insertions, 14 deletions
@@ -868,20 +868,22 @@ static bool process_arg(char *p, char *q) param++; set_warning: - for (i = 0; i <= ERR_WARN_MAX; i++) - if (!nasm_stricmp(param, warnings[i].name)) - break; - if (i <= ERR_WARN_MAX) - warning_on_global[i] = do_warn; - else if (!nasm_stricmp(param, "all")) - for (i = 1; i <= ERR_WARN_MAX; i++) - warning_on_global[i] = do_warn; - else if (!nasm_stricmp(param, "none")) - for (i = 1; i <= ERR_WARN_MAX; i++) - warning_on_global[i] = !do_warn; - else - nasm_error(ERR_NONFATAL | ERR_NOFILE | ERR_USAGE, - "invalid warning `%s'", param); + for (i = 0; i <= ERR_WARN_MAX; i++) { + if (!nasm_stricmp(param, warnings[i].name)) + break; + } + if (i <= ERR_WARN_MAX) { + warning_on_global[i] = do_warn; + } else if (!nasm_stricmp(param, "all")) { + for (i = 1; i <= ERR_WARN_MAX; i++) + warning_on_global[i] = do_warn; + } else if (!nasm_stricmp(param, "none")) { + for (i = 1; i <= ERR_WARN_MAX; i++) + warning_on_global[i] = !do_warn; + } else { + nasm_error(ERR_NONFATAL | ERR_NOFILE | ERR_USAGE, + "invalid warning `%s'", param); + } break; case 'M': |