gzz-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gzz-commits] gzz ./TODO Documentation/Manuscripts/Paper/pape...


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







reply via email to

[Prev in Thread] Current Thread [Next in Thread]