[Zodb-checkins] CVS: Zope3/src/persistence - cache.py:1.4

Jeremy Hylton jeremy@zope.com
11 Mar 2003 17:10:11 -0500


On Tue, 2003-03-11 at 16:55, Tim Peters wrote:
> [Jeremy Hylton]
> > +            # The _p_atime field is seconds since the start of the day.
> > +            # When we start a new day, we'll expect to see most of the
> > +            # _p_atime values be less than now.
> 
> We actually expect them to be greater than now, right?  At the start of a
> new day, now will be close to 0.

Right.  The comment should say greater.

> Also wonder if this is still sorting in the intended order (not obvious).

No it wasn't in the right order.  Added a comment explaining the order
it should be in.  How is it that almost every time I call sort() I also
have to call reverse()?

Thanks,
Jeremy