ahatanak accepted this revision.
ahatanak added a comment.

LGTM



================
Comment at: lib/Sema/SemaDeclAttr.cpp:2295
+      MissingIntroduced =
+          MissingIntroduced ? Decl->getIntroduced().empty() : false;
+      MissingDeprecated =
----------------
I feel like using "if" is easier to understand than a conditional operator, but 
it's up to you:

```
if (MissingIntroduced)
  MissingIntroduced = Decl->getIntroduced().empty();
```


Repository:
  rL LLVM

https://reviews.llvm.org/D39913



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D39913: [ObjC] warn... Akira Hatanaka via Phabricator via cfe-commits

Reply via email to