Absolutely correct. I didn't like making indexes object specific to begin with, and hence must have gotten ahead of myself. I think the right way to do this would put the indexes into the class namespace, but I was lazy.

UPDATE: In fact, the same argument means that the accessor/mutator creation won't work either because new objects will clobber the old closures. I think the solution is to make everything package scoped, but clearly I was, in fact, over-simplifying.