|
From: | Benja Fallenstein |
Subject: | Re: [Gzz-commits] gzz ./TODO Documentation/Manuscripts/Paper/pape... |
Date: | Sun, 20 Oct 2002 23:16:21 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1) Gecko/20020913 Debian/1.1-1 |
Hi Tuomas! Tuomas J. Lukka wrote:
Index: gzz/gzz/impl/SimpleVStreamTexter.java+ + /** XXX WRONG IMPLEMENTATION!!!!!!!+ */ public XuIndexer getXuIndexer() { return xuIndex; }Index: gzz/gzz/index/IndexManager.java + /** Get the Xanadu model link indexer which is to be used with + * the given space.+ * <b>Note: </b> This does NOT mean an indexer which is implemented BY the + * space, this is an indexer implemented somewhere else FOR the space. + */ public static XuIndexer getXuIndexer(Space s) { return ((IndexedVStreamCellTexter)s.getCellTexter()).getXuIndexer();
Actually, with the semantics you want I think IndexManager is wrong. Having the Space object know about the other space that implements the xu links for this space seems strange. I'd prefer if the IndexManager knew about the associations, but the Space didn't.
- Benja
[Prev in Thread] | Current Thread | [Next in Thread] |