print() inside a script sometime may not print the args

[nmondal]

This is coming from here:
nmondal/cowj#35

The reason why the print() was not working is a premature optimization in the rhino code base.
Specifically:
org/mozilla/javascript/engine/Builtins.java

Has this code:

public static void print(Context cx, Scriptable thisObj, Object[] args, Function f)
      throws IOException {
    Builtins self = getSelf(thisObj);
    for (Object arg : args) {
      self.stdout.write(ScriptRuntime.toString(arg));
    }
    self.stdout.write('\n'); // bug here, assumes line buffering and auto flush 
  }

Problem happens with this, self.stdout is not flushed. If one does flush it, the result is immediate.
This is a bug and we should let Mozilla folks replace the last line as:

self.stdout.println()

That should do it.

[p-bakker]

Feel like providing a PR with a testcase and a fix?

[nmondal]

Sure.. @p-bakker let's do it!

[nmondal]

@p-bakker here we go : #1358

[p-bakker]

@rbri how does this PR look to you?

[nmondal]

@p-bakker ... things run! Now .. we need to get some committers.. how?

[p-bakker]

As @rbri is one of the committers, I think/hope he'll take care of it :-)

Otherwise, just give it a bit of time 'till one of the other comitters gets around to it, all volunteers here.

[rbri]

Thanks again.