owenpan added inline comments.
================ Comment at: lib/Format/UnwrappedLineParser.cpp:1066-1072 + if (Style.isCpp()) { + nextToken(); + if (FormatTok->Tok.is(tok::kw_namespace)) { + parseNamespace(); + return; + } + } ---------------- owenpan wrote: > ``` > if (!Style.isCpp()) > break; > case tok::kw_inline: > nextToken(); > if (FormatTok->Tok.is(tok::kw_namespace)) { > parseNamespace(); > return; > } > ``` I forgot to include LLVM_FALLTHROUGH to suppress the warning: ``` if (!Style.isCpp()) break; LLVM_FALLTHROUGH; case tok::kw_inline: nextToken(); if (FormatTok->Tok.is(tok::kw_namespace)) { parseNamespace(); return; } ``` Repository: rC Clang https://reviews.llvm.org/D51036 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits