Author: kono
Date: 2010-09-17 18:34:18 -0700 (Fri, 17 Sep 2010)
New Revision: 21942

Modified:
   
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/AbstractVisualMappingFunction.java
Log:
All fields are immutable.

Modified: 
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/AbstractVisualMappingFunction.java
===================================================================
--- 
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/AbstractVisualMappingFunction.java
  2010-09-18 01:30:48 UTC (rev 21941)
+++ 
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/AbstractVisualMappingFunction.java
  2010-09-18 01:34:18 UTC (rev 21942)
@@ -3,15 +3,24 @@
 import org.cytoscape.view.model.VisualProperty;
 import org.cytoscape.view.vizmap.VisualMappingFunction;
 
+/**
+ * All fields are immutable.
+ * 
+ * @author kono
+ *
+ * @param <K>
+ * @param <V>
+ */
 public abstract class AbstractVisualMappingFunction<K, V> implements
                VisualMappingFunction<K, V> {
        
+       // TODO: Are these necessary?
        public static final String DISCRETE = "Discrete Mapping";
        public static final String CONTINUOUS = "Continuous Mapping";
        public static final String PASSTHROUGH = "Passthrough Mapping";
 
-       // Mapping attribute name.  This is mutable.
-       protected String attrName;
+       // Mapping attribute name.  This is immutable.
+       protected final String attrName;
        
        // Type of attribute
        protected final Class<K> attrType;

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to