-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
1. core.rs 173 行,to_owned() may be inefficient。这里其实不需要 to_owned,所有权拿走就行。 2. login.rs 117 行,to_owned() may be inefficient, 这里建议用 clone_into(),因为私有的 clone_into() 可能可以避免拷贝。但这里其实是一样的,而且 clone_into 接口使用麻烦,但为了抑制 clippy 建议,这里直接用 clone() 了。 3. post.rs 858 行,建议同上。但这里仍没有区别,而且在这里对 JsonValue 的字段使用 clone_into 在这里非常麻烦,需要做相当多的转换,直接用 clone() 抑制 clippy 的 warn 建议。
- Loading branch information
Showing
3 changed files
with
3 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters