From 2c0e26241202e1caf1b9babb697f31a6fabe2cb6 Mon Sep 17 00:00:00 2001
From: rjy7 <rjy7@440791e1-c2bf-4c1e-ad7c-caf3f8b0ebe8>
Date: Tue, 6 Jul 2010 01:45:09 +0000
Subject: [PATCH] NIHVIVO-646 Small style changes. A note in
 DataPropertyComparator.

---
 .../mannlib/vitro/webapp/beans/DataPropertyComparator.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java
index da32fc61c..de7a5b280 100644
--- a/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java
+++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java
@@ -20,7 +20,9 @@ public class DataPropertyComparator implements Comparator<Individual> {
 
         int result;
         
-        // This pushes null values to the end of the list
+        // Push null values to the end of the list.
+        // Is this generally what's wanted? Or should this class be 
+        // NullLastDataPropertyComparator?
         if (dps1 == null) {
             result = (dps2 == null) ? 0 : 1;
         } else if (dps2 == null) {