BDS review todos

SVN: 2888

bds/source/java/ch/systemsx/cisd/bds/ExperimentIdentifier.java

@@ -75,6 +75,7 @@ public class ExperimentIdentifier implements IStorable
     /**
      * Returns the group code;
      */
+    // TODO 2007-12-03 Tomasz Pylak review: should not we use the term organization as everywhere else instead of group?
     public final String getGroupCode()
     {
         return groupCode;

bds/source/java/ch/systemsx/cisd/bds/storage/IDirectory.java

@@ -48,6 +48,9 @@ public interface IDirectory extends INode, Iterable<INode>
      * @return the new node. It will be a {@link ILink} if <code>file</code> is a symbolic link, a {@link IDirectory}
      *         if <code>file</code> is a folder, or {@link IFile} if <code>file</code> is a plain file.
      */
+    // TODO 2007-12-03 Tomasz Pylak review: this generic interface should not use java.io.File. Is the 'move' parameter
+    // possible to implement in HDF5? Maybe those operations should be done before, depending on the implementation
+    // which is used?
     public INode addFile(final File file, final boolean move);
 
     /**

bds/source/java/ch/systemsx/cisd/bds/storage/filesystem/File.java

@@ -34,6 +34,7 @@ import ch.systemsx.cisd.common.utilities.FileUtilities;
  * 
  * @author Franz-Josef Elmer
  */
+// TODO 2007-12-03 Tomasz Pylak review: Name of this class should be changed, it's very confusing
 final class File extends AbstractNode implements IFile
 {
     File(final java.io.File file)

bds/source/java/ch/systemsx/cisd/bds/storage/filesystem/ILinkMaker.java

@@ -42,6 +42,7 @@ public interface ILinkMaker
             public final File tryCreateLink(final File file, final File destDir, final String nameOrNull)
                     throws EnvironmentFailureException
             {
+                // TODO 2007-12-03 Tomasz Pylak review: should not we just make a copy here? 
                 return null;
             }
         };