Java cleanup, convert ArrayList and Vector to use generics
Change-Id: Ic668b46872ee0bfd259ca335aed9d68fb545c3a4
This commit is contained in:
committed by
Michael Stahl
parent
6bf09ecf1d
commit
4c9e62c6e3
@@ -41,11 +41,11 @@ public class JavaFinder implements MethodFinder {
|
||||
private static final String FIRST_PARAM =
|
||||
"drafts.com.sun.star.script.framework.runtime.XScriptContext";
|
||||
|
||||
private Vector classpath = null;
|
||||
private Vector<String> classpath = null;
|
||||
|
||||
private JavaFinder() {}
|
||||
|
||||
private JavaFinder(Vector classpath) {
|
||||
private JavaFinder(Vector<String> classpath) {
|
||||
this.classpath = classpath;
|
||||
}
|
||||
|
||||
@@ -59,7 +59,7 @@ public class JavaFinder implements MethodFinder {
|
||||
return finder;
|
||||
}
|
||||
|
||||
public static JavaFinder getInstance(Vector classpath) {
|
||||
public static JavaFinder getInstance(Vector<String> classpath) {
|
||||
return new JavaFinder(classpath);
|
||||
}
|
||||
|
||||
@@ -126,7 +126,7 @@ public class JavaFinder implements MethodFinder {
|
||||
}
|
||||
|
||||
if (result.size() != 0)
|
||||
return (ScriptEntry[])result.toArray(empty);
|
||||
return result.toArray(empty);
|
||||
return empty;
|
||||
}
|
||||
|
||||
@@ -136,7 +136,7 @@ public class JavaFinder implements MethodFinder {
|
||||
|
||||
for (int i = 0; i < len; i++) {
|
||||
try {
|
||||
String s = (String)classpath.elementAt(i);
|
||||
String s = classpath.elementAt(i);
|
||||
s = SVersionRCFile.toFileURL(s);
|
||||
|
||||
if (s != null)
|
||||
@@ -146,7 +146,7 @@ public class JavaFinder implements MethodFinder {
|
||||
}
|
||||
}
|
||||
|
||||
return new URLClassLoader((URL[])urls.toArray(new URL[0]));
|
||||
return new URLClassLoader(urls.toArray(new URL[0]));
|
||||
}
|
||||
|
||||
private ClassLoader getClassLoader(File basedir) {
|
||||
@@ -175,7 +175,7 @@ public class JavaFinder implements MethodFinder {
|
||||
File f;
|
||||
for (int i = 0; i < urls.length; i++) {
|
||||
try {
|
||||
f = (File)files.get(i);
|
||||
f = files.get(i);
|
||||
urlpath = SVersionRCFile.toFileURL(f.getAbsolutePath());
|
||||
|
||||
if (urlpath != null)
|
||||
@@ -215,7 +215,7 @@ public class JavaFinder implements MethodFinder {
|
||||
ArrayList<String> result = new ArrayList<String>();
|
||||
for (int i = 0; i < classFiles.size(); i++)
|
||||
{
|
||||
File classFile = (File)classFiles.get(i);
|
||||
File classFile = classFiles.get(i);
|
||||
String className = classFile.getName();
|
||||
className = className.substring(0, className.lastIndexOf(CLASS_SUFFIX));
|
||||
boolean finished = false;
|
||||
@@ -223,7 +223,7 @@ public class JavaFinder implements MethodFinder {
|
||||
|
||||
for (int j = 0; j < javaFiles.size() && finished == false; j++)
|
||||
{
|
||||
File javaFile = (File)javaFiles.get(j);
|
||||
File javaFile = javaFiles.get(j);
|
||||
String javaName = javaFile.getName();
|
||||
javaName = javaName.substring(0, javaName.lastIndexOf(JAVA_SUFFIX));
|
||||
|
||||
@@ -240,6 +240,6 @@ public class JavaFinder implements MethodFinder {
|
||||
}
|
||||
}
|
||||
}
|
||||
return (String[])result.toArray(new String[0]);
|
||||
return result.toArray(new String[0]);
|
||||
}
|
||||
}
|
||||
|
@@ -43,7 +43,7 @@ public class LocalOffice
|
||||
public static final LocalOffice create(
|
||||
ClassLoader parent, String officePath, int port)
|
||||
{
|
||||
Vector path = new Vector();
|
||||
Vector<String> path = new Vector<String>();
|
||||
path.addElement(officePath + "/program/classes/ridl.jar");
|
||||
path.addElement(officePath + "/program/classes/jurt.jar");
|
||||
path.addElement(officePath + "/program/classes/unoil.jar");
|
||||
|
@@ -51,9 +51,9 @@ public class OfficeDocument
|
||||
return false;
|
||||
}
|
||||
|
||||
public Enumeration getParcels() {
|
||||
public Enumeration<String> getParcels() {
|
||||
|
||||
Vector parcels = new Vector();
|
||||
Vector<String> parcels = new Vector<String>();
|
||||
ZipFile zp = null;
|
||||
|
||||
try
|
||||
|
@@ -57,11 +57,11 @@ public class SVersionRCFile {
|
||||
/* Make sure this is in LowerCase !!!!! */
|
||||
private static final String SCRIPTF = "scriptf";
|
||||
|
||||
private static final HashMap files = new HashMap(3);
|
||||
private static final HashMap<String, SVersionRCFile> files = new HashMap<String, SVersionRCFile>(3);
|
||||
|
||||
private File sversionrc = null;
|
||||
private OfficeInstallation defaultversion = null;
|
||||
private Vector versions = null;
|
||||
private Vector<OfficeInstallation> versions = null;
|
||||
private long lastModified = 0;
|
||||
|
||||
public SVersionRCFile() {
|
||||
@@ -70,7 +70,7 @@ public class SVersionRCFile {
|
||||
|
||||
public SVersionRCFile(String name) {
|
||||
sversionrc = new File(name);
|
||||
versions = new Vector(5);
|
||||
versions = new Vector<OfficeInstallation>(5);
|
||||
}
|
||||
|
||||
public static SVersionRCFile createInstance() {
|
||||
@@ -81,7 +81,7 @@ public class SVersionRCFile {
|
||||
SVersionRCFile result = null;
|
||||
|
||||
synchronized(SVersionRCFile.class) {
|
||||
result = (SVersionRCFile)files.get(name);
|
||||
result = files.get(name);
|
||||
|
||||
if (result == null) {
|
||||
result = new SVersionRCFile(name);
|
||||
@@ -99,7 +99,7 @@ public class SVersionRCFile {
|
||||
return defaultversion;
|
||||
}
|
||||
|
||||
public Enumeration getVersions() throws IOException {
|
||||
public Enumeration<OfficeInstallation> getVersions() throws IOException {
|
||||
|
||||
long l = sversionrc.lastModified();
|
||||
|
||||
@@ -209,7 +209,7 @@ public class SVersionRCFile {
|
||||
else
|
||||
ov = new SVersionRCFile(args[0]);
|
||||
|
||||
Enumeration enumer;
|
||||
Enumeration<OfficeInstallation> enumer;
|
||||
|
||||
try {
|
||||
enumer = ov.getVersions();
|
||||
@@ -220,7 +220,7 @@ public class SVersionRCFile {
|
||||
}
|
||||
|
||||
while (enumer.hasMoreElements()) {
|
||||
OfficeInstallation oi = (OfficeInstallation)enumer.nextElement();
|
||||
OfficeInstallation oi = enumer.nextElement();
|
||||
System.out.println("Name: " + oi.getName() + ", Path: " + oi.getPath() +
|
||||
", URL: " + oi.getURL());
|
||||
}
|
||||
|
@@ -38,13 +38,14 @@ import java.awt.event.ActionEvent;
|
||||
import java.awt.event.ActionListener;
|
||||
|
||||
import com.sun.star.script.framework.container.ParcelDescriptor;
|
||||
import com.sun.star.script.framework.container.ScriptEntry;
|
||||
|
||||
import org.openoffice.idesupport.zip.ParcelZipper;
|
||||
|
||||
public class ConfigurePanel extends JPanel {
|
||||
|
||||
private File basedir;
|
||||
private Vector classpath;
|
||||
private Vector<String> classpath;
|
||||
private ParcelDescriptor descriptor;
|
||||
|
||||
private MethodPanel methodPanel;
|
||||
@@ -53,7 +54,7 @@ public class ConfigurePanel extends JPanel {
|
||||
public static final String DIALOG_TITLE =
|
||||
"Choose What to Export as Scripts";
|
||||
|
||||
public ConfigurePanel(String basedir, Vector classpath,
|
||||
public ConfigurePanel(String basedir, Vector<String> classpath,
|
||||
ParcelDescriptor descriptor) {
|
||||
|
||||
this.basedir = new File(basedir);
|
||||
@@ -62,7 +63,7 @@ public class ConfigurePanel extends JPanel {
|
||||
initUI();
|
||||
}
|
||||
|
||||
public ConfigurePanel(String basedir, Vector classpath)
|
||||
public ConfigurePanel(String basedir, Vector<String> classpath)
|
||||
throws IOException {
|
||||
|
||||
this.basedir = new File(basedir);
|
||||
@@ -72,7 +73,7 @@ public class ConfigurePanel extends JPanel {
|
||||
initUI();
|
||||
}
|
||||
|
||||
public void reload(String basedir, Vector classpath,
|
||||
public void reload(String basedir, Vector<String> classpath,
|
||||
ParcelDescriptor descriptor) {
|
||||
|
||||
if (basedir != null)
|
||||
@@ -90,7 +91,7 @@ public class ConfigurePanel extends JPanel {
|
||||
scriptPanel.reload(descriptor.getScriptEntries());
|
||||
}
|
||||
|
||||
public void reload(String basedir, Vector classpath)
|
||||
public void reload(String basedir, Vector<String> classpath)
|
||||
throws IOException {
|
||||
|
||||
if (basedir != null)
|
||||
@@ -108,7 +109,7 @@ public class ConfigurePanel extends JPanel {
|
||||
}
|
||||
|
||||
public ParcelDescriptor getConfiguration() throws Exception {
|
||||
Enumeration scripts = scriptPanel.getScriptEntries();
|
||||
Enumeration<ScriptEntry> scripts = scriptPanel.getScriptEntries();
|
||||
descriptor.setScriptEntries(scripts);
|
||||
return descriptor;
|
||||
}
|
||||
|
@@ -33,7 +33,7 @@ import org.openoffice.idesupport.JavaFinder;
|
||||
public class MethodPanel extends JPanel {
|
||||
|
||||
private File basedir;
|
||||
private Vector classpath;
|
||||
private Vector<String> classpath;
|
||||
private final static String FIRST_PARAM =
|
||||
"drafts.com.sun.star.script.framework.runtime.XScriptContext";
|
||||
|
||||
@@ -42,7 +42,7 @@ public class MethodPanel extends JPanel {
|
||||
private JList list;
|
||||
private ScriptEntry[] values;
|
||||
|
||||
public MethodPanel(File basedir, Vector classpath, String language) {
|
||||
public MethodPanel(File basedir, Vector<String> classpath, String language) {
|
||||
this.basedir = basedir;
|
||||
this.classpath = classpath;
|
||||
|
||||
@@ -50,7 +50,7 @@ public class MethodPanel extends JPanel {
|
||||
initUI();
|
||||
}
|
||||
|
||||
public void reload(File basedir, Vector classpath, String language) {
|
||||
public void reload(File basedir, Vector<String> classpath, String language) {
|
||||
this.basedir = basedir;
|
||||
this.classpath = classpath;
|
||||
|
||||
|
@@ -81,7 +81,7 @@ public class ScriptPanel extends JPanel {
|
||||
model.removeAll();
|
||||
}
|
||||
|
||||
public Enumeration getScriptEntries() {
|
||||
public Enumeration<ScriptEntry> getScriptEntries() {
|
||||
return model.getScriptEntries();
|
||||
}
|
||||
|
||||
@@ -121,11 +121,11 @@ public class ScriptPanel extends JPanel {
|
||||
final String[] columnNames = {"Exported Method",
|
||||
"Script Name"};
|
||||
|
||||
private Vector scripts;
|
||||
private Vector<ScriptEntry> scripts;
|
||||
private int nextRow;
|
||||
|
||||
public ScriptTableModel(ScriptEntry[] entries) {
|
||||
scripts = new Vector(entries.length + 11);
|
||||
scripts = new Vector<ScriptEntry>(entries.length + 11);
|
||||
for (int i = 0; i < entries.length; i++) {
|
||||
scripts.addElement(entries[i]);
|
||||
}
|
||||
@@ -162,7 +162,7 @@ public class ScriptPanel extends JPanel {
|
||||
nextRow = 0;
|
||||
}
|
||||
|
||||
public Enumeration getScriptEntries() {
|
||||
public Enumeration<ScriptEntry> getScriptEntries() {
|
||||
return scripts.elements();
|
||||
}
|
||||
|
||||
@@ -170,7 +170,7 @@ public class ScriptPanel extends JPanel {
|
||||
String result = "";
|
||||
ScriptEntry entry;
|
||||
|
||||
entry = (ScriptEntry)scripts.elementAt(row);
|
||||
entry = scripts.elementAt(row);
|
||||
|
||||
if (col == 0)
|
||||
result = entry.getLanguageName();
|
||||
@@ -188,7 +188,7 @@ public class ScriptPanel extends JPanel {
|
||||
}
|
||||
|
||||
public void setValueAt(Object value, int row, int col) {
|
||||
ScriptEntry entry = (ScriptEntry)scripts.elementAt(row);
|
||||
ScriptEntry entry = scripts.elementAt(row);
|
||||
entry.setLogicalName((String)value);
|
||||
fireTableCellUpdated(row, col);
|
||||
}
|
||||
|
Reference in New Issue
Block a user