apply review. fix docs.

Author: toyama0919

src/sagemaker/local/data.py

@@ -224,39 +224,34 @@ class NoneSplitter(Splitter):
     # non-utf8 charactors.
     _textchars = bytearray({7, 8, 9, 10, 12, 13, 27} | set(range(0x20, 0x100)) - {0x7F})
 
-    def split(self, file):
+    def split(self, filename):
         """Split a file into records using a specific strategy.
 
         For this NoneSplitter there is no actual split happening and the file
         is returned as a whole.
 
         Args:
-            file (str): path to the file to split
+            filename (str): path to the file to split
 
         Returns: generator for the individual records that were split from
         the file
         """
-        with open(file, "rb") as f:
+        with open(filename, "rb") as f:
             buf = f.read()
             if not self._is_binary(buf):
                 buf = buf.decode()
             yield buf
 
     def _is_binary(self, buf):
         """binary check.
-
-        binary or text check.
-
-        binary charactor is below.(non-utf8)
-        0x07-0x0a, 0x0c-0x0d, 0x20-0x7e, 0x80-0xFF
-
-        if there are include, it's binary judgment.
+        Check whether `buf` contains binary data.
+        Returns true if `buf` contains any non-utf-8 characters.
 
         Args:
-            buf (bytes): bytes in target file.
+                    buf (bytes): data to inspect
 
         Returns:
-            is binary(True) or text(False).
+                   True if data is binary, otherwise False
         """
         return bool(buf.translate(None, self._textchars))