fix doc

Author: HerrCai0907

clang-tools-extra/clang-tidy/bugprone/ReturnConstRefFromParameterCheck.cpp

@@ -15,13 +15,6 @@ using namespace clang::ast_matchers;
 
 namespace clang::tidy::bugprone {
 
-std::optional<TraversalKind>
-ReturnConstRefFromParameterCheck::getCheckTraversalKind() const {
-  // Use 'AsIs' to make sure the return type is exactly the same as the
-  // parameter type.
-  return TK_AsIs;
-}
-
 void ReturnConstRefFromParameterCheck::registerMatchers(MatchFinder *Finder) {
   Finder->addMatcher(
       returnStmt(hasReturnValue(declRefExpr(to(parmVarDecl(hasType(

clang-tools-extra/clang-tidy/bugprone/ReturnConstRefFromParameterCheck.h

@@ -13,8 +13,9 @@
 
 namespace clang::tidy::bugprone {
 
-/// Detects the function which returns the const reference from parameter which
-/// causes potential use after free if the caller uses xvalue as argument.
+/// Detects the function which returns the const reference parameter as const
+/// reference. This might causes potential use after free errors if the caller
+/// uses xvalue as arguments.
 ///
 /// For the user-facing documentation see:
 /// http://clang.llvm.org/extra/clang-tidy/checks/bugprone/return-const-ref-from-parameter.html
@@ -24,7 +25,11 @@ class ReturnConstRefFromParameterCheck : public ClangTidyCheck {
       : ClangTidyCheck(Name, Context) {}
   void registerMatchers(ast_matchers::MatchFinder *Finder) override;
   void check(const ast_matchers::MatchFinder::MatchResult &Result) override;
-  std::optional<TraversalKind> getCheckTraversalKind() const override;
+  std::optional<TraversalKind> getCheckTraversalKind() const override {
+    // Use 'AsIs' to make sure the return type is exactly the same as the
+    // parameter type.
+    return TK_AsIs;
+  }
   bool isLanguageVersionSupported(const LangOptions &LangOpts) const override {
     return LangOpts.CPlusPlus;
   }

clang-tools-extra/docs/clang-tidy/checks/bugprone/return-const-ref-from-parameter.rst

@@ -3,11 +3,12 @@
 bugprone-return-const-ref-from-parameter
 ========================================
 
-Detects the function which returns the const reference from parameter which
-causes potential use after free if the caller uses xvalue as argument.
+Detects the function which returns the const reference parameter as const
+reference. This might causes potential use after free errors if the caller
+uses xvalue as arguments.
 
-In c++, const reference parameter can accept xvalue which will be destructed
-after the call. When the function returns this parameter also as const reference,
+In C++, const reference parameters can accept xvalues which will be destructed
+after the call. When the function returns such a parameter also as const reference,
 then the returned reference can be used after the object it refers to has been
 destroyed.