Array.toString() and toLocaleString() improvements from KDE KJS

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Harri Porten]

I have problems reaching KDE's websvn server so I'm posting patches by linking  
to the CVS mail archive:  
  
The tests I added in  
   
 http://lists.kde.org/?l=kde-commits&m=112241237705027&w=2  
  
demonstrate a set of problems that the current JSC (and also KDE's previous)  
implementation of Array.toString() and toLocaleString() has. The patch I added  
  
 http://lists.kde.org/?l=kde-commits&m=112241230116304&w=2 
 
tries to implement the behavior as I understand it. Maybe there's more to it. 
Could well be that the code should be factored out. 
 
Would appreciate if possible changes to the patch would be fed back to KDE. 
Thanks.

[Geoffrey Garen]

Created attachment 3123 [details]
Patch

Uploaded JSC patch.

[Geoffrey Garen]

Created attachment 3124 [details]
fast/js/toString-overrides.html

Uploaded layout test.