[ZODB-Dev] BTrees and Mutables, was Re: [IndexedCatalog] bug in default indexing

[Nicholas Henke]

On Mon, 03 Mar 2003 16:07:34 -0500
Shane Hathaway <shane@zope.com> wrote:

> Christian Reis wrote:
> > On Mon, Mar 03, 2003 at 02:00:02PM -0500, Nicholas Henke wrote:
> > 
> >>>>+            index[value] = idx
> >>>>+            idx.append(object)
> >>>
> >>>Shouldn't these two lines be reversed?
> >>
> >>Nope -- here is a wuote from the ZODB part of this thread ( the Re:
> >>[ZODB-Dev] BTrees and Mutables). This is from Shane Hathaway, dated
> >>16.21 03/02/11
> >>
> >>That's correct, although the purist way is like this:
> >>
> >>     l = tree['bar']
> >>     tree['bar'] = l
> >>     l.append(4)

Indexes.py, line ~292 or so, still has the append happening before the
re-assignment. This was CVS as of a few minutes ago.

Nic
-- 
Nicholas Henke
Penguin Herder & Linux Cluster System Programmer
Liniac Project - Univ. of Pennsylvania