Skip to content
Snippets Groups Projects
Commit c5e6a5f5 authored by Piotr Gawron's avatar Piotr Gawron
Browse files

comparators are more verbose

parent 5aefb13c
No related branches found
No related tags found
2 merge requests!630WIP: Resolve "The privileges of a new user are not saved in some cases",!581Resolve "export to sbml layout creates partial lines for operators"
...@@ -3,6 +3,8 @@ package lcsb.mapviewer.common.comparator; ...@@ -3,6 +3,8 @@ package lcsb.mapviewer.common.comparator;
import java.awt.geom.Line2D; import java.awt.geom.Line2D;
import java.util.Comparator; import java.util.Comparator;
import org.apache.log4j.Logger;
import lcsb.mapviewer.common.Configuration; import lcsb.mapviewer.common.Configuration;
/** /**
...@@ -13,6 +15,8 @@ import lcsb.mapviewer.common.Configuration; ...@@ -13,6 +15,8 @@ import lcsb.mapviewer.common.Configuration;
*/ */
public class LineComparator implements Comparator<Line2D> { public class LineComparator implements Comparator<Line2D> {
private static Logger logger = Logger.getLogger(LineComparator.class);
private PointComparator pointComparator; private PointComparator pointComparator;
/** /**
...@@ -45,8 +49,12 @@ public class LineComparator implements Comparator<Line2D> { ...@@ -45,8 +49,12 @@ public class LineComparator implements Comparator<Line2D> {
return -1; return -1;
} }
if (pointComparator.compare(arg0.getP1(), arg1.getP1()) != 0) { if (pointComparator.compare(arg0.getP1(), arg1.getP1()) != 0) {
logger.debug("Line start different: [" + arg0.getP1() + ", " + arg0.getP2() + "] - [" + arg1.getP1() + ", "
+ arg1.getP2() + "]");
return pointComparator.compare(arg0.getP1(), arg1.getP1()); return pointComparator.compare(arg0.getP1(), arg1.getP1());
} else if (pointComparator.compare(arg0.getP2(), arg1.getP2()) != 0) { } else if (pointComparator.compare(arg0.getP2(), arg1.getP2()) != 0) {
logger.debug("Line end different: [" + arg0.getP1() + ", " + arg0.getP2() + "] - [" + arg1.getP1() + ", "
+ arg1.getP2() + "]");
return pointComparator.compare(arg0.getP2(), arg1.getP2()); return pointComparator.compare(arg0.getP2(), arg1.getP2());
} else { } else {
return 0; return 0;
......
...@@ -15,7 +15,6 @@ public class ListComparator<T> implements Comparator<List<T>> { ...@@ -15,7 +15,6 @@ public class ListComparator<T> implements Comparator<List<T>> {
/** /**
* Default class logger. * Default class logger.
*/ */
@SuppressWarnings("unused")
private Logger logger = Logger.getLogger(ListComparator.class); private Logger logger = Logger.getLogger(ListComparator.class);
private Comparator<T> objectComparator; private Comparator<T> objectComparator;
...@@ -37,6 +36,7 @@ public class ListComparator<T> implements Comparator<List<T>> { ...@@ -37,6 +36,7 @@ public class ListComparator<T> implements Comparator<List<T>> {
} }
if (arg0.size() != arg1.size()) { if (arg0.size() != arg1.size()) {
logger.debug("Number of elements in list different: " + arg0.size() + ", " + arg1.size());
return new IntegerComparator().compare(arg0.size(), arg1.size()); return new IntegerComparator().compare(arg0.size(), arg1.size());
} }
for (int i = 0; i < arg0.size(); i++) { for (int i = 0; i < arg0.size(); i++) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment