Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Issue 6806009: Set returncode to subprocess2.TIMED_OUT instead of -9 when a process times out. (Closed)

Created:
9 years, 8 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, Dirk Pranke, M-A Ruel
Visibility:
Public.

Description

Set returncode to subprocess2.TIMED_OUT instead of -9 when a process times out. R=dpranke@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=80775

Patch Set 1 #

Total comments: 2

Patch Set 2 : Rephrased docstring #

Unified diffs Side-by-side diffs Delta from patch set Stats (+62 lines, -36 lines) Patch
M subprocess2.py View 1 6 chunks +61 lines, -35 lines 0 comments Download
M tests/subprocess2_test.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
9 years, 8 months ago (2011-04-06 13:46:58 UTC) #1
M-A Ruel
(ping)
9 years, 8 months ago (2011-04-07 01:53:42 UTC) #2
Dirk Pranke
9 years, 8 months ago (2011-04-07 02:03:46 UTC) #3
LGTM with nits ...

http://codereview.chromium.org/6806009/diff/1/subprocess2.py
File subprocess2.py (right):

http://codereview.chromium.org/6806009/diff/1/subprocess2.py#newcode26
subprocess2.py:26: TIMED_OUT = -2001
Where did -2001 come from?

http://codereview.chromium.org/6806009/diff/1/subprocess2.py#newcode200
subprocess2.py:200: - The process will be kill with error code TIMED_OUT after
|timeout| seconds
You're not actually killing with an error code; I would rephrase this as:

"- The process will be killed after |timeout| seconds and returncode set to
TIMED_OUT"

Powered by Google App Engine
This is Rietveld 408576698