First of all, thank you for working on this!

  There is already another patch out there that is doing the same thing: 
http://llvm-reviews.chandlerc.com/D2555. Not quite sure how to move forward. 
I'll ping that patch once more and pick it up if nothing happens. Generally:
  - I think it doesn't really make sense to parse try and cat independently. 
Especially, many coding styles require the "}" on the same line as the 
following "catch".
  - We have a unit testing infrastructure in 
cfe/unittests/Format/FormatTest.cpp, which we prefer over the file-check-based 
tests for various reasons.

http://llvm-reviews.chandlerc.com/D2731
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to