Skip to content

Commit cbd9ca4

Browse files
Harmon758Byron
authored andcommitted
Remove and replace compat.unicode
1 parent b5dd2f0 commit cbd9ca4

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

‎git/cmd.py

+2-4
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@
2525
defenc,
2626
force_bytes,
2727
PY3,
28-
# just to satisfy flake8 on py3
29-
unicode,
3028
safe_decode,
3129
is_posix,
3230
is_win,
@@ -920,15 +918,15 @@ def __unpack_args(cls, arg_list):
920918
if not isinstance(arg_list, (list, tuple)):
921919
# This is just required for unicode conversion, as subprocess can't handle it
922920
# However, in any other case, passing strings (usually utf-8 encoded) is totally fine
923-
if not PY3 and isinstance(arg_list, unicode):
921+
if not PY3 and isinstance(arg_list, str):
924922
return [arg_list.encode(defenc)]
925923
return [str(arg_list)]
926924

927925
outlist = []
928926
for arg in arg_list:
929927
if isinstance(arg_list, (list, tuple)):
930928
outlist.extend(cls.__unpack_args(arg))
931-
elif not PY3 and isinstance(arg_list, unicode):
929+
elif not PY3 and isinstance(arg_list, str):
932930
outlist.append(arg_list.encode(defenc))
933931
# END recursion
934932
else:

‎git/compat.py

+3-5
Original file line numberDiff line numberDiff line change
@@ -32,18 +32,16 @@
3232
defenc = sys.getfilesystemencoding()
3333

3434
if PY3:
35-
unicode = str
3635
binary_type = bytes
3736
else:
3837
if defenc == 'ascii':
3938
defenc = 'utf-8'
40-
unicode = unicode
4139
binary_type = str
4240

4341

4442
def safe_decode(s):
4543
"""Safely decodes a binary string to unicode"""
46-
if isinstance(s, unicode):
44+
if isinstance(s, str):
4745
return s
4846
elif isinstance(s, bytes):
4947
return s.decode(defenc, 'surrogateescape')
@@ -53,7 +51,7 @@ def safe_decode(s):
5351

5452
def safe_encode(s):
5553
"""Safely decodes a binary string to unicode"""
56-
if isinstance(s, unicode):
54+
if isinstance(s, str):
5755
return s.encode(defenc)
5856
elif isinstance(s, bytes):
5957
return s
@@ -63,7 +61,7 @@ def safe_encode(s):
6361

6462
def win_encode(s):
6563
"""Encode unicodes for process arguments on Windows."""
66-
if isinstance(s, unicode):
64+
if isinstance(s, str):
6765
return s.encode(locale.getpreferredencoding(False))
6866
elif isinstance(s, bytes):
6967
return s

0 commit comments

Comments
 (0)