8186011: Fix samples/java_completion.js and samples/disassemble.js
authorsdama
Tue, 26 Sep 2017 00:37:37 +0530
changeset 47260 be020513a185
parent 47259 0b347d8efb40
child 47261 ae970828ec0c
8186011: Fix samples/java_completion.js and samples/disassemble.js Summary: replaced continuation property to method as API changed and removed invalid test in jdk9 modular system Reviewed-by: hannesw, sundar Contributed-by: srinivas.dama@oracle.com
src/sample/nashorn/disassemble.js
src/sample/nashorn/java_completion.js
--- a/src/sample/nashorn/disassemble.js	Mon Sep 25 11:52:40 2017 +0200
+++ /dev/null	Thu Jan 01 00:00:00 1970 +0000
@@ -1,75 +0,0 @@
-/*
- * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- *   - Redistributions of source code must retain the above copyright
- *     notice, this list of conditions and the following disclaimer.
- *
- *   - Redistributions in binary form must reproduce the above copyright
- *     notice, this list of conditions and the following disclaimer in the
- *     documentation and/or other materials provided with the distribution.
- *
- *   - Neither the name of Oracle nor the names of its
- *     contributors may be used to endorse or promote products derived
- *     from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS
- * IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE COPYRIGHT OWNER OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
- * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
- * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
- * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-// Usage: jjs disassemble.js -- <.class-file-path>
-
-// Simple .class disassembler that uses bundled ObjectWeb ASM
-// classes in jdk8. WARNING: Bundled ObjectWeb ASM classes are
-// not part of official jdk8 API. It can be changed/removed
-// without notice. So, this script is brittle by design!
-
-// This example demonstrates passing arguments to script
-// from jjs command line, nio and ASM usage.
-
-// classes used
-var FileSystems = Java.type("java.nio.file.FileSystems");
-var Files = Java.type("java.nio.file.Files");
-var System = Java.type("java.lang.System");
-var PrintWriter = Java.type("java.io.PrintWriter");
-
-// WARNING: uses non-API classes of jdk8!
-var ClassReader = Java.type("jdk.internal.org.objectweb.asm.ClassReader");
-var TraceClassVisitor = Java.type("jdk.internal.org.objectweb.asm.util.TraceClassVisitor");
-
-// convert file name to Path instance
-function path(file) {
-    return FileSystems.default.getPath(file);
-}
-
-// read all file content as a byte[]
-function readAllBytes(file) {
-    return Files.readAllBytes(path(file));
-}
-
-// disassemble .class byte[] and prints output to stdout
-function disassemble(bytecode) {
-    var pw = new PrintWriter(System.out);
-    new ClassReader(bytecode).accept(new TraceClassVisitor(pw), 0);
-}
-
-// check for command line arg (for .class file name)
-if (arguments.length == 0 || !arguments[0].endsWith('.class')) {
-    print("Usage: jjs disassemble -- <.class file>");
-    exit(1);
-}
-
-// disassemble the given .class file
-disassemble(readAllBytes(arguments[0]));
--- a/src/sample/nashorn/java_completion.js	Mon Sep 25 11:52:40 2017 +0200
+++ b/src/sample/nashorn/java_completion.js	Tue Sep 26 00:37:37 2017 +0530
@@ -38,4 +38,4 @@
 var analysis = repl.sourceCodeAnalysis()
 var code = "System."
 var suggestions = analysis.completionSuggestions(code, code.length, [0])
-suggestions.forEach(function(s) print(s.continuation))
+suggestions.forEach(function(s) print(s.continuation()))