diff options
| author |  Kurtis Rader <krader@skepticism.us> | 2016-05-03 16:23:30 -0700 |
|---|---|---|
| committer | Kurtis Rader <krader@skepticism.us> | 2016-05-03 17:14:56 -0700 |
| commit | fc44cffac50e6096528f2bde456d91a4e40ea7c9 (patch) | |
| tree | 3c6c928aa0a8568b6e37a0698deb464575ba3fe2 /src/builtin_printf.cpp | |
| parent | 5c8763be0e68bbdec3fdd1edb5754f4c421098e1 (diff) | |
restyle switch blocks to match project style
I missed restyling a few "switch" blocks to make them consistent with the rest
of the code base. This fixes that oversight. This should be the final step in
restyling the C++ code to have a consistent style. This also includes a few
trivial cleanups elsewhere.
I also missed restyling the "complete" module when working my way from a to z
so this final change includes restyling that module.
Total lint errors decreased 36%. Cppcheck errors went from 47 to 24. Oclint P2
errors went from 819 to 778. Oclint P3 errors went from 3252 to 1842.
Resolves #2902.
Diffstat (limited to 'src/builtin_printf.cpp')
| -rw-r--r-- | src/builtin_printf.cpp | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/src/builtin_printf.cpp b/src/builtin_printf.cpp index a3df00d3..a755d27a 100644 --- a/src/builtin_printf.cpp +++ b/src/builtin_printf.cpp @@ -596,21 +596,24 @@ int builtin_printf_state_t::print_formatted(const wchar_t *format, int argc, wch for (;; f++, direc_length++) { switch (*f) { case L'I': - case L'\'': + case L'\'': { modify_allowed_format_specifiers(ok, "aAceEosxX", false); break; + } case '-': case '+': - case ' ': + case ' ': { break; - case L'#': + } + case L'#': { modify_allowed_format_specifiers(ok, "cdisu", false); break; - case '0': + } + case '0': { modify_allowed_format_specifiers(ok, "cs", false); break; - default: - goto no_more_flag_characters; + } + default: { goto no_more_flag_characters; } } } no_more_flag_characters:; |