findwith.js revision 1206:829e3b95b319
1/* 2 * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved. 3 * 4 * Redistribution and use in source and binary forms, with or without 5 * modification, are permitted provided that the following conditions 6 * are met: 7 * 8 * - Redistributions of source code must retain the above copyright 9 * notice, this list of conditions and the following disclaimer. 10 * 11 * - Redistributions in binary form must reproduce the above copyright 12 * notice, this list of conditions and the following disclaimer in the 13 * documentation and/or other materials provided with the distribution. 14 * 15 * - Neither the name of Oracle nor the names of its 16 * contributors may be used to endorse or promote products derived 17 * from this software without specific prior written permission. 18 * 19 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS 20 * IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, 21 * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR 22 * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR 23 * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, 24 * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, 25 * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR 26 * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF 27 * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING 28 * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS 29 * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 30 */ 31 32// 'with' statement is not considered a good practive. In strict mode, 33// with statements result in early error being thrown. 34 35// This script finds 'with' usage in scripts a given directory (recursively). 36// If no directory is specified the current working directory is scanned 37// for scripts. 38 39if (arguments.length == 0) { 40 arguments[0] = "."; 41} 42 43var File = Java.type("java.io.File"); 44var file = new File(arguments[0]); 45if (!file.exists()) { 46 print(arguments[0] + " is neither a directory nor a file"); 47 exit(1); 48} 49 50var Files = Java.type("java.nio.file.Files"); 51var Parser = Java.type("jdk.nashorn.api.tree.Parser"); 52var SimpleTreeVisitor = Java.type("jdk.nashorn.api.tree.SimpleTreeVisitorES5_1"); 53 54var parser = Parser.create("-scripting", "--const-as-var"); 55var protoFound = false; 56 57function checkFile(file) { 58 // print("checking " + file); 59 var ast = parser.parse(file, print); 60 if (!ast) { 61 return; 62 } 63 64 // locate __proto__ usage and warn 65 ast.accept(visitor = new (Java.extend(SimpleTreeVisitor)) { 66 lineMap: null, 67 68 printWarning: function(node) { 69 var pos = node.startPosition; 70 var line = this.lineMap.getLineNumber(pos); 71 var column = this.lineMap.getColumnNumber(pos); 72 print("WARNING: 'with' usage in " + file + " @ " + line + ":" + column); 73 }, 74 75 visitCompilationUnit: function(node, extra) { 76 this.lineMap = node.lineMap; 77 Java.super(visitor).visitCompilationUnit(node, extra); 78 }, 79 80 visitWith: function(node, extra) { 81 this.printWarning(node); 82 Java.super(visitor).visitWith(node, extra); 83 } 84 }, null); 85} 86 87if (file.isDirectory()) { 88 Files.walk(file.toPath()) 89 .filter(function(p) Files.isRegularFile(p)) 90 .filter(function(p) p.toFile().name.endsWith('.js')) 91 .forEach(checkFile); 92} else { 93 checkFile(file); 94} 95 96if (protoFound) { 97 print("__proto__ is non-standard. Use Object.get/setPrototypeOf instead"); 98} 99