Closed Bug 302849 Opened 19 years ago Closed 19 years ago

Setting locale in js shell

Categories

(Core :: JavaScript Engine, enhancement)

x86
Linux
enhancement
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: igor, Assigned: igor)

Details

Attachments

(1 file)

Currently js shell does not call setlocale which prevents testing
locale-sensitive behavior of functions like toLocaleString() in the shell. I
suggest to call setlocale(LC_ALL, "") at the shell startup so shell behavior
would be sensitive to locale-specific environment.
Attached patch ImplementationSplinter Review
Patch for js.c to call setlocale(LC_ALL, "") at startup.
Comment on attachment 191127 [details] [diff] [review]
Implementation

Is it possible to apply this to the trunk?
Attachment #191127 - Flags: review?(shaver)
Comment on attachment 191127 [details] [diff] [review]
Implementation

As long as we don't need to condition that for any platforms that lack
setlocale, r=shaver
Attachment #191127 - Flags: review?(shaver) → review+
Man, I would love this in the browser to allow me to set locales for various tests.
Assignee: general → igor.bukanov
I committed the fix: I did not have write access at time of getting review+ and then I forgot about the patch for 4 months :(
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
verified checkin.
Status: RESOLVED → VERIFIED
Flags: testcase-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: