aboutsummaryrefslogtreecommitdiff
path: root/src/model/guild
diff options
context:
space:
mode:
authorZeyla Hellyer <[email protected]>2018-07-15 23:32:50 -0700
committerZeyla Hellyer <[email protected]>2018-07-15 23:33:59 -0700
commit9da766976929417c4b8f487f8ec05b6f8b3f43ef (patch)
tree83cfae143ad7c1c4423d5ac35bf71d6e0bc6b882 /src/model/guild
parentSupport multiple prefixes for command-groups (#343) (diff)
downloadserenity-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.rs4
-rw-r--r--src/model/guild/guild_id.rs20
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)
}