aaron.ballman added inline comments.

================
Comment at: test/clang-tidy/misc-enum-misuse.cpp:3
@@ +2,3 @@
+
+enum Empty {
+};
----------------
szepet wrote:
> Could you specify which part of the file seems off? I have run the clang 
> format with the same options on testfiles as on the others.
```
enum A { A = 1,
         B = 2,
         C = 4,
         D = 8,
         E = 16,
         F = 32,
         G = 63
};
```
should be:
```
enum A {
  A = 1,
  B = 2,
  C = 4,
  D = 8,
  E = 16,
  F = 32,
  G = 63
};
```
is what I was thinking was incorrect, but perhaps clang-format allows such 
constructs?

================
Comment at: test/clang-tidy/misc-enum-misuse.cpp:66
@@ +65,3 @@
+    int k = T | Q;
+  return 0;
+  Empty EmptyVal;
----------------
Do you intend to have the `return 0;` here?


https://reviews.llvm.org/D22507



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to