[ZODB-Dev] transaction "Attempts" class

[Jim Fulton]

On Thu, Mar 29, 2012 at 2:50 AM, Chris McDonough <chrism at plope.com> wrote:
> On Wed, 2012-03-28 at 18:37 -0400, Chris McDonough wrote:
>> On Wed, 2012-03-28 at 17:06 -0400, Jim Fulton wrote:
>> > On Wed, Mar 28, 2012 at 4:37 PM, Chris McDonough <chrism at plope.com> wrote:
>> > > On Wed, 2012-03-28 at 14:21 -0400, Jim Fulton wrote:

...

>> > > In the meantime, I think the existing attempts context manager still
>> > > needs the small fix I proposed in my original message. Â Can you confirm
>> > > that my understanding of the its intent seems roughly correct?
>> >
>> > I didn't see a statement of intent.  I think your fix us good,
>> > but the code (that I wrote and your fix) makes my head hurt. :)
>> >
>> > I'd say, make some test cases for the bug and make it pass.
>
> https://mail.zope.org/pipermail/checkins/2012-March/059068.html
>
> Look reasonable?  If so I'll go ahead and merge it and base the
> decorator code on it.

+1

Jim

-- 
Jim Fulton
http://www.linkedin.com/in/jimfulton