Remove unnecessary bytes() calls (encode#6626)

jdufresne · rpkilby · commit b37aa284eb4a · 2019-05-30T15:05:27.000-07:00
diff --git a/rest_framework/renderers.py b/rest_framework/renderers.py
@@ -88,7 +88,7 @@ def render(self, data, accepted_media_type=None, renderer_context=None):
         Render `data` into JSON, returning a bytestring.
         """
         if data is None:
-            return bytes()
+            return b''
 
         renderer_context = renderer_context or {}
         indent = self.get_indent(accepted_media_type, renderer_context)
diff --git a/rest_framework/response.py b/rest_framework/response.py
@@ -73,7 +73,7 @@ def rendered_content(self):
                 'renderer returned unicode, and did not specify '
                 'a charset value.'
             )
-            return bytes(ret.encode(charset))
+            return ret.encode(charset)
 
         if not ret:
             del self['Content-Type']
diff --git a/rest_framework/test.py b/rest_framework/test.py
@@ -185,7 +185,7 @@ def _encode_data(self, data, format=None, content_type=None):
 
             # Coerce text to bytes if required.
             if isinstance(ret, str):
-                ret = bytes(ret.encode(renderer.charset))
+                ret = ret.encode(renderer.charset)
 
         return ret, content_type
 

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ def rendered_content(self):`
`73`	`73`	`'renderer returned unicode, and did not specify '`
`74`	`74`	`'a charset value.'`
`75`	`75`	`)`
`76`		`- return bytes(ret.encode(charset))`
	`76`	`+ return ret.encode(charset)`
`77`	`77`
`78`	`78`	`if not ret:`
`79`	`79`	`del self['Content-Type']`