diff -u -r1.7 -r1.8
--- ConditionsConstants.java	15 Oct 2013 23:14:47 -0000	1.7
+++ ConditionsConstants.java	15 Oct 2013 23:24:47 -0000	1.8
@@ -5,6 +5,7 @@

  */
 public final class ConditionsConstants {
     

+    /** Prevent instantiation of this class, which is only a holder of static constant values. */

     private ConditionsConstants() {}
 
     /** Table with ECal channel data. */

@@ -37,6 +38,6 @@

     /** Conditions key for SVT gain data. */
     public static final String SVT_GAINS = "svt_gains";
     

-    /** Conditions key fo SVT time shifts by sensor. */

+    /** Conditions key for SVT time shifts by sensor. */

     public static final String SVT_TIME_SHIFTS = "svt_time_shifts";
 }

diff -u -r1.4 -r1.5
--- ConditionsRecord.java	11 Oct 2013 00:06:36 -0000	1.4
+++ ConditionsRecord.java	15 Oct 2013 23:24:47 -0000	1.5
@@ -1,7 +1,6 @@

 package org.lcsim.hps.conditions;
 
 import java.sql.Blob;

-import java.sql.Connection;

 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.Date;

diff -u -r1.6 -r1.7
--- DatabaseConditionsConverter.java	4 Oct 2013 06:02:19 -0000	1.6
+++ DatabaseConditionsConverter.java	15 Oct 2013 23:24:47 -0000	1.7
@@ -1,7 +1,5 @@

 package org.lcsim.hps.conditions;
 

-import java.sql.Connection;
-

 import org.lcsim.conditions.ConditionsConverter;
 
 /**

diff -u -r1.4 -r1.5
--- EcalBadChannelCollection.java	15 Oct 2013 23:14:47 -0000	1.4
+++ EcalBadChannelCollection.java	15 Oct 2013 23:24:47 -0000	1.5
@@ -2,5 +2,8 @@

 
 import org.lcsim.hps.conditions.ChannelCollection;
 

+/**
+ * A collection of bad channel IDs in the ECAL.
+ */

 public class EcalBadChannelCollection extends ChannelCollection {
 }

diff -u -r1.4 -r1.5
--- EcalChannelMap.java	4 Oct 2013 06:02:19 -0000	1.4
+++ EcalChannelMap.java	15 Oct 2013 23:24:47 -0000	1.5
@@ -6,13 +6,12 @@

  * This class maps ID values from the database to detailed ECal channel information.
  * There should really only be one of these data structures per job, as the EcalChannel 
  * objects are used as unique identifiers in the {@link EcalConditions} class.

- * 
- * FIXME: When adding new channels, there should be data structures setup for lookup using
- *        using x, y or crate, slot, channel.  See Sho's bit packing routines in original
- *        EcalConditions class for ideas on how to make keys for them.

  */
 public class EcalChannelMap extends HashMap<Integer, EcalChannel> {
     

+    /**
+     * Class constructor.
+     */

     EcalChannelMap() {        
     }
     

@@ -22,8 +21,6 @@

      * @param slot The slot number.
      * @param channelNumber The channel number.
      * @return The matching channel or null if does not exist.

-     * 
-     * FIXME: Improve performance of this method from O(N). 

      */
     public EcalChannel find(int crate, int slot, int channelNumber) {
         for (EcalChannel channel : values()) {

diff -u -r1.3 -r1.4
--- EcalChannelMapConverter.java	11 Oct 2013 00:05:36 -0000	1.3
+++ EcalChannelMapConverter.java	15 Oct 2013 23:24:47 -0000	1.4
@@ -2,7 +2,6 @@

 
 import static org.lcsim.hps.conditions.ConditionsConstants.ECAL_CHANNELS;
 

-import java.sql.Connection;

 import java.sql.ResultSet;
 import java.sql.SQLException;
 

diff -u -r1.1 -r1.2
--- EcalConditionsConverter.java	9 Oct 2013 21:00:30 -0000	1.1
+++ EcalConditionsConverter.java	15 Oct 2013 23:24:47 -0000	1.2
@@ -19,7 +19,7 @@

 public class EcalConditionsConverter extends DatabaseConditionsConverter<EcalConditions> {
        
     /**

-     * Load ECal conditions into the <code>conditions</code> instance variable.

+     * Create ECAL conditions object containing all data for the current run.

      */
     public EcalConditions getData(ConditionsManager manager, String name) {
         

diff -u -r1.6 -r1.7
--- EcalGainConverter.java	11 Oct 2013 00:05:36 -0000	1.6
+++ EcalGainConverter.java	15 Oct 2013 23:24:47 -0000	1.7
@@ -1,6 +1,5 @@

 package org.lcsim.hps.conditions.ecal;
 

-import java.sql.Connection;

 import java.sql.ResultSet;
 import java.sql.SQLException;
 

diff -u -r1.4 -r1.5
--- SvtChannelMapConverter.java	11 Oct 2013 00:07:24 -0000	1.4
+++ SvtChannelMapConverter.java	15 Oct 2013 23:24:48 -0000	1.5
@@ -2,7 +2,6 @@

 
 import static org.lcsim.hps.conditions.ConditionsConstants.SVT_CHANNELS;
 

-import java.sql.Connection;

 import java.sql.ResultSet;
 import java.sql.SQLException;
 

@@ -17,7 +16,7 @@

 public class SvtChannelMapConverter extends DatabaseConditionsConverter<SvtChannelMap> {
 
     /**

-     * Create the objects from the database.

+     * Create the channel map from the conditions database.

      * @param manager The current conditions manager.
      * @param name The name of the conditions set.
      */