Add read() as a synonym for snarf() in the js shell

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
trivial
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Paul Biggar, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
Created attachment 463491 [details] [diff] [review]
Patch for the 'read' function.

In other JS shells (v8, jsc), the function to read a file into a string is called 'read'. We call it "snarf".

This adds a synonym called "read". This makes one of the fixes for bug 580532 simpler to send upstream to webkit.

There don't seem to be tests for the shell, so tested by hand.
(Reporter)

Updated

7 years ago
Blocks: 580532

Updated

7 years ago
Attachment #463491 - Flags: review?(jim)

Updated

7 years ago
Attachment #463491 - Flags: review?(jim) → review+
(Reporter)

Comment 1

7 years ago
This was accidentally committed as part of bug 578158, but it was decided to leave it in since it was approved anyway.
Whiteboard: fixed-in-tracemonkey
(Reporter)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.