2008/6/5 Karl Hasselström <kha@xxxxxxxxxxx>: > On 2008-06-05 13:03:55 +0100, Catalin Marinas wrote: >> 2008/6/5 Karl Hasselström <kha@xxxxxxxxxxx>: >> The same goes for the "create" function, the repository shouldn't >> need to know the directory or files structure that a Stack or Branch >> are using. I think we get a clearer separation this way. > > Yes, you're right. Knowledge of internals belong in the class itself. > > We'll have to be watchful against attempts to create objects other > than via the officially designated factories, though. Python doesn't > really have any mechanisms that help us here. Doesn't need to be enforced. The Stack and Branch objects can be initialised by themselves provided that the repository is passed as argument at construction. The factory is more for convenience, passing the repository to the newly created object. If you really want enforcing this, it can probably be done by hacking __new__ and __init__. -- Catalin -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html