https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/138757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HerrCai0907 wrote:
> Since :: are specified before std::move, I am not entirely sure whether it
> would be required for the custom function. In my testing it worked both with
> and without ::.
with :: means is a full name including namespace, without :: means only match
function name.
https:
https://github.com/HerrCai0907 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/138757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DimitrijeDobrota updated
https://github.com/llvm/llvm-project/pull/138757
Rate limit ยท GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Ari
@@ -219,6 +219,11 @@ Changes in existing checks
tolerating fix-it breaking compilation when functions is used as pointers
to avoid matching usage of functions within the current compilation unit.
+- Improved :doc:`cppcoreguidelines-rvalue-reference-param-not-moved