Closed Bug 399199 Opened 14 years ago Closed 14 years ago

jsfile.c fails to compile

Categories

(Core :: JavaScript Engine, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jeff.watkins.spam, Unassigned)

Details

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-gb) AppleWebKit/523.10.1 (KHTML, like Gecko) Version/3.0.4 Safari/523.10
Build Identifier: Latest CVS source

jsfile.c: In function ‘file_toURL’:
jsfile.c:2062: error: too many arguments to function ‘js_NewString’



Reproducible: Always

Steps to Reproduce:
1. Add the following lines to Makefile.ref near the top:
    #BEGIN: JW
    JS_HAS_FILE_OBJECT= 1
    JS_THREADSAFE=1
    DIST=../../target/dist
    #END: JW

2. Make clean (make -f Makefile.ref clean)
3. Make


Actual Results:  
It didn't compile.

Expected Results:  
Compiling would have been nice.

I'll post a patch as soon as this bug is filed.
I couldn't figure out how to make CVS spit out a patch file, so I used diff instead.
Use |cvs diff -up8|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Thanks, I'm more a subversion guy...
Attachment #284178 - Attachment is obsolete: true
Comment on attachment 284179 [details] [diff] [review]
Proper CVS patch for jsfile.c

Trivial patch.
Attachment #284179 - Flags: review+
Attachment #284179 - Flags: approval1.9?
Comment on attachment 284179 [details] [diff] [review]
Proper CVS patch for jsfile.c

No need for an approval as the file is not a part of browser builds.
Attachment #284179 - Flags: approval1.9?
jsfile.c: 3.49
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.