From b80be68e088f56f5b2ab5371b92a478a14937e8b Mon Sep 17 00:00:00 2001 From: Aymeric Augustin Date: Fri, 6 Sep 2013 10:34:32 -0500 Subject: [PATCH] Fixed #21052 -- Small performance optimization. --- django/utils/encoding.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/django/utils/encoding.py b/django/utils/encoding.py index 470005bb46..667c3def45 100644 --- a/django/utils/encoding.py +++ b/django/utils/encoding.py @@ -60,8 +60,7 @@ def force_text(s, encoding='utf-8', strings_only=False, errors='strict'): If strings_only is True, don't convert (some) non-string-like objects. """ - # Handle the common case first, saves 30-40% when s is an instance of - # six.text_type. This function gets called often in that setting. + # Handle the common case first for performance reasons. if isinstance(s, six.text_type): return s if strings_only and is_protected_type(s): @@ -115,8 +114,7 @@ def force_bytes(s, encoding='utf-8', strings_only=False, errors='strict'): If strings_only is True, don't convert (some) non-string-like objects. """ - if isinstance(s, six.memoryview): - s = bytes(s) + # Handle the common case first for performance reasons. if isinstance(s, bytes): if encoding == 'utf-8': return s @@ -124,6 +122,8 @@ def force_bytes(s, encoding='utf-8', strings_only=False, errors='strict'): return s.decode('utf-8', errors).encode(encoding, errors) if strings_only and is_protected_type(s): return s + if isinstance(s, six.memoryview): + return bytes(s) if isinstance(s, Promise): return six.text_type(s).encode(encoding, errors) if not isinstance(s, six.string_types):