Re: [PATCH] Fix code to follow the "Don’t use else after a return" rule.

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Thu Oct 16 08:01:52 PDT 2014


LGTM with a nit.

================
Comment at: lib/ASTMatchers/ASTMatchersInternal.cpp:252
@@ -251,3 +251,3 @@
   const StringRef Pattern = Name;
   if (Pattern.startswith("::")) {
     return FullName == Pattern;
----------------
You can drop the {} too now.

http://reviews.llvm.org/D5826






More information about the cfe-commits mailing list