diff options
| author | Zeyla Hellyer <[email protected]> | 2018-07-15 23:32:50 -0700 |
|---|---|---|
| committer | Zeyla Hellyer <[email protected]> | 2018-07-15 23:33:59 -0700 |
| commit | 9da766976929417c4b8f487f8ec05b6f8b3f43ef (patch) | |
| tree | 83cfae143ad7c1c4423d5ac35bf71d6e0bc6b882 /src/model/guild | |
| parent | Support multiple prefixes for command-groups (#343) (diff) | |
| download | serenity-9da766976929417c4b8f487f8ec05b6f8b3f43ef.tar.xz serenity-9da766976929417c4b8f487f8ec05b6f8b3f43ef.zip | |
Fix some clippy lints
Some lints were not resolved due to causing API changes. Most lints in the
framework were left unfixed.
Diffstat (limited to 'src/model/guild')
| -rw-r--r-- | src/model/guild/audit_log.rs | 4 | ||||
| -rw-r--r-- | src/model/guild/guild_id.rs | 20 |
2 files changed, 12 insertions, 12 deletions
diff --git a/src/model/guild/audit_log.rs b/src/model/guild/audit_log.rs index 1c24d98..e4a344c 100644 --- a/src/model/guild/audit_log.rs +++ b/src/model/guild/audit_log.rs @@ -277,7 +277,7 @@ mod u64_handler { } fn visit_str<E: de::Error>(self, string: &str) -> StdResult<u64, E> { - string.parse().map_err(|e| de::Error::custom(e)) + string.parse().map_err(de::Error::custom) } } @@ -315,7 +315,7 @@ mod option_u64_handler { } fn visit_str<E: de::Error>(self, string: &str) -> StdResult<Option<u64>, E> { - string.parse().map(Some).map_err(|e| de::Error::custom(e)) + string.parse().map(Some).map_err(de::Error::custom) } } diff --git a/src/model/guild/guild_id.rs b/src/model/guild/guild_id.rs index a209369..9c3120a 100644 --- a/src/model/guild/guild_id.rs +++ b/src/model/guild/guild_id.rs @@ -54,7 +54,7 @@ impl GuildId { self._ban(user.into(), (ban_options.dmd(), ban_options.reason())) } - fn _ban(&self, user: UserId, ban_options: (u8, &str)) -> Result<()> { + fn _ban(self, user: UserId, ban_options: (u8, &str)) -> Result<()> { let (dmd, reason) = ban_options; if dmd > 7 { @@ -124,7 +124,7 @@ impl GuildId { } fn _create_channel( - &self, + self, name: &str, kind: ChannelType, category: Option<ChannelId>, @@ -177,7 +177,7 @@ impl GuildId { } fn _create_integration( - &self, + self, integration_id: IntegrationId, kind: &str, ) -> Result<()> { @@ -232,7 +232,7 @@ impl GuildId { self._delete_emoji(emoji_id.into()) } - fn _delete_emoji(&self, emoji_id: EmojiId) -> Result<()> { + fn _delete_emoji(self, emoji_id: EmojiId) -> Result<()> { http::delete_emoji(self.0, emoji_id.0) } @@ -246,7 +246,7 @@ impl GuildId { self._delete_integration(integration_id.into()) } - fn _delete_integration(&self, integration_id: IntegrationId) -> Result<()> { + fn _delete_integration(self, integration_id: IntegrationId) -> Result<()> { http::delete_guild_integration(self.0, integration_id.0) } @@ -265,7 +265,7 @@ impl GuildId { self._delete_role(role_id.into()) } - fn _delete_role(&self, role_id: RoleId) -> Result<()> { + fn _delete_role(self, role_id: RoleId) -> Result<()> { http::delete_role(self.0, role_id.0) } @@ -300,7 +300,7 @@ impl GuildId { self._edit_emoji(emoji_id.into(), name) } - fn _edit_emoji(&self, emoji_id: EmojiId, name: &str) -> Result<Emoji> { + fn _edit_emoji(self, emoji_id: EmojiId, name: &str) -> Result<Emoji> { let map = json!({ "name": name, }); @@ -327,7 +327,7 @@ impl GuildId { self._edit_member(user_id.into(), f) } - fn _edit_member<F>(&self, user_id: UserId, f: F) -> Result<()> + fn _edit_member<F>(self, user_id: UserId, f: F) -> Result<()> where F: FnOnce(EditMember) -> EditMember { let map = utils::vecmap_to_json_map(f(EditMember::default()).0); @@ -368,7 +368,7 @@ impl GuildId { self._edit_role(role_id.into(), f) } - fn _edit_role<F>(&self, role_id: RoleId, f: F) -> Result<Role> + fn _edit_role<F>(self, role_id: RoleId, f: F) -> Result<Role> where F: FnOnce(EditRole) -> EditRole { let map = utils::vecmap_to_json_map(f(EditRole::default()).0); @@ -622,7 +622,7 @@ impl GuildId { self._unban(user_id.into()) } - fn _unban(&self, user_id: UserId) -> Result<()> { + fn _unban(self, user_id: UserId) -> Result<()> { http::remove_ban(self.0, user_id.0) } |