curdeius accepted this revision.
curdeius added a comment.
I'd like to see a test with braces inside the try (intoducing scope), just to
verify it doesn't break.
E.g.:
verifyFormat("class A {\n"
" int a;\n"
" A() try : a(0), b{1}, c{2} {\n"
" { // New scope.\n"
" }\n"
" } catch (...) {\n"
" throw;\n"
" }\n"
"};\n");
Otherwise LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93296/new/
https://reviews.llvm.org/D93296
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
