mprobst accepted this revision. mprobst added inline comments. This revision is now accepted and ready to land.
================ Comment at: unittests/Format/FormatTestJS.cpp:371 +TEST_F(FormatTestJS, IIFE) { + verifyFormat("(function() {\n" + "var a = 1;\n" ---------------- danbeam wrote: > mprobst wrote: > > consider adding a test with comments between the tokens (which should work > > with `startsSequence`). > not done yet, but seems uncommon. > > do you want me to add a FIXME? I think that's OK the way it is. https://reviews.llvm.org/D32989 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits