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

Issue 8367044: mgo: fix possible race

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 12 months ago by rog
Modified:
10 years, 12 months ago
Reviewers:
niemeyer, mp+157374
Visibility:
Public.

Description

mgo: fix possible race https://code.launchpad.net/~rogpeppe/mgo/eliminate-acquire-race/+merge/157374 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : mgo: fix possible race #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -4 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M session.go View 1 chunk +6 lines, -4 lines 0 comments Download

Messages

Total messages: 3
rog
Please take a look.
10 years, 12 months ago (2013-04-05 14:07:39 UTC) #1
niemeyer
Thanks, but I already have a local change for this after we talked yesterday. I ...
10 years, 12 months ago (2013-04-05 14:18:48 UTC) #2
rog
10 years, 12 months ago (2013-04-05 14:29:59 UTC) #3
ok, i thought you might, but proposed just in case.

On 5 April 2013 15:18,  <n13m3y3r@gmail.com> wrote:
> Thanks, but I already have a local change for this after we talked
> yesterday. I haven't pushed yet because it's shelved in the middle of
> other major changes that I'm hoping to release today or over the
> weekend.
>
> https://codereview.appspot.com/8367044/
Sign in to reply to this message.

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