Forwarding this contribution from Ahmed to core-libs-dev alias as the change is going to be in "jdk/java.scripting/javax.script" code.

PS. I'll send out webrev after build, test.

Thanks Ahmed,
-Sundar


-------- Forwarded Message --------
Subject:        Re: [PATCH] javax,script.ScriptEngineFactory Typos
Date:   Fri, 10 Jul 2015 07:46:40 +0200
From:   Ahmed Ashour <asash...@yahoo.com>
To:     nashorn-...@openjdk.java.net



Dear all,

Please find below a proposed patch based on jdk9/dev.

Thanks,
Ahmed


diff -r b526c2584b4b
src/java.scripting/share/classes/javax/script/ScriptEngineFactory.java
---
a/src/java.scripting/share/classes/javax/script/ScriptEngineFactory.java
Wed Jul 08 21:54:32 2015 -0400
+++
b/src/java.scripting/share/classes/javax/script/ScriptEngineFactory.java
Thu Jul 09 08:10:26 2015 +0200
@@ -160,7 +160,6 @@
      *      }
      *      ret += ")";
      *      return ret;
-     * }
      * } </pre>
      *
      * @param obj The name representing the object whose method is to
be invoked. The
@@ -190,8 +189,6 @@
      *
      * @param toDisplay The String to be displayed by the returned
statement.
      * @return The string used to display the String in the syntax of
the scripting language.
-     *
-     *
      */
     public String getOutputStatement(String toDisplay);

@@ -208,14 +205,12 @@
      *          retval += statements[i] + ";\n";
      *      }
      *      return retval += "?>";
-     * }
      * }</pre>
      *
      *  @param statements The statements to be executed.  May be
return values of
      *  calls to the <code>getMethodCallSyntax</code> and
<code>getOutputStatement</code> methods.
      *  @return The Program
      */
-
     public String getProgram(String... statements);

     /**
@@ -225,5 +220,5 @@
      *
      * @return A new <code>ScriptEngine</code> instance.
      */
-    public  ScriptEngine getScriptEngine();
+    public ScriptEngine getScriptEngine();
 }



Reply via email to