vsapsai added inline comments.
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:7207-7217
+ } else {
+ FixIts.push_back(FixItHint::CreateInsertion(
+ SelectorLocs[I], SelectorSlotNames[I]));
+ }
+ }
+ } else {
+ FixIts.push_back(FixItHint::CreateReplacement(UseRange,
Replacement));
----------------
aaron.ballman wrote:
> You can elide a lot of braces here.
Just to clarify, is it OK if if-branch is with braces but else-branch without?
I.e.
```
if (...) {
line 1
line 2
} else
the only line
```
https://reviews.llvm.org/D44589
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
