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

Reply via email to