diff --git a/src/enginuity/maps/DataCell.java b/src/enginuity/maps/DataCell.java index fd17427a..4e4fa8dc 100644 --- a/src/enginuity/maps/DataCell.java +++ b/src/enginuity/maps/DataCell.java @@ -1,17 +1,16 @@ package enginuity.maps; -import com.sun.corba.se.spi.activation._ActivatorImplBase; -import enginuity.maps.Scale; -import enginuity.maps.Table; import java.awt.Color; import java.awt.Font; import java.awt.event.MouseEvent; import java.awt.event.MouseListener; import java.io.Serializable; import java.text.DecimalFormat; + import javax.swing.JLabel; import javax.swing.border.LineBorder; -import org.nfunk.jep.*; + +import org.nfunk.jep.JEP; public class DataCell extends JLabel implements MouseListener, Serializable { diff --git a/src/enginuity/swing/XMLFilter.java b/src/enginuity/swing/XMLFilter.java index 3861caa9..285d754c 100644 --- a/src/enginuity/swing/XMLFilter.java +++ b/src/enginuity/swing/XMLFilter.java @@ -8,13 +8,13 @@ import javax.swing.filechooser.FileFilter; public class XMLFilter extends FileFilter { - private Hashtable filters = null; + private Hashtable filters = null; private String description = null; private String fullDescription = null; private boolean useExtensionsInDescription = true; public XMLFilter() { - this.filters = new Hashtable(); + this.filters = new Hashtable(); this.addExtension("xml"); this.setDescription("ECU Definition Files"); } @@ -46,9 +46,6 @@ public class XMLFilter extends FileFilter { } public void addExtension(String extension) { - if (filters == null) { - filters = new Hashtable(5); - } filters.put(extension.toLowerCase(), this); fullDescription = null; } diff --git a/src/enginuity/xml/DOMRomUnmarshaller.java b/src/enginuity/xml/DOMRomUnmarshaller.java index cfab6f47..6f6a742b 100644 --- a/src/enginuity/xml/DOMRomUnmarshaller.java +++ b/src/enginuity/xml/DOMRomUnmarshaller.java @@ -163,7 +163,8 @@ public class DOMRomUnmarshaller { return romID; } - private Table copyTable(Table input) { + // Will this function be used? It is not used now and could be removed... + private Table copyTable(Table input) { Table output = input; return output; } diff --git a/src/enginuity/xml/DOMSettingsUnmarshaller.java b/src/enginuity/xml/DOMSettingsUnmarshaller.java index 9c690dfd..782e8a0f 100644 --- a/src/enginuity/xml/DOMSettingsUnmarshaller.java +++ b/src/enginuity/xml/DOMSettingsUnmarshaller.java @@ -1,17 +1,15 @@ package enginuity.xml; -import com.sun.org.apache.xerces.internal.parsers.DOMParser; -import enginuity.Settings; import java.awt.Color; import java.awt.Dimension; import java.awt.Font; import java.awt.Point; import java.io.File; -import java.io.FileInputStream; -import org.w3c.dom.Document; + import org.w3c.dom.Node; import org.w3c.dom.NodeList; -import org.xml.sax.InputSource; + +import enginuity.Settings; public class DOMSettingsUnmarshaller { @@ -51,10 +49,6 @@ public class DOMSettingsUnmarshaller { unmarshallAttribute(n, "x", 800))); } else if (n.getNodeType() == Node.ELEMENT_NODE && n.getNodeName().equalsIgnoreCase("location")) { - // set default location in center of screen - Dimension screenSize = java.awt.Toolkit.getDefaultToolkit().getScreenSize(); - Point location = new Point(((int)(screenSize.getWidth() - settings.getWindowSize().getWidth()) / 2), - ((int)(screenSize.getHeight() - settings.getWindowSize().getHeight()) / 2)); settings.setWindowLocation(new Point(unmarshallAttribute(n, "x", 0), unmarshallAttribute(n, "y", 0))); @@ -176,29 +170,32 @@ public class DOMSettingsUnmarshaller { unmarshallAttribute(colorNode, "b", 155)); } - private String unmarshallText(Node textNode) { - StringBuffer buf = new StringBuffer(); + // Will this function be used? It is not used now and could be removed... + private String unmarshallText(Node textNode) { + StringBuffer buf = new StringBuffer(); + Node n; + NodeList nodes = textNode.getChildNodes(); - Node n; - NodeList nodes = textNode.getChildNodes(); + for (int i = 0; i < nodes.getLength(); i++) { + n = nodes.item(i); - for (int i = 0; i < nodes.getLength(); i++){ - n = nodes.item(i); - - if (n.getNodeType() == Node.TEXT_NODE) { - buf.append(n.getNodeValue()); - } else { - // expected a text-only node (skip) - } + if (n.getNodeType() == Node.TEXT_NODE) { + buf.append(n.getNodeValue()); + } + else { + // expected a text-only node (skip) + } + } + return buf.toString(); } - return buf.toString(); - } - - private String unmarshallAttribute(Node node, String name, String defaultValue) { - Node n = node.getAttributes().getNamedItem(name); - return (n!=null)?(n.getNodeValue()):(defaultValue); - } + + private String unmarshallAttribute(Node node, String name, + String defaultValue) { + Node n = node.getAttributes().getNamedItem(name); + return (n != null) ? (n.getNodeValue()) : (defaultValue); + } + // Will this function be used? It is not used now and could be removed... private Double unmarshallAttribute(Node node, String name, double defaultValue) { return Double.parseDouble(unmarshallAttribute(node, name, defaultValue+"")); }