329908 - readFile should throw an error if the file being requested does not exist

Status: RESOLVED FIXED

(Rhino Graveyard :: Core, defect, P5)

Description

[cedwards]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en) AppleWebKit/417.9 (KHTML, like Gecko) Safari/417.8
Build Identifier: 

Calling readFile with a non-existent file currently returns an empty string; it should thow an IOException if the file does not exist.

Reproducible: Always

Steps to Reproduce:
1. run "print(readFile("doesnotexist"))
2.
3.

Actual Results:  
an empty string is returned

Expected Results:  
IOException if the file is not found

Comment 1

[Igor Bukanov]

Reassigning to the reporter as I have neither interest nor time for Rhino.

See also bug 288433.

Comment 2

[Mike Douglas]

Attachment: trivial patch

Anything else needed?

Comment 3

[cedwards]

Perfect.  Thanks!

Comment 4

[Hannes Wallnoefer]

Committed a slightly modified patch.

Checking in toolsrc/org/mozilla/javascript/tools/shell/Global.java;
/cvsroot/mozilla/js/rhino/toolsrc/org/mozilla/javascript/tools/shell/Global.java,v  <--  Global.java
new revision: 1.55; previous revision: 1.54
done