aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Analysis/FormatString.cpp
diff options
context:
space:
mode:
authorRichard Trieu <rtrieu@google.com>2016-02-18 22:34:54 +0000
committerRichard Trieu <rtrieu@google.com>2016-02-18 22:34:54 +0000
commitcc3949d99af2b2ea2f31c1694eeef7cea0f484f0 (patch)
tree7b6a787cb2352bf0f9ced314eaff8aad71618459 /clang/lib/Analysis/FormatString.cpp
parent7a08381403b54cd8998f3c922f18b65867e3c07c (diff)
downloadllvm-cc3949d99af2b2ea2f31c1694eeef7cea0f484f0.zip
llvm-cc3949d99af2b2ea2f31c1694eeef7cea0f484f0.tar.gz
llvm-cc3949d99af2b2ea2f31c1694eeef7cea0f484f0.tar.bz2
Remove use of builtin comma operator.
Cleanup for upcoming Clang warning -Wcomma. No functionality change intended. llvm-svn: 261271
Diffstat (limited to 'clang/lib/Analysis/FormatString.cpp')
-rw-r--r--clang/lib/Analysis/FormatString.cpp16
1 files changed, 12 insertions, 4 deletions
diff --git a/clang/lib/Analysis/FormatString.cpp b/clang/lib/Analysis/FormatString.cpp
index 0948bc0..1c42ec0 100644
--- a/clang/lib/Analysis/FormatString.cpp
+++ b/clang/lib/Analysis/FormatString.cpp
@@ -190,13 +190,21 @@ clang::analyze_format_string::ParseLengthModifier(FormatSpecifier &FS,
return false;
case 'h':
++I;
- lmKind = (I != E && *I == 'h') ? (++I, LengthModifier::AsChar)
- : LengthModifier::AsShort;
+ if (I != E && *I == 'h') {
+ ++I;
+ lmKind = LengthModifier::AsChar;
+ } else {
+ lmKind = LengthModifier::AsShort;
+ }
break;
case 'l':
++I;
- lmKind = (I != E && *I == 'l') ? (++I, LengthModifier::AsLongLong)
- : LengthModifier::AsLong;
+ if (I != E && *I == 'l') {
+ ++I;
+ lmKind = LengthModifier::AsLongLong;
+ } else {
+ lmKind = LengthModifier::AsLong;
+ }
break;
case 'j': lmKind = LengthModifier::AsIntMax; ++I; break;
case 'z': lmKind = LengthModifier::AsSizeT; ++I; break;