Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(536)

Unified Diff: third_party/WebKit/Source/devtools/scripts/jsdoc_validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java

Issue 2441163002: DevTools: clean up scripts folder (Closed)
Patch Set: Fix sys.path for chromevox to load rjsmin Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/scripts/jsdoc_validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java
diff --git a/third_party/WebKit/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java b/third_party/WebKit/Source/devtools/scripts/jsdoc_validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java
similarity index 87%
rename from third_party/WebKit/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java
rename to third_party/WebKit/Source/devtools/scripts/jsdoc_validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java
index fa4392c64ed60ae98c3fbbf8e5d4049e04110baf..719afdbf2427738d6e20ba1d5b5f4e8fd2f59915 100644
--- a/third_party/WebKit/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java
+++ b/third_party/WebKit/Source/devtools/scripts/jsdoc_validator/src/org/chromium/devtools/jsdoc/JsDocValidator.java
@@ -20,7 +20,6 @@ import java.util.concurrent.Executors;
import java.util.concurrent.Future;
public class JsDocValidator {
-
private static final String FILES_LIST_NAME = "--files-list-name";
private void run(String[] args) {
@@ -44,20 +43,20 @@ public class JsDocValidator {
}
private String[] readFileNames(String[] args) {
- if (args.length != 2) {
- System.err.println("A single file name is expected to follow " + FILES_LIST_NAME);
- System.exit(1);
- }
- try {
- List<String> list = Files.readAllLines(
- FileSystems.getDefault().getPath(args[1]), Charset.forName("UTF-8"));
- return list.toArray(new String[list.size()]);
- } catch (IOException e) {
- System.err.println("Unable to read list file " + args[1]);
- e.printStackTrace();
- System.exit(1);
- return new String[0];
- }
+ if (args.length != 2) {
+ System.err.println("A single file name is expected to follow " + FILES_LIST_NAME);
+ System.exit(1);
+ }
+ try {
+ List<String> list = Files.readAllLines(
+ FileSystems.getDefault().getPath(args[1]), Charset.forName("UTF-8"));
+ return list.toArray(new String[list.size()]);
+ } catch (IOException e) {
+ System.err.println("Unable to read list file " + args[1]);
+ e.printStackTrace();
+ System.exit(1);
+ return new String[0];
+ }
}
private void runWithExecutor(String[] args, ExecutorService executor) {
@@ -96,8 +95,7 @@ public class JsDocValidator {
for (LogEntry entry : entries) {
System.err.println(entry.record.text);
}
- if (!entries.isEmpty())
- System.err.println("Total errors: " + entries.size());
+ if (!entries.isEmpty()) System.err.println("Total errors: " + entries.size());
}
public static void main(String[] args) {

Powered by Google App Engine
This is Rietveld 408576698