Code review comment for lp:~lifeless/bzr/lsprof_lockout

Revision history for this message
Martin Packman (gz) wrote :

The added locking makes sense to me, and tests pass locally (once an unrelated regression was removed). There's still the issue of bug 579185 before threaded profiling is actually much good for anything, but this is a clear prerequisite.

review: Approve

« Back to merge proposal