From b6794fa6d1791c842ba977d8a85e229047324ad5 Mon Sep 17 00:00:00 2001 From: Rapptz Date: Tue, 5 Sep 2023 03:07:25 -0400 Subject: [PATCH] Slightly improve performance of constructing Permissions using kwargs --- discord/permissions.py | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/discord/permissions.py b/discord/permissions.py index f363d1328..8cb59879b 100644 --- a/discord/permissions.py +++ b/discord/permissions.py @@ -141,9 +141,12 @@ class Permissions(BaseFlags): self.value = permissions for key, value in kwargs.items(): - if key not in self.VALID_FLAGS: - raise TypeError(f'{key!r} is not a valid permission name.') - setattr(self, key, value) + try: + flag = self.VALID_FLAGS[key] + except KeyError: + raise TypeError(f'{key!r} is not a valid permission name.') from None + else: + self._set_flag(flag, value) def is_subset(self, other: Permissions) -> bool: """Returns ``True`` if self has the same or fewer permissions as other.""" @@ -360,8 +363,9 @@ class Permissions(BaseFlags): A list of key/value pairs to bulk update permissions with. """ for key, value in kwargs.items(): - if key in self.VALID_FLAGS: - setattr(self, key, value) + flag = self.VALID_FLAGS.get(key) + if flag is not None: + self._set_flag(flag, value) def handle_overwrite(self, allow: int, deny: int) -> None: # Basically this is what's happening here.