diff options
author | Shawn O. Pearce <sop@google.com> | 2009-01-29 10:57:32 -0800 |
---|---|---|
committer | Shawn O. Pearce <sop@google.com> | 2009-01-29 10:57:32 -0800 |
commit | 50d4ee060ac88cd5d14d4f2154e28430cb1c8dad (patch) | |
tree | 33623423d8adc49f8e3b31bad2080b35d3b12caf | |
parent | a1a6d20449aa1c4a3069413bb3f98efa7d300a73 (diff) |
Remove comment about sorting change dependencies
Sorting by the quite stable change id is reasonable. On current
Gerrit2 systems its sequential, due to having a single sequence
generator assigning change ids in order.
Signed-off-by: Shawn O. Pearce <sop@google.com>
-rw-r--r-- | appjar/src/main/java/com/google/gerrit/client/data/ChangeDetail.java | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/appjar/src/main/java/com/google/gerrit/client/data/ChangeDetail.java b/appjar/src/main/java/com/google/gerrit/client/data/ChangeDetail.java index 10e5ad3dcc..faaf2a4e17 100644 --- a/appjar/src/main/java/com/google/gerrit/client/data/ChangeDetail.java +++ b/appjar/src/main/java/com/google/gerrit/client/data/ChangeDetail.java @@ -159,7 +159,6 @@ public class ChangeDetail { Collections.sort(neededBy, new Comparator<ChangeInfo>() { public int compare(final ChangeInfo o1, final ChangeInfo o2) { - // TODO sort neededBy by something more reasonable than Id return o1.getId().get() - o2.getId().get(); } }); |