Eugene.Zelenko added inline comments.

================
Comment at: clang-tidy/ClangTidyCheck.h:109
+  ///     not supported.
+  virtual llvm::StringRef fixDescription() { return ""; }
+
----------------
return {} could be used instead.


================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:266
+    return I->second.first;
+  return "";
+}
----------------
return {} could be used instead.


================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:274
+    return I->second.second;
   return "";
 }
----------------
return {} could be used instead.


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59932/new/

https://reviews.llvm.org/D59932



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to