format with Locale.US (tabulapdf#355)

qmdx · Apr 18, 2020 · eec86f5 · eec86f5
1 parent 6278064
commit eec86f5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/main/java/technology/tabula/QuickSort.java b/src/main/java/technology/tabula/QuickSort.java
@@ -25,7 +25,7 @@
 /**
  * An implementation of Quicksort.
  * 
- * @see http://de.wikipedia.org/wiki/Quicksort
+ * @see <a href="http://de.wikipedia.org/wiki/Quicksort">wikipedia</a>
  * 
  * @author UWe Pachler
  */

diff --git a/src/main/java/technology/tabula/Rectangle.java b/src/main/java/technology/tabula/Rectangle.java
@@ -4,14 +4,15 @@
 import java.awt.geom.Rectangle2D;
 import java.util.Comparator;
 import java.util.List;
+import java.util.Locale;
 
 @SuppressWarnings("serial")
 public class Rectangle extends Rectangle2D.Float {
 
 	/**
 	 * Ill-defined comparator, from when Rectangle was Comparable.
 	 * 
-	 * @see https://github.com/tabulapdf/tabula-java/issues/116
+	 * @see <a href="https://github.com/tabulapdf/tabula-java/issues/116">PR 116</a>
 	 * @deprecated with no replacement
 	 */
 	@Deprecated
@@ -151,7 +152,7 @@ public String toString() {
 		StringBuilder sb = new StringBuilder();
 		String s = super.toString();
 		sb.append(s.substring(0, s.length() - 1));
-		sb.append(String.format(",bottom=%f,right=%f]", this.getBottom(), this.getRight()));
+		sb.append(String.format(Locale.US, ",bottom=%f,right=%f]", this.getBottom(), this.getRight()));
 		return sb.toString();
 	}
 

diff --git a/src/main/java/technology/tabula/Ruling.java b/src/main/java/technology/tabula/Ruling.java
@@ -8,6 +8,7 @@
 import java.util.Comparator;
 import java.util.Formatter;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import java.util.TreeMap;
 
@@ -291,7 +292,7 @@ public double getAngle() {
     public String toString() {
         StringBuilder sb = new StringBuilder();
         Formatter formatter = new Formatter(sb);
-        String rv = formatter.format("%s[x1=%f y1=%f x2=%f y2=%f]", this.getClass().toString(), this.x1, this.y1, this.x2, this.y2).toString();
+        String rv = formatter.format(Locale.US, "%s[x1=%f y1=%f x2=%f y2=%f]", this.getClass().toString(), this.x1, this.y1, this.x2, this.y2).toString();
         formatter.close();
         return rv;
     }