Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit b2c46f7

Browse filesBrowse files
authored
Merge pull request #1788 from EliahKagan/void-finalizer
In handle_process_output don't forward finalizer result
2 parents 597e1a6 + 7057b9b commit b2c46f7
Copy full SHA for b2c46f7

File tree

1 file changed

+1
-4
lines changed
Filter options

1 file changed

+1
-4
lines changed

‎git/cmd.py

Copy file name to clipboardExpand all lines: git/cmd.py
+1-4Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,6 @@ def handle_process_output(
111111
112112
This function returns once the finalizer returns.
113113
114-
:return: Result of finalizer
115114
:param process: :class:`subprocess.Popen` instance
116115
:param stdout_handler: f(stdout_line_string), or None
117116
:param stderr_handler: f(stderr_line_string), or None
@@ -205,9 +204,7 @@ def pump_stream(
205204
stderr_handler(error_str) # type: ignore
206205

207206
if finalizer:
208-
return finalizer(process)
209-
else:
210-
return None
207+
finalizer(process)
211208

212209

213210
def dashify(string: str) -> str:

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.