From 8981467b299ed862909f77aef112dc7ad31c16be Mon Sep 17 00:00:00 2001 From: "Ray J. Greenwell" Date: Mon, 2 Feb 2015 13:33:27 -0800 Subject: [PATCH] Fix copy/paste error... that was itself copy/pasted. Strings were always being shoved into a Comparable array. Does the array type matter? Apparently not. We should just rip out these tests and always store in a Comparable[]. --- src/main/java/com/samskivert/depot/KeySet.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/samskivert/depot/KeySet.java b/src/main/java/com/samskivert/depot/KeySet.java index 69dfecd..2f7964f 100644 --- a/src/main/java/com/samskivert/depot/KeySet.java +++ b/src/main/java/com/samskivert/depot/KeySet.java @@ -52,7 +52,7 @@ public abstract class KeySet extends WhereClause Comparable[] keyArray; if (first instanceof Integer) { keyArray = new Integer[keys.size()]; - } else if (first instanceof Integer) { + } else if (first instanceof String) { keyArray = new String[keys.size()]; } else { keyArray = new Comparable[keys.size()]; @@ -96,7 +96,7 @@ public abstract class KeySet extends WhereClause Comparable[] keyArray; if (first instanceof Integer) { keyArray = new Integer[keys.size()]; - } else if (first instanceof Integer) { + } else if (first instanceof String) { keyArray = new String[keys.size()]; } else { keyArray = new Comparable[keys.size()];