Skip to content

Commit b891389

Browse files
committed
Expand is_clone_like comment
1 parent 3beb6b1 commit b891389

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

clippy_lints/src/methods/implicit_clone.rs

+3
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ pub fn check(cx: &LateContext<'_>, method_name: &str, expr: &hir::Expr<'_>, recv
3232
}
3333

3434
/// Returns true if the named method can be used to clone the receiver.
35+
/// Note that `to_string` is not flagged by `implicit_clone`. So other lints that call
36+
/// `is_clone_like` and that do flag `to_string` must handle it separately. See, e.g.,
37+
/// `is_to_owned_like` in `unnecessary_to_owned.rs`.
3538
pub fn is_clone_like(cx: &LateContext<'_>, method_name: &str, method_def_id: hir::def_id::DefId) -> bool {
3639
match method_name {
3740
"to_os_string" => is_diag_item_method(cx, method_def_id, sym::OsStr),

0 commit comments

Comments
 (0)