Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1406)

Issue 96125: Fix import_lock+fork() deadlock. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 1 month ago by Thomas Wouters (gmail)
Modified:
9 years, 6 months ago
Reviewers:
gregory.p.smith
Base URL:
http://svn.python.org/view/*checkout*/python/branches/release26-maint/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Added documentation. #

Patch Set 3 : Add error checks, revert AIX change. #

Total comments: 2

Patch Set 4 : Doc glitch fix. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+104 lines, -16 lines) Patch
M Doc/c-api/init.rst View 2 3 1 chunk +16 lines, -0 lines 0 comments Download
M Include/import.h View 1 1 chunk +8 lines, -0 lines 0 comments Download
M Lib/test/test_fork1.py View 1 2 chunks +41 lines, -0 lines 0 comments Download
M Modules/posixmodule.c View 1 2 3 chunks +31 lines, -3 lines 0 comments Download
M Python/import.c View 1 2 6 chunks +8 lines, -13 lines 0 comments Download

Messages

Total messages: 4
gregory.p.smith
LGTM
10 years, 1 month ago (2009-07-25 17:12:56 UTC) #1
gregory.p.smith
http://codereview.appspot.com/96125/diff/2008/3003 File Doc/c-api/init.rst (right): http://codereview.appspot.com/96125/diff/2008/3003#newcode534 Line 534: of Python's internal locks behind held by a ...
9 years, 11 months ago (2009-09-09 18:24:56 UTC) #2
Thomas Wouters (gmail)
http://codereview.appspot.com/96125/diff/2008/3003 File Doc/c-api/init.rst (right): http://codereview.appspot.com/96125/diff/2008/3003#newcode534 Line 534: of Python's internal locks behind held by a ...
9 years, 11 months ago (2009-09-09 18:37:28 UTC) #3
gregory.p.smith
9 years, 11 months ago (2009-09-09 18:43:07 UTC) #4
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b