[PATCH] D25587: Introduce llvm FormatVariadic

[Bob Haarman via llvm-commits]

inglorion accepted this revision.
inglorion added a comment.
This revision is now accepted and ready to land.

Thanks! One last inline nit, then LGTM.



================
Comment at: include/llvm/Support/FormatCommon.h:57
+        }
+        default:
+          S.indent(PadAmount);
----------------
You can now also replace this with case AlignStyle::Left: as suggested in http://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations


https://reviews.llvm.org/D25587