Clean up checkstyle warnings and formatting.
Patch by: jat
Unreviewed
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@4667 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/dev/core/src/com/google/gwt/dev/ArgHandlerOutDirDeprecated.java b/dev/core/src/com/google/gwt/dev/ArgHandlerOutDirDeprecated.java
index 13549a8..9845eaa 100644
--- a/dev/core/src/com/google/gwt/dev/ArgHandlerOutDirDeprecated.java
+++ b/dev/core/src/com/google/gwt/dev/ArgHandlerOutDirDeprecated.java
@@ -1,5 +1,17 @@
-/**
+/*
+ * Copyright 2008 Google Inc.
*
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
*/
package com.google.gwt.dev;
@@ -14,15 +26,17 @@
public class ArgHandlerOutDirDeprecated extends ArgHandlerOutDir {
OptionOutDir option;
-
+
public ArgHandlerOutDirDeprecated(OptionOutDir option) {
this.option = option;
}
-
+
+ @Override
public String getPurpose() {
return super.getPurpose() + " (deprecated)";
}
+ @Override
public boolean isUndocumented() {
return true;
}
@@ -31,5 +45,5 @@
public void setDir(File dir) {
option.setOutDir(dir);
}
-
+
}
diff --git a/dev/core/src/com/google/gwt/dev/Link.java b/dev/core/src/com/google/gwt/dev/Link.java
index a84e8cf..e590129 100644
--- a/dev/core/src/com/google/gwt/dev/Link.java
+++ b/dev/core/src/com/google/gwt/dev/Link.java
@@ -99,6 +99,11 @@
return extraDir;
}
+ @Deprecated
+ public File getOutDir() {
+ return outDir;
+ }
+
public File getWarDir() {
return warDir;
}
@@ -107,19 +112,14 @@
this.extraDir = extraDir;
}
- public void setWarDir(File warDir) {
- this.warDir = warDir;
- }
-
- @Deprecated
- public File getOutDir() {
- return outDir;
- }
-
@Deprecated
public void setOutDir(File outDir) {
this.outDir = outDir;
}
+
+ public void setWarDir(File warDir) {
+ this.warDir = warDir;
+ }
}
public static void legacyLink(TreeLogger logger, ModuleDef module,
diff --git a/dev/core/src/com/google/gwt/dev/Precompile.java b/dev/core/src/com/google/gwt/dev/Precompile.java
index c16b494..daa5afc 100644
--- a/dev/core/src/com/google/gwt/dev/Precompile.java
+++ b/dev/core/src/com/google/gwt/dev/Precompile.java
@@ -159,14 +159,14 @@
jjsOptions.setAggressivelyOptimize(aggressivelyOptimize);
}
- public void setDraftCompile(boolean draft) {
- jjsOptions.setDraftCompile(draft);
- }
-
public void setDisableUpdateCheck(boolean disabled) {
disableUpdateCheck = disabled;
}
+ public void setDraftCompile(boolean draft) {
+ jjsOptions.setDraftCompile(draft);
+ }
+
public void setEnableAssertions(boolean enableAssertions) {
jjsOptions.setEnableAssertions(enableAssertions);
}
diff --git a/dev/core/src/com/google/gwt/dev/js/EvalFunctionsAtTopScope.java b/dev/core/src/com/google/gwt/dev/js/EvalFunctionsAtTopScope.java
index 9455037..b22a69c 100644
--- a/dev/core/src/com/google/gwt/dev/js/EvalFunctionsAtTopScope.java
+++ b/dev/core/src/com/google/gwt/dev/js/EvalFunctionsAtTopScope.java
@@ -1,3 +1,18 @@
+/*
+ * Copyright 2009 Google Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
package com.google.gwt.dev.js;
import com.google.gwt.dev.js.ast.JsBlock;
@@ -15,15 +30,23 @@
import java.util.Stack;
/**
- * Force all functions to be evaluated at the top of the lexical scope in
- * which they reside. This makes {@link StaticEvalVisitor} simpler in that we
- * no longer have to worry about function declarations within expressions.
- * After this runs, only statements can contain declarations. Moved functions
- * will end up just before the statement in which they presently reside.
+ * Force all functions to be evaluated at the top of the lexical scope in which
+ * they reside. This makes {@link StaticEvalVisitor} simpler in that we no
+ * longer have to worry about function declarations within expressions. After
+ * this runs, only statements can contain declarations. Moved functions will end
+ * up just before the statement in which they presently reside.
*/
public class EvalFunctionsAtTopScope extends JsModVisitor {
+
+ public static void exec(JsProgram jsProgram) {
+ EvalFunctionsAtTopScope fev = new EvalFunctionsAtTopScope();
+ fev.accept(jsProgram);
+ }
+
private final Set<JsFunction> dontMove = new HashSet<JsFunction>();
+
private final Stack<ListIterator<JsStatement>> itrStack = new Stack<ListIterator<JsStatement>>();
+
private final Stack<JsBlock> scopeStack = new Stack<JsBlock>();
@Override
@@ -76,8 +99,8 @@
/*
* Reinsert this function into the statement immediately before the
* current statement. The current statement will have already been
- * returned from the current iterator's next(), so we have to
- * backshuffle one step to get in front of it.
+ * returned from the current iterator's next(), so we have to backshuffle
+ * one step to get in front of it.
*/
ListIterator<JsStatement> itr = itrStack.peek();
itr.previous();
@@ -104,9 +127,4 @@
scopeStack.push(x.getGlobalBlock());
return true;
}
-
- public static void exec(JsProgram jsProgram) {
- EvalFunctionsAtTopScope fev = new EvalFunctionsAtTopScope();
- fev.accept(jsProgram);
- }
-}
\ No newline at end of file
+}
diff --git a/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java b/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java
index 075bc13..cafdacc 100644
--- a/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java
+++ b/dev/core/src/com/google/gwt/dev/resource/impl/ResourceOracleImpl.java
@@ -15,6 +15,12 @@
*/
package com.google.gwt.dev.resource.impl;
+import com.google.gwt.core.ext.TreeLogger;
+import com.google.gwt.dev.resource.Resource;
+import com.google.gwt.dev.resource.ResourceOracle;
+import com.google.gwt.dev.util.msg.Message0;
+import com.google.gwt.dev.util.msg.Message1String;
+
import java.io.File;
import java.io.IOException;
import java.io.InputStream;
@@ -34,12 +40,6 @@
import java.util.jar.JarFile;
import java.util.zip.ZipFile;
-import com.google.gwt.core.ext.TreeLogger;
-import com.google.gwt.dev.resource.Resource;
-import com.google.gwt.dev.resource.ResourceOracle;
-import com.google.gwt.dev.util.msg.Message0;
-import com.google.gwt.dev.util.msg.Message1String;
-
/**
* The normal implementation of {@link ResourceOracle}.
*