external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-17 11:18:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

use StringUtils to compare ignoring case (that will actually save Ncards + 1 string allocations per search)

Browse Source
This commit is contained in:
Maxmtg
2013-02-13 06:49:14 +00:00
parent 08dc701a60
commit df6f583c13
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/main/java/forge/gui/deckeditor/CDeckEditorUI.java
View File

@@ -38,6 +38,9 @@ import javax.swing.SwingConstants;
import javax.swing.SwingUtilities;
import javax.swing.Timer;
import org.apache.commons.lang.CharUtils;
import org.apache.commons.lang.StringUtils;
import forge.Card;
import forge.deck.DeckBase;
import forge.gui.CardContainer;
@@ -285,13 +288,13 @@ public enum CDeckEditorUI implements CardContainer {
startIdx %= numItems;
final int increment = reverse ? numItems - 1 : 1;
int stopIdx = (startIdx + numItems - increment) % numItems;
String searchStr = str.toString().toLowerCase(Locale.ENGLISH);
String searchStr = str.toString();
boolean found = false;
for (int idx = startIdx; true; idx = (idx + increment) % numItems) {
for (int idx = startIdx;; idx = (idx + increment) % numItems) {
@SuppressWarnings("unchecked")
EditorTableModel<? extends InventoryItem> tableModel =
(EditorTableModel<? extends InventoryItem>)tableView.getTable().getModel();
if (tableModel.rowToCard(idx).getKey().getName().toLowerCase(Locale.ENGLISH).contains(searchStr)) {
if (StringUtils.containsIgnoreCase(tableModel.rowToCard(idx).getKey().getName(), searchStr)) {
tableView.selectAndScrollTo(idx);
found = true;
break;
@@ -390,6 +393,8 @@ public enum CDeckEditorUI implements CardContainer {
// shift down is ok. anything else is likely to be a hotkey and should not be added to the string
return;
}
// Suggested change:
// if (!CharUtils.isAsciiPrintable(e.getKeyChar())) return;
str.append(e.getKeyChar());
}