=== Safari-111 === | |
2003-10-22 Maciej Stachowiak <mjs@apple.com> | |
Fix broken build. | |
* kjs/simple_number.h: | |
2003-10-22 Maciej Stachowiak <mjs@apple.com> | |
Merged 64-bit compilation fixes, and fixes for handling negative 0 | |
from upstream kjs. | |
* kjs/internal.cpp: | |
* kjs/simple_number.h: | |
(KJS::SimpleNumber): fixed constants; added negZero constant. | |
(KJS::SimpleNumber::is): adjusted to use long and not int. | |
(KJS::SimpleNumber::value): ditto. | |
(KJS::SimpleNumber::fits): ditto; also don't allow -0 to fit, so | |
we don't lose the distinction between -0 and +0. | |
(KJS::SimpleNumber::make): adjusted to use long. | |
2003-10-18 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3367015 -- interdependent variable declarations in for loop don't work (they go backwards) | |
* kjs/nodes.h: (KJS::ForNode::ForNode): Add a new overload of the constructor for when the | |
first parameter is a variable declaration list. Call reverseList as we do in other constructors | |
that take lists that are built backwards. | |
* kjs/nodes.cpp: (ForNode::reverseList): Added. New helper function. | |
=== Safari-110 === | |
=== Safari-109 === | |
2003-10-06 Darin Adler <darin@apple.com> | |
* kjs/create_hash_table: Remove stray semicolon. | |
* kjs/array_object.lut.h: | |
* kjs/date_object.lut.h: | |
* kjs/lexer.lut.h: | |
* kjs/math_object.lut.h: | |
* kjs/number_object.lut.h: | |
* kjs/string_object.lut.h: | |
Regenerated. | |
=== Safari-108 === | |
2003-10-02 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3441656 -- constructor bad for objs created w/ function as prototype (www.moock.org/asdg/codedepot) | |
* kjs/nodes.cpp: (FuncDeclNode::processFuncDecl): Set up the constructor as | |
as specified in the JavaScript spec. We were already doing this right in the | |
other place we make functions. | |
2003-09-30 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
Rolled in Harri Porten's change to accept non-breaking space in JavaScript. | |
* kjs/lexer.cpp: (Lexer::isWhiteSpace): Accept 00A0 as "whitespace". | |
2003-09-25 Maciej Stachowiak <mjs@apple.com> | |
Roll out build system change since it did not actually work. :-( | |
* JavaScriptCore.pbproj/project.pbxproj: | |
* Makefile.am: | |
2003-09-25 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
* JavaScriptCore.pbproj/project.pbxproj: Don't hack install name. Instead | |
of embedding into Safari, embed into WebKit as sub-umbrella. | |
* Makefile.am: Don't forget to rebuild if the user removes | |
JavaScript.framework from symroots manually. | |
=== Safari-107 === | |
2003-09-24 Darin Adler <darin@apple.com> | |
Reviewed by Ken. | |
- fixed 3421107 -- some dates that other browsers can parse can't be parsed by KJS's Date.parse() | |
* kjs/date_object.cpp: (KJS::KRFCDate_parseDate): Added code to be more strict about month names, | |
to allow a time zone after date even if the date omits the time, and to understand AM and PM. | |
2003-09-22 Darin Adler <darin@apple.com> | |
* JavaScriptCore.pbproj/project.pbxproj: Rename Mixed build style to OptimizedWithSymbols. | |
2003-09-22 Darin Adler <darin@apple.com> | |
Reviewed by Ken. | |
* kjs/config.h: Added HAVE_SYS_PARAM_H, since KJS does look for this header, and we do | |
indeed have it. Just something I noticed in passing while cleaning up configure.in. | |
2003-09-20 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3419380 -- JavaScript Date.getTimezoneOffset is off by one hour (during daylight savings) | |
* kjs/date_object.cpp: (DateProtoFuncImp::call): The daylight savings correction | |
in here was incorrect. Perhaps I should have corrected it for the non-BSD case too, | |
but I'm not sure the issue is the same. | |
2003-09-17 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
* kjs/date_object.cpp: Removed our CF-based implementations of gmtime, localtime, | |
mktime, timegm, and time, since they no longer have the slow "hit the filesystem | |
every time" behavior. | |
=== Safari-100 === | |
=== Safari-99 === | |
=== Safari-98 === | |
=== Safari-97 === | |
=== Safari-96 === | |
2003-08-27 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John | |
- fixed rdar://problem/3397316 - sherlock crash: KJS::Collector::allocate(unsigned long) | |
* kjs/internal.cpp: | |
(InterpreterImp::InterpreterImp): Hold the lock a bit longer, so | |
the call to initGlobalObject is covered. | |
=== Safari-95 === | |
2003-08-24 Darin Adler <darin@apple.com> | |
Reviewed by John. | |
- fixed 3098350 -- opt. params to date methods are ignored (can't set end date in Exchange/Outlook web cal.) | |
* kjs/date_object.cpp: (DateProtoFuncImp::call): Added code to handle the optional parameters. | |
Strangely, the table of functions already had the right number of parameters listed, but the | |
code to look at the parameter values was missing. | |
=== Safari-94 === | |
2003-08-17 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- fixed 3247528 -- encodeURI missing from JavaScriptCore (needed by Crystal Reports) | |
- fixed 3381297 -- escape method does not escape the null character | |
- fixed 3381299 -- escape method produces incorrect escape sequences ala WinIE, rather than correct ala Gecko | |
- fixed 3381303 -- unescape method treats escape sequences as Latin-1 ala WinIE rather than as UTF-8 ala Gecko | |
- fixed 3381304 -- unescape method garbles strings with bad escape sequences in them | |
* kjs/function.h: Added constants for decodeURI, decodeURIComponent, encodeURI, and | |
encodeURIComponent. | |
* kjs/function.cpp: | |
(encode): Added. New helper function for escape, encodeURI, and encodeURIComponent. | |
(decode): Added. New helper function for unescape, decodeURI, and decodeURIComponent. | |
(GlobalFuncImp::call): Added decodeURI, decodeURIComponent, encodeURI, and encodeURIComponent | |
implementations. Changed escape and unescape to use new helper functions, which fixes | |
the four problems above. | |
* kjs/internal.cpp: (InterpreterImp::initGlobalObject): Add decodeURI, decodeURIComponent, | |
encodeURI, and encodeURIComponent to the global object. | |
* kjs/ustring.h: Added a length to the CString class so it can hold strings with null | |
characters in them, not just null-terminated strings. This allows a null character from | |
a UString to survive the process of UTF-16 to UTF-8 decoding. Added overloads to | |
UString::append, UString::UTF8String, UTF8SequenceLength, decodeUTF8Sequence, | |
convertUTF16OffsetsToUTF8Offsets, and convertUTF8OffsetsToUTF16Offsets. | |
* kjs/ustring.cpp: | |
(CString::CString): Set up the length properly in all the constructors. Also add a new | |
constructor that takes a length. | |
(CString::append): Use and set the length properly. | |
(CString::operator=): Use and set the length properly. | |
(operator==): Use and the length and memcmp instead of strcmp. | |
(UString::append): Added new overloads for const char * and for a single string to make | |
it more efficient to build up a UString from pieces. The old way, a UString was created | |
and destroyed each time you appended. | |
(UTF8SequenceLength): New. Helper for decoding UTF-8. | |
(decodeUTF8Sequence): New. Helper for decoding UTF-8. | |
(UString::UTF8String): New. Decodes from UTF-16 to UTF-8. Same as the function that | |
was in regexp.cpp, except has proper handling for UTF-16 surrogates. | |
(compareStringOffsets): Moved from regexp.cpp. | |
(createSortedOffsetsArray): Moved from regexp.cpp. | |
(convertUTF16OffsetsToUTF8Offsets): New. Converts UTF-16 offsets to UTF-8 offsets, given | |
a UTF-8 string. Same as the function that was in regexp.cpp, except has proper handling | |
for UTF-16 surrogates. | |
(convertUTF8OffsetsToUTF16Offsets): New. Converts UTF-8 offsets to UTF-16 offsets, given | |
a UTF-8 string. Same as the function that was in regexp.cpp, except has proper handling | |
for UTF-16 surrogates. | |
- fixed 3381296 -- regular expression matches with UTF-16 surrogates will treat sequences as two characters | |
* kjs/regexp.cpp: | |
(RegExp::RegExp): Use the new UString::UTF8String function instead a function in this file. | |
(RegExp::match): Use the new convertUTF16OffsetsToUTF8Offsets (and the corresponding | |
reverse) instead of convertCharacterOffsetsToUTF8ByteOffsets in this file. | |
=== Safari-93 === | |
2003-08-14 Vicki Murley <vicki@apple.com> | |
Reviewed by John. | |
* JavaScriptCore.pbproj/project.pbxproj: deleted JavaScriptCore.order from the project. | |
2003-08-14 Vicki Murley <vicki@apple.com> | |
Reviewed by John. | |
* JavaScriptCore.order: Removed. We now link to the order file at /AppleInternal/OrderFiles. | |
* JavaScriptCore.pbproj/project.pbxproj: change sectorder flag to point to /AppleInternal/OrderFiles/JavaScriptCore.order | |
=== JavaScriptCore-92.1 === | |
2003-08-07 Darin Adler <darin@apple.com> | |
Reviewed by John Sullivan. | |
- fixed 3365527 -- subscripting JavaScript strings does not work (leads to hang at www.newmagna.com.au) | |
The JavaScript specification says nothing about this, but other browsers seem to give | |
read-only access to the characters in a string as if the string was an array of characters. | |
* kjs/array_object.cpp: | |
(ArrayInstanceImp::get): Update to use a public toArrayIndex function instead of our own getArrayIndex | |
function, so we can share with string. | |
(ArrayInstanceImp::put): Ditto. | |
(ArrayInstanceImp::hasProperty): Ditto. | |
(ArrayInstanceImp::setLength): Ditto. | |
* kjs/ustring.h: Add toArrayIndex. | |
* kjs/ustring.cpp: (UString::toArrayIndex): Added. Implements the rule from array. | |
* kjs/identifier.h: Add a forwarding function so we can use toArrayIndex. | |
* kjs/string_object.cpp: | |
(StringInstanceImp::get): Return a single character string if the property name is an array index. | |
(StringInstanceImp::hasProperty): Return true for property names that are suitable array indices. | |
* JavaScriptCore.pbproj/project.pbxproj: Let Xcode be Xcode. | |
=== Safari-92 === | |
2003-08-07 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
- fixed 3366975 - repro hang in KJS::Value::Value entering text at eil.com | |
* kjs/string_object.cpp: | |
(StringProtoFuncImp::call): When doing a match against a regexp | |
with the global flag set, make sure to return null rather than an | |
empty array when there is no match. This is what other browsers do. | |
2003-08-05 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John. | |
* kjs/list.cpp: | |
(List::copyTail): Test for loop termination with < instead of !=, | |
since i starts at 1 but size could be 0. Do the same for the other | |
loop for consistency's sake. | |
2003-08-01 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John. | |
- fixed 3222621 - Cryptic "anonymous function hack" messages in console (10.2.4) | |
* kjs/lexer.cpp: | |
(Lexer::lex): Remove useless debug spew. | |
=== Safari-91 === | |
2003-07-30 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed problem where some JavaScriptCore symbols had no namespace or prefix | |
* kjs/grammar.y: Added a define for yylloc to make it use the kjs prefix. | |
This is the same thing done for the rest of the symbols automatically by yacc, | |
but for some reason it's not done for yyloc. Also make automatic() function static. | |
* kjs/grammar.cpp: Regenerated. | |
* kjs/lexer.cpp: Use kjsyylloc instead of yyloc. | |
* pcre/pcre.h: Add defines to prepend kjs prefixes for all the PCRE functions. | |
2003-07-30 Darin Adler <darin@apple.com> | |
* Makefile.am: Include the subdirectory with the PCRE code in it. | |
2003-07-30 John Sullivan <sullivan@apple.com> | |
- JavaScriptCore part of fix for 3284525 -- AutoFill fills in | |
only e-mail address field of New Account form on Apple Store Japan | |
Reviewed by Darin | |
* JavaScriptCore.pbproj/project.pbxproj: | |
Mark pcre.h as a Private header | |
2003-07-28 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Richard. | |
- fixed 3240814 - LEAK: 120 byte leak in JavaScript parser in Sherlock Movies channel | |
* kjs/internal.cpp: | |
(Parser::parse): ref() and deref() the program node, to make sure to clean up properly, | |
before deleting it. | |
(InterpreterImp::checkSyntax): Likewise. | |
=== Safari-90 === | |
2003-07-22 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John. | |
Remove -seg_addr_table_filename to fix build. | |
* JavaScriptCore.pbproj/project.pbxproj: | |
2003-07-17 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John. | |
- fixed 3330344 - Please change allowable client to "JavaScriptGlue" from "JSGlue" | |
* JavaScriptCore.pbproj/project.pbxproj: Changed allowable client | |
to "JavaScriptGlue" | |
2003-07-13 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- do some improvements Maciej suggested while reviewing the array index change | |
* kjs/array_object.cpp: | |
(getArrayIndex): Return a flag to say whether the index was value separately, to avoid | |
in-band signalling. | |
(ArrayInstanceImp::get): Update for new getArrayIndex parameters. | |
(ArrayInstanceImp::put): Ditto. | |
(ArrayInstanceImp::hasProperty): Ditto. | |
(ArrayInstanceImp::setLength): Ditto. | |
* kjs/ustring.cpp: (UString::toStrictUInt32): Check for overflow in a way that avoids doing | |
a divide every time through the loop. But note that it adds an extra branch to the loop. | |
I wonder which is worse. | |
2003-07-12 Darin Adler <darin@apple.com> | |
Fixed broken build. | |
* kjs/identifier.h: Add toULong back. It's still used in WebCore (and maybe in JavaScriptGlue, | |
for all I know). | |
2003-07-12 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3272777 -- array object indices treated as integers by Safari, but as strings in other web browsers | |
JavaScriptCore did not implement the proper rule for what an array index is. | |
* kjs/array_object.cpp: | |
(getArrayIndex): Added. Implements the rule from the specification, which also provides a handy | |
"not an array index" value of 2^32-1. | |
(ArrayInstanceImp::get): Use getArrayIndex. | |
(ArrayInstanceImp::put): Ditto. | |
(ArrayInstanceImp::hasProperty): Ditto. | |
(ArrayInstanceImp::setLength): Ditto. | |
* kjs/identifier.h: Removed now-unused toULong, and added toStrictUInt32, in both cases forwarding | |
functions that forward to UString. | |
* kjs/ustring.h: Added toStringUInt32. | |
* kjs/ustring.cpp: (UString::toStrictUInt32): Added. Converts a string to a 32-bit unsigned integer, | |
and rejects any string that does not exactly match the way the integer would be formatted on output. | |
This is the rule documented in the ECMA language standard. | |
=== Safari-89 === | |
2003-07-10 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
- fixed 3302021 - v74 and v85 hang with http://e-www.motorola.com/ | |
The crux of this was saving and restoring the prototype objects | |
for all the standard types when saving and restoring for the page | |
cache. | |
* kjs/internal.cpp: | |
(InterpreterImp::saveBuiltins): | |
(InterpreterImp::restoreBuiltins): | |
* kjs/internal.h: | |
* kjs/interpreter.cpp: | |
(Interpreter::saveBuiltins): | |
(Interpreter::restoreBuiltins): | |
(SavedBuiltins::SavedBuiltins): | |
(SavedBuiltins::~SavedBuiltins): | |
* kjs/interpreter.h: | |
* kjs/property_map.cpp: | |
2003-07-07 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John. | |
- fixed 3295916 - b/c JavaScriptCore and WebCore are installing in wrong location, private headers are public | |
* WebCore.pbproj/project.pbxproj: Install in WebKit.framework/Versions/A/Frameworks. | |
=== Safari-88 === | |
2003-07-02 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Ken. | |
- fixed 3096961 - JavaScriptCore should link only to what it uses, shouldn't drag in Cocoa.framework | |
* JavaScriptCore.pbproj/project.pbxproj: Don't link Cocoa.framework; | |
just pull in CoreFoundation and CoreServices. | |
* kjs/date_object.cpp: Include CoreServices.h instead of Carbon.h | |
(the stuff we want is in CarbonCore). | |
2003-06-20 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- improved the property map sorting technique so that the indices | |
are separate for each property map, and also preserve the ordering | |
when property maps are saved and restored | |
* kjs/property_map.cpp: | |
(PropertyMap::put): Don't bother setting the index for _singleEntry, since there's | |
no need to sort a single entry. Use the per-table lastIndexUsed instead of a global. | |
(PropertyMap::expand): Don't use the index (uninitialized now) out of a _singleEntry | |
when putting it in a newly-created map; just use 0. Compute a value for the new map's | |
lastIndexUsed as we walk through the elements we are adding to it (using the same old | |
indices from the old map). | |
=== Safari-85.1 === | |
=== Safari-85 === | |
2003-06-13 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3178438 -- return elements in order of addition in for..in loop (other browsers seem to) | |
- fixed 3292067 -- REGRESSION (64-65): albertsons.com "Shop A to Z" menus are not sorted alphabetically | |
* kjs/property_map.h: Add index field to hash table entry and index parameter to insert function. | |
* kjs/property_map.cpp: | |
(PropertyMap::put): Set an index for new map entries to an ever-increasing number based on a global. | |
(PropertyMap::insert): Take an index parameter. | |
(PropertyMap::expand): Preserve the indices as we rehash the table. | |
(comparePropertyMapEntryIndices): Added. Compares two property map entries by index. | |
(PropertyMap::addEnumerablesToReferenceList): Sort the proprty map entries by index before adding | |
them to the reference list. | |
=== Safari-84 === | |
2003-06-10 Vicki Murley <vicki@apple.com> | |
Reviewed by john. | |
* JavaScriptCore.order: new order file for 1.0 | |
=== Safari-83 === | |
2003-06-04 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3224031 -- can't search at rakuten.co.jp b/c of extra characters inserted by regexp replace (8-bit char) | |
Use PCRE UTF-8 regular expressions instead of just chopping off high bytes. | |
* kjs/regexp.h: Redo field names, remove some unused stuff. | |
* kjs/regexp.cpp: | |
(convertToUTF8): Added. | |
(compareStringOffsets): Added. | |
(createSortedOffsetsArray): Added. | |
(convertCharacterOffsetsToUTF8ByteOffsets): Added. | |
(convertUTF8ByteOffsetsToCharacterOffsets): Added. | |
(RegExp::RegExp): Set the PCRE_UTF8 flag, and convert the UString to UTF-8 instead of | |
using ascii() on it. | |
(RegExp::~RegExp): Remove unneeded if statement (pcre_free is 0-tolerant as free is). | |
(RegExp::match): Convert the UString to UTF-8 and convert the character offsets to and | |
from UTF-8 byte offsets. Also do fixes for the "no offset vector" case so we get the | |
correct position and matched string. | |
* JavaScriptCore.pbproj/project.pbxproj: Add a PCRE header that was missing before. | |
=== Safari-82 === | |
=== Safari-81 === | |
2003-05-21 Vicki Murley <vicki@apple.com> | |
Reviewed by john | |
- fixed 3234553: Safari and its frameworks should link using order files | |
* JavaScriptCore.order: Added. | |
* JavaScriptCore.pbproj/project.pbxproj: set SECTORDER_FLAGS = -sectorder __TEXT __text JavaScriptCore.order | |
=== Safari-80 === | |
2003-05-19 Maciej Stachowiak <mjs@apple.com> | |
- fixed 3261096 - Make WebKit an umbrella framework | |
* JavaScriptCore.pbproj/project.pbxproj: In a B&I build, compile as a | |
sub-umbrella of WebKit. | |
2003-05-16 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Ken. | |
- fixed 3254063 - REGRESSION: hang in KJS PropertyMap with many items in iDisk pictures folder | |
* kjs/property_map.cpp: | |
(PropertyMap::expand): Fixed to maintain key count properly - otherwise the hashtable | |
could get completely full, resulting in disaster. | |
(PropertyMap::checkConsistency): Fixed compilation. Fixed to know about deleted | |
sentinel. Fixed to search with double-hashing instead of linear probing. | |
=== Safari-79 === | |
2003-05-15 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Chris. | |
- fixed 3259673 - REGRESSION: marvel.com thinks I don't have the flash plugin any more | |
* kjs/nodes.cpp: | |
(ContinueNode::execute): Return a Continue completion, not a Break | |
completion, in the normal non-exception case. | |
2003-05-12 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
- fixed 3254484 - Add a way to print JavaScript exceptions to the console via the debug menu | |
- improved JavaScript error message format | |
* kjs/error_object.cpp: | |
(ErrorProtoFuncImp::call): Include line number in toString output. | |
* kjs/internal.cpp: | |
(Parser::parse): Remove redundant fprintf. | |
* kjs/interpreter.cpp: | |
(Interpreter::evaluate): Log if the flag is on. Include filename in log output. | |
(Interpreter::shouldPrintExceptions): Check the global flag. | |
(Interpreter::setShouldPrintExceptions): Set the global flag. | |
* kjs/interpreter.h: | |
* kjs/nodes.cpp: | |
(Node::throwError): Add variants that include value and expression or label in format. | |
(NewExprNode::evaluate): Improve error message. | |
(FunctionCallNode::evaluate): Improve error message. | |
(RelationalNode::evaluate): Improve error message. | |
(ContinueNode::execute): Improve error message. | |
(BreakNode::execute): Improve error message. | |
(LabelNode::execute): Improve error message. | |
* kjs/nodes.h: | |
=== Safari-78 === | |
2003-05-07 Vicki Murley <vicki@apple.com> | |
Reviewed by darin. | |
- modify the Mixed build style to build optimized with symbols | |
* JavaScriptCore.pbproj/project.pbxproj: removed OPTIMIZATION_CFLAGS | |
2003-05-05 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Don. | |
- fixed 3239961 - www.phiffer.com doesn't work; uses "var top; top = n;" | |
* kjs/nodes.cpp: | |
(VarDeclNode::evaluate): Check if the property exists with | |
getDirect() instead of hasProperty(). | |
=== Safari-77 === | |
2003-04-29 Darin Adler <darin@apple.com> | |
Reviewed by John. | |
- fixed 2959353 -- eliminate globally initialized objects from JavaScriptCore | |
* JavaScriptCore.pbproj/project.pbxproj: Added fpconst.cpp. | |
* kjs/fpconst.cpp: Added. Defines KJS::NaN and KJS::Inf in a way that does not require a | |
framework init routine. | |
* kjs/identifier.h: Use a new KJS_IDENTIFIER_EACH_GLOBAL macro so we can do things to | |
the entire set of identifiers easily. Also added an init function that sets up these globals | |
in a way that does not require a framework init routine. | |
* kjs/identifier.cpp: (Identifier::init): Initialize the property ane globals in a way that | |
does not require a framework init routine. | |
* kjs/internal.cpp: (InterpreterImp::initGlobalObject): Call Identifier::init. | |
* kjs/ustring.h: Remove UChar::null and UString::null, and add UString::null(). We can't have | |
a global object of a class that has a constructor if we want to avoid framework init routines, | |
and luckily very little code relies on these. | |
* kjs/ustring.cpp: | |
(UCharReference::ref): Use our own global specific to this function rather than returning | |
UChar::null when past the end of the string. This is dangerous because if the caller modifies | |
it, that affects what all subsequent callers will see. | |
(UString::Rep::create): Added assertions. | |
(UString::UString): Got rid of code here that used to set up UString::null. | |
(UString::null): Added. Returns a global null string, and can be used in some of the places | |
where we used to use the UString::null global. | |
(UString::operator[]): Fixed case where this used to return UChar::null to return '\0' instead. | |
* kjs/regexp.cpp: (RegExp::match): Change uses of UString::null to UString::null(). | |
2003-04-25 Darin Adler <darin@apple.com> | |
- fixed 3241344 -- REGRESSION: top of page missing on wired.com and cnn.com | |
Caused by the ResolveNode speedup. Roll it out until I can figure out why. | |
* kjs/nodes.cpp: (ResolveNode::evaluate): Go back to using evaluateReference. | |
2003-04-25 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- a couple improvements that give a 6.6% speedup on iBench JavaScript | |
* kjs/nodes.cpp: (ResolveNode::evaluate): Don't use evaluateReference. | |
* kjs/object.cpp: (ObjectImp::get): Do the prototype work with the ValueImp, not a wrapper. | |
Contributes a tiny bit to the speedup, but cleaner anyway. | |
(ObjectImp::hasProperty): Same thing here. | |
2003-04-25 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- move from linear probing to double hashing, gives an 0.7% speedup in iBench JavaScript | |
* kjs/property_map.h: Remove the hash function. | |
* kjs/property_map.cpp: Added statistics for rehashes and removes. | |
Moved from linear probing to double hashing, using the hash modulo | |
(table size minus one) plus one for the probing distance. | |
* kjs/ustring.h: Use unsigned instead of int for hash function result. | |
=== Safari-75 === | |
2003-04-18 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Ken. | |
Improved List pool for 3% speed improvement on cvs-js-ibench | |
* kjs/list.cpp: Replaced the roving cursor with a free list and | |
raised the high water mark to 384. | |
2003-04-12 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Don. | |
- JavaScriptCore part of fix for 3158769 - JavaScript triggers not as async as they used to be | |
Added a way to get the current interpreter lock count, so Sherlock | |
can unlock the interpreter inside JS method implementations that | |
spend a long time waiting for I/O, allowing more efficient | |
multi-threaded operation. | |
* kjs/internal.cpp: | |
(lockInterpreter): | |
(unlockInterpreter): | |
(InterpreterImp::lock): | |
(InterpreterImp::lockCount): | |
* kjs/internal.h: | |
* kjs/interpreter.cpp: | |
(Interpreter::lockCount): | |
* kjs/interpreter.h: | |
=== Safari-73 === | |
=== Safari-72 === | |
=== Safari-71 === | |
2003-03-31 Darin Adler <darin@apple.com> | |
* English.lproj/InfoPlist.strings: Changed "1.0 Beta" to "1.0 Beta 2". | |
* JavaScriptCore.pbproj/project.pbxproj: Changed "1.0 Beta" to "1.0 Beta 2". | |
=== Safari-69 === | |
2003-03-24 Trey Matteson <trey@apple.com> | |
Pass -seg_addr_table_filename <FILENAME> to ld. This makes our frameworks in | |
SYMROOT actually work for symbol resolution because they will have the correct | |
prebinding address. It also fixes obscure B&I problems with prebinding | |
reported by Matt Reda. | |
Note the reason all this is tricky for our projects is that we have a different | |
install location for Jaguar and Panther. The purpose of this arg is to declare | |
at link time our eventual location, which allows the prebinding address to be | |
found in /AppleInternal/Developer/seg_addr_table. We use a funky back-tick | |
expression within OTHER_LDFLAGS to get a conditional value depending on the | |
build train we are in. | |
This can all go away once we only build on Panther and don't embed the | |
frameworks inside the Safari.app wrapper. | |
In addition I fixed the OTHER_LDFLAGS settings in our build styles to be | |
additive instead of overriding, so we have the args we used for B&I in force | |
when building outside of B&I. | |
Reviewed by Maciej. | |
* JavaScriptCore.pbproj/project.pbxproj: | |
=== Safari-68 === | |
2003-03-16 Trey Matteson <trey@apple.com> | |
3198135 - need to fix our projects so SYMROOT is not stripped | |
Tweaked stripping options: B&I build does not COPY_PHASE_STRIP. | |
Deployment build still does. | |
We strip manually as part of the install that we do ourselves. | |
Reviewed by Maciej. | |
* JavaScriptCore.pbproj/project.pbxproj: | |
=== Safari-67 === | |
=== Safari-66 === | |
2003-03-10 Darin Adler <darin@apple.com> | |
Reviewed by Ken. | |
- fixed 3193099 -- date parsing can't handle the time zone format that date formatting produces | |
* kjs/date_object.cpp: (KJS::KRFCDate_parseDate): Allow a "GMT" prefix before the time zone offset. | |
=== Safari-65 === | |
2003-03-04 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- got rid of some framework initialization (working on bug 2959353) | |
* kjs/identifier.h: Turn Identifier:null into Identifier:null(). | |
* kjs/identifier.cpp: Removed Identifier:null and added Identifier:null(). | |
* kjs/internal.cpp: Made NaN_Bytes and Inf_Bytes const. | |
* kjs/completion.h: Use Identifier:null() instead of Identifier:null. | |
* kjs/function.h: Ditto. | |
* kjs/function_object.cpp: (FunctionObjectImp::construct): Ditto. | |
* kjs/nodes.cpp: (FuncExprNode::evaluate): Use Identifier:null() instead of Identifier:null. | |
2003-03-02 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Trey. | |
- fixed 3158833 - ebay prefs page is so slow, it seems like a hang. | |
92% speed improvement on ebay prefs page. | |
1% speed improvement on js-ibench and js-performance plt suites. | |
There were a couple of problems with the identifier hash table that | |
I fixed: | |
* kjs/identifier.cpp: | |
(void Identifier::remove): Adjust the shrink threshold to avoid | |
constantly growing and shrinking. | |
* kjs/ustring.cpp: | |
(UString::Rep::computeHash): Use a better hash function that | |
avoids collisions for obvious data sets. | |
=== Safari-64 === | |
=== Safari-63 === | |
2003-02-26 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
- fixed 3156705 - REGRESSION: javascript menus improperly placed at umich.edu store | |
* kjs/nodes.cpp: | |
(StatListNode::execute): If the first statement's completion is | |
not normal, return immediately. | |
2003-02-21 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- fixed 3142355 -- nil-deref in CFTimeZoneCopyAbbreviation | |
The real problem wasn't with the current time zone, but with the UTC time zone. | |
The poor sod had a broken /usr/share/zoneinfo directory, with a 0-byte-long UTC file. | |
* kjs/date_object.cpp: (UTCTimeZone): Use CFTimeZoneCreateWithTimeIntervalFromGMT(NULL, 0.0) | |
to get the universal time zone instead of getting it by name. | |
=== Safari-62 === | |
2003-02-18 Darin Adler <darin@apple.com> | |
Reviewed by Trey and Ken. | |
- fixed 3142355 -- nil-deref in CFTimeZoneCopyAbbreviation | |
Although I can't reproduce this bug, it seems that it's caused by CFTimeZoneCopyDefault returning NULL. | |
I'm almost certain that the UTC time zone will be created successfully in this case, so I'll just use that. | |
* kjs/date_object.cpp: | |
(UTCTimeZone): Added. Gets the UTC time zone (once in a global). | |
(CopyLocalTimeZone): Added. Gets the local time zone, but falls back to UTC. | |
(gmtimeUsingCF): Use UTCTimeZone. | |
(localtimeUsingCF): Use CopyLocalTimeZone. | |
(mktimeUsingCF): Use CopyLocalTimeZone. | |
(timegmUsingCF): Use UTCTimeZone. | |
2003-02-12 Darin Adler <darin@apple.com> | |
Reviewed by Dave. | |
- fixed 3145442 -- toString(16) is not working, causing non-ASCII characters in mac.com homepage to be munged | |
* kjs/number_object.cpp: (NumberProtoFuncImp::call): Add handling for toString with a radix other than | |
10 passed as an argument. | |
2003-02-11 Trey Matteson <trey@apple.com> | |
Set -seg1addr in our build styles, but not for the B&I build. | |
This makes our SYMROOTS from B&I usable to determine symbols from crash | |
logs from the field. | |
Also nuked DeploymentFat build style. | |
Reviewed by Ken. | |
* JavaScriptCore.pbproj/project.pbxproj: | |
2003-02-04 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
* JavaScriptCore.pbproj/project.pbxproj: Updated to build the framework | |
standalone instead of embedded when doing a B&I build for Panther. | |
=== Safari-55 === | |
2003-01-29 Darin Adler <darin@apple.com> | |
Reviewed by John. | |
* kjs/scope_chain.cpp: Rolled out the fix to bug 3137084. | |
It caused a massive storage leak, and probably didn't even fix the bug. | |
2003-01-28 Darin Adler <darin@apple.com> | |
Reviewed by Ken. | |
- fixed 3157318 -- hang at time zone page after clicking on map at www.time.gov | |
* kjs/date_object.cpp: (KJS::KRFCDate_parseDate): Allow a comma after the day. | |
Given how this code is structured, it allows commas in unwanted contexts too, but | |
that's almost certainly harmless. | |
2003-01-28 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- fixed 3144918 -- Can't drill down multiple levels of categories when selling on ebay | |
if first item in list is chosen | |
The bug was caused by having array values in the property map past the storageLength cutoff | |
in an array object; those values would not be seen when you do a get. | |
* kjs/array_object.cpp: | |
(ArrayInstanceImp::put): Implement a new rule for resizing the storage that is independent | |
of the length. The old rule would sometimes make the storage very big if you added two elements | |
in a row that both had large, but consecutive indexes. This eliminates any cases where we | |
make sparse entries in the property map below the sparse array cutoff. | |
(ArrayInstanceImp::resizeStorage): Don't ever make storage size bigger than the cutoff unless | |
the caller specifically requests it. | |
(ArrayInstanceImp::setLength): Change this so it only makes the storage smaller, never larger. | |
We will actually enlarge the storage when putting elements in. | |
2003-01-25 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
* kjs/Makefile.am: Add dependencies so the .lut.h files get rebuilt if the script changes. | |
=== Safari-54 === | |
2003-01-22 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- fixed 3137084 -- Many non-reproducible crashers in ContextImp::mark / ScopeChain::mark | |
* kjs/scope_chain.cpp: (ScopeChain::push): Add assertion. | |
(ScopeChain::release): Fix while loop so that it decrements refCount of the first node in | |
the chain too. | |
2003-01-21 Darin Adler <darin@apple.com> | |
- correct our copyrights to 2003; copyright is based on year of publication, not year worked on | |
2003-01-16 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
- made minor tweaks to work better with Mozilla's JavaScript tests. | |
* kjs/testkjs.cpp: | |
(VersionFunctionImp::call): Implemented | |
(main): Ignore files named -f (hack to match -f <filename syntax | |
that moz JavaScript tests expect). Also use return code 3 instead | |
of 1 for uncaught exception. | |
2003-01-16 Darin Adler <darin@apple.com> | |
* kjs/number_object.cpp: (NumberObjectImp::construct): | |
Fix build, remove stray space. | |
2003-01-16 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- rolled in a change from the KJS folks | |
* kjs/number_object.h: Use ObjectImp *, not Object, for the proto. | |
* kjs/number_object.cpp: | |
(NumberInstanceImp::NumberInstanceImp): Use ObjectImp *, not Object, for the proto. | |
(NumberPrototypeImp::NumberPrototypeImp): Pass ObjectImp. | |
(NumberObjectImp::construct): Use ObjectImp. | |
=== Safari-52 === | |
2003-01-14 Darin Adler <darin@apple.com> | |
Reviewed by Ken. | |
- rolled in a change from the KJS folks | |
Fixes a bug where the date functions would not accept non-strings. | |
And provides a bit of a speedup. | |
* kjs/date_object.h: Change parameter type for parseDate. | |
* kjs/date_object.cpp: | |
(DateObjectFuncImp::call): Always call toString, don't check the type. | |
(KJS::parseDate): Take a UString parameter, not a String parameter. | |
2003-01-13 Darin Adler <darin@apple.com> | |
* kjs/ustring.h: Fix spelling of occurrence. | |
2003-01-12 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- turned more recursion into iteration, and fixed some backwards stuff | |
* kjs/grammar.y: Use the normal idiom for CaseClauses and FormalParameterList | |
rather than using append(). | |
* kjs/grammar.cpp: Regenerated. | |
* kjs/nodes.h: Change ClauseListNode and ParameterNode to use the normal idiom, | |
and got rid of append methods. Also added friend declarations and calls to reverseList(). | |
* kjs/nodes.cpp: | |
(StatListNode::ref): Iteration, not recursion. | |
(StatListNode::deref): Iteration, not recursion. | |
(StatListNode::execute): Iteration, not recursion. | |
(StatListNode::processVarDecls): Iteration, not recursion. | |
(CaseClauseNode::reverseList): Added. | |
(ClauseListNode::ref): Iteration, not recursion. | |
(ClauseListNode::deref): Iteration, not recursion. | |
(ClauseListNode::processVarDecls): Iteration, not recursion. | |
(CaseBlockNode::reverseLists): Added. | |
(ParameterNode::ref): Iteration, not recursion. | |
(ParameterNode::deref): Iteration, not recursion. | |
(FuncDeclNode::reverseParameterList): Added. | |
(FuncExprNode::reverseParameterList): Added. | |
(SourceElementsNode::ref): Iteration, not recursion. | |
(SourceElementsNode::deref): Iteration, not recursion. | |
(SourceElementsNode::execute): Use variable name of n to match other functions. | |
(SourceElementsNode::processFuncDecl): Ditto. | |
(SourceElementsNode::processVarDecls): Ditto. | |
* kjs/nodes2string.cpp: | |
(SourceStream::operator<<): Used a switch statement for a bit of added clarity. | |
(ElementNode::streamTo): Iteration, not recursion. | |
(PropertyValueNode::streamTo): Iteration, not recursion. | |
(ArgumentListNode::streamTo): Iteration, not recursion. | |
(StatListNode::streamTo): Iteration, not recursion, and fixed order. | |
(VarDeclListNode::streamTo): Iteration, not recursion. | |
(ClauseListNode::streamTo): Used for statement to match other functions. | |
(CaseBlockNode::streamTo): Used for statement to match other functions. | |
(ParameterNode::streamTo): Iteration, not recursion. | |
(SourceElementsNode::streamTo): Iteration, not recursion, and fixed order that has been | |
backwards since I changed how this works in nodes.cpp. | |
2003-01-11 Darin Adler <darin@apple.com> | |
Reviewed by John. | |
- changes inspired by things I noticed reviewing diffs vs. KDE when preparing the tarball | |
* kjs/function.cpp: (GlobalFuncImp::call): Use strtol when strtoll is | |
not available. Do #ifndef NDEBUG, not #if !NDEBUG. | |
* kjs/function.h: Do #ifndef NDEBUG, not #if !NDEBUG. | |
* kjs/internal.cpp: | |
(InterpreterImp::initGlobalObject): Do #ifndef NDEBUG, not #if !NDEBUG. | |
(KJS::printInfo): Remove case for ListType and remove default case that just | |
ends up suppressing the "missing case" warning and does no good. | |
* kjs/interpreter.cpp: (Interpreter::evaluate): Do #ifndef NDEBUG, not #if !NDEBUG. | |
* kjs/nodes.cpp: | |
(Node::finalCheck): Fix accidentally-deleted code in an ifdef we never compile. | |
(FunctionCallNode::evaluate): Remove bogus XXX comment. Maciej put this comment in, | |
and together we determined it's not needed. | |
(TypeOfNode::evaluate): Ditto. | |
* kjs/object.cpp: Remove assert that refers to ListType. | |
* kjs/value.h: Remove ListType. | |
2003-01-09 Darin Adler <darin@apple.com> | |
* JavaScriptCore.pbproj/project.pbxproj: Add the year 2003, remove CFBundleIconFile, | |
bump marketing version to 0.8.1 and version to 52u to keep up with the branch, | |
remove CFHumanReadableCopyright, remove NSPrincipalClass. | |
* English.lproj/InfoPlist.strings: Updated to match above changes. | |
2003-01-05 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by no one cause I'm just changing copyright strings. | |
* JavaScriptCore.pbproj/project.pbxproj: Added non-Apple copyrights to | |
copyright strings. | |
* English.lproj/InfoPlist.strings: Likewise. | |
2003-01-05 Darin Adler <darin@apple.com> | |
* JavaScriptCore.pbproj/project.pbxproj: Fix "Apple Compupter" typo. | |
Remove unneeded CFBundleLongVersionString we don't use anywhere else. | |
2003-01-02 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
- fixed 3138213 -- browser hangs trying to open Apple travel site | |
* kjs/date_object.cpp: (timetUsingCF): Check for very-negative year numbers too. | |
=== Alexander-48 === | |
=== Alexander-47 === | |
2002-12-30 Darin Adler <darin@apple.com> | |
Reviewed by Don and Maciej. | |
- follow-on to my fix for 3134693 that fixes one more case of recursion and simplifies further | |
* kjs/grammar.y: Remove SourceElementNode and just use a StatementNode instead. | |
Reverse SourceElements rule so the recursive rule comes first as in the original | |
KJS code (avoids actual parser recursion). | |
* kjs/grammar.cpp: Regenerated. | |
* kjs/grammar.cpp.h: Regenerated. | |
* kjs/grammar.h: Regenerated. | |
* kjs/nodes.h: Make processFuncDecl a virtual function in StatementNode so that we can | |
use a StatementNode instead of a SourceElementNode. Add a call to reverseList in BlockNode | |
to correct the order of the linked list in SourceElementsNode, to replace the technique | |
where we reversed it in the parser. Remove SourceElementNode class, and make the element in | |
SourceElementsNode be a StatementNode instead. | |
* kjs/nodes.cpp: Remove SourceElementNode code. | |
(StatementNode::processFuncDecl): Added empty function. | |
(BlockNode::reverseList): Added. Used to make the SourceElements list ordered correctly. | |
* kjs/nodes2string.cpp: Remove SourceElementNode code. | |
=== Alexander-46 === | |
2002-12-28 Darin Adler <darin@apple.com> | |
Reviewed by Gramps and Ken. | |
Checked in by Ken. | |
- fixed 3134693 -- carsdirect.com crash on used car search, due to large JavaScript array | |
The parser was using recursion to handle many types of lists. | |
This meant that we crashed out of stack space when any of the lists were extra big. | |
I applied the same sort of fix we had already applied a while back for argument lists for | |
all the other types of lists, including the list of ElementNode that was the reason for | |
the crash reported here. | |
* kjs/grammar.y: Removed ElisionNode altogether and just use a count. | |
Use specific node types for PropertyNameAndValueList and PropertyName. | |
* kjs/grammar.cpp: Regenerated. | |
* kjs/grammar.cpp.h: Regenerated. | |
* kjs/grammar.h: Regenerated. | |
* kjs/nodes.h: Elide "ElisionNode", changing objects to keep elision counts instead. | |
Make the ObjectLiteralNode list field be PropertyValueNode, not just Node. | |
Make PropertyValueNode fields have specific types. Add new reverse list functions, calls | |
to those functions in the constructors, and friend declarations as needed so the class | |
that holds the head of a list can reverse the list during parsing. | |
* kjs/nodes.cpp: | |
(ElementNode::ref): Use iteration instead of recursion. Also elide "elision". | |
(ElementNode::deref): Ditto. | |
(ElementNode::evaluate): Use iteration instead of recursion, taking advantage of | |
the fact that the linked list is reversed. Also use the elision count rather than | |
an elision list. | |
(ArrayNode::reverseElementList): Reverse the list so we can iterate normally. | |
(ArrayNode::ref): Elide "elision". | |
(ArrayNode::deref): Ditto. | |
(ArrayNode::evaluate): Use elision count instead of elision list. | |
(ObjectLiteralNode::reverseList): Reverse the list so we can iterate normally. | |
(PropertyValueNode::ref): Use iteration instead of recursion. | |
(PropertyValueNode::deref): Use iteration instead of recursion. | |
(PropertyValueNode::evaluate): Use iteration instead of recursion, taking advantage | |
of the fact that the linked list is reversed. | |
(ArgumentListNode::ref): Change code to match the other similar cases we had to revise. | |
(ArgumentListNode::deref): Ditto. | |
(ArgumentListNode::evaluateList): Ditto. | |
(ArgumentsNode::reverseList): Ditto. | |
(VarDeclListNode::ref): Use iteration instead of recursion. | |
(VarDeclListNode::deref): Ditto. | |
(VarDeclListNode::evaluate): Use iteration instead of recursion, taking advantage | |
of the fact that the linked list is reversed. | |
(VarDeclListNode::processVarDecls): Ditto. | |
(VarStatementNode::reverseList): Reverse the list so we can iterate normally. | |
(FunctionBodyNode::FunctionBodyNode): Use BlockNode as the base class, removing | |
most of the FunctionBodyNode class. | |
* kjs/nodes2string.cpp: | |
(ElementNode::streamTo): Update for using a count for elision, and reverse linking. | |
(ArrayNode::streamTo): Update for using a count for elision. | |
(PropertyValueNode::streamTo): Update for reverse linking. | |
(ArgumentListNode::streamTo): Update for reverse linking. This has been wrong for | |
a while, since we added the reverse a long time ago. | |
(VarDeclListNode::streamTo): Update for reverse linking. | |
(ParameterNode::streamTo): Update for reverse linking. | |
=== Alexander-45 === | |
2002-12-22 Darin Adler <darin@apple.com> | |
Reviewed by Don and John. | |
- fixed 3134449 -- Date.UTC returns NaN (invalid date) | |
Did more testing of the date functions and made them behave like the other browsers. | |
There were three problems: | |
1) We did a validity check that other browsers don't do (hence the NaN). | |
2) We treated passed-in dates as local time even in Date.UTC (hence a wrong result | |
once I fixed the NaN). | |
3) The results of ToUTCString (and ToGMTString) weren't formatted quite the same | |
as other browsers. | |
Also found a couple of silly but unrelated coding mistakes. | |
* kjs/date_object.cpp: | |
(timetUsingCF): Added. Has the guts of mktimeUsingCF, but without the CFGregorianDateIsValid | |
check. Other browsers accept invalid dates. Also takes a time zone parameter. | |
(mktimeUsingCF): Calls timetUsingCF with the current time zone. | |
(timegmUsingCF): Calls timetUsingCF with the UTC time zone. | |
(formatDate): Remove the includeComma flag. | |
(formatDateUTCVariant): Added. For use instead of formatDate with the includeComma flag. | |
Puts the day before the month name. | |
(DateProtoFuncImp::call): Use the new formatDateUTCVariant for ToGMTString and ToUTCString. | |
Without this change the date didn't match other browsers. | |
(DateObjectImp::DateObjectImp): Use UTCPropertyName. Somehow I declared this and didn't use | |
it before. | |
(DateObjectImp::construct): Pass -1 for is_dst literally instead of using invalidDate. | |
Changing this to invalidDate was just a mistake (although no real difference in compiled | |
code since invalidDate is just -1). | |
(DateObjectFuncImp::call): Call timegm for the UTC case instead of mktime. | |
=== Alexander-44 === | |
=== Alexander-43 === | |
2002-12-20 Trey Matteson <trey@apple.com> | |
We now build with symbols the B&I. Deployment builds are without symbols, | |
so it is easy to generate a non-huge app as a one-off. | |
Reviewed by Darin | |
* JavaScriptCore.pbproj/project.pbxproj: | |
=== Alexander-42 === | |
=== Alexander-41 === | |
=== Alexander-40 === | |
2002-12-18 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by John. | |
- fixed 3131171 - Change Alex versions to satisfy both marketing and B&I requirements | |
* English.lproj/InfoPlist.strings: | |
* JavaScriptCore.pbproj/project.pbxproj: | |
2002-12-17 Darin Adler <darin@apple.com> | |
Reviewed by Trey. | |
* JavaScriptCore.pbproj/project.pbxproj: Removed signature. | |
=== Alexander-39 === | |
=== Alexander-38 === | |
2002-12-16 Darin Adler <darin@apple.com> | |
Reviewed by Don and Maciej. | |
- fixed 3129115 -- need Apple copyright added to open source documents | |
* tons of files: Added our copyright to files we modified, and updated all to standard format. | |
- other changes | |
* JavaScriptCore.pbproj/project.pbxproj: Set MACOSX_DEPLOYMENT_TARGET to 10.2. | |
Also removed completion.cpp. | |
* kjs/completion.cpp: Removed. | |
* kjs/completion.h: Made the Completion constructor inline. | |
* kjs/grammar.y: Removed an obsolete "pretend ifdef". No need to put these in APPLE_CHANGES now. | |
=== Alexander-37 === | |
=== JavaScriptCore-37u2 === | |
2002-12-15 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
* JavaScriptCore.pbproj/project.pbxproj: Bump version to 37u2. | |
2002-12-14 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
* JavaScriptCore.pbproj/project.pbxproj: Make dtoa.h visible as an SPI so I can | |
use it inside QString. | |
2002-12-14 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Ken. | |
- further corrections to number printing. | |
* kjs/ustring.cpp: | |
(UString::from): Make number printing match the ECMA standard | |
algorithm. | |
2002-12-14 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Dave. | |
- fix toString() conversion for numbers less than 1. Negative | |
exponents are still wrong though (things like 1E-34). | |
* kjs/ustring.cpp: | |
(UString::from): Don't print empty string for numbers less than 1, | |
and remember to add extra 0s after the decimal for negative | |
decimal positions. | |
=== Alexander-37u1 === | |
=== Alexander-36 === | |
2002-12-12 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Darin. | |
- fixed 3056449 - can't select state at tucows.com | |
* kjs/array_instance.h: | |
* kjs/array_object.cpp: | |
(ArrayInstanceImp::propList): Add numeric proprties that are in | |
special storage. | |
* kjs/array_object.h: | |
* kjs/object.h: Make propList a virtual method. | |
2002-12-11 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Don. | |
- Add kjsprint global function in Development build for ease of debugging. | |
- Print uncaught JavaScript exceptions to the console in Development. | |
- Improve wording of exception error messages. | |
* kjs/function.cpp: | |
(GlobalFuncImp::call): | |
* kjs/function.h: | |
* kjs/internal.cpp: | |
(InterpreterImp::initGlobalObject): | |
* kjs/interpreter.cpp: | |
(Interpreter::evaluate): | |
* kjs/nodes.cpp: | |
(NewExprNode::evaluate): | |
(FunctionCallNode::evaluate): | |
(RelationalNode::evaluate): | |
2002-12-10 John Sullivan <sullivan@apple.com> | |
Fixed more "Alexander"s that were lurking in places I forgot | |
to look before. | |
Reviewed by Darin | |
* Makefile.am: | |
"rm -rf $(SYMROOTS)/Safari.app/Frameworks/JavaScriptCore.framework" | |
2002-12-09 Darin Adler <darin@apple.com> | |
Reviewed by Maciej. | |
* JavaScriptCore.pbproj/project.pbxproj: Bump versions to 0.8 and 35u. | |
* English.lproj/InfoPlist.strings: In here too. | |
2002-12-09 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by Ken. | |
- fixed 3059637 - all articles missing at excite.com sports page | |
- fixed 3065903 - most of content missing at excite.com news page | |
These bugs both came up because a JavaScript function has a var | |
declaration that collides with a function parameter name. | |
* kjs/nodes.cpp: | |
(VarDeclNode::processVarDecls): Don't set the property to | |
undefined if a property with that name is already set on the | |
global object. Otherwise we may clobber function parameters with | |
undefined even before hitting a possible var initializer. | |
2002-12-06 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by: Darin Adler | |
- made framework embedding work correctly with buildit | |
* JavaScriptCore.pbproj/project.pbxproj: Give framework a relative | |
install path, don't install it the normal way, and copy it | |
manually to /AppleInternal/Library/Frameworks if installing. | |
=== Alexander-35 === | |
2002-12-04 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by: Richard Williamson | |
Added explicit lock/unlock methods so Sherlock can grab the | |
interpreter lock as needed. | |
- partially addressed 3084320 - JavaScriptCore crash | |
* kjs/internal.cpp: | |
(InterpreterImp::InterpreterImp): | |
(InterpreterImp::lock): | |
(InterpreterImp::unlock): | |
* kjs/internal.h: | |
* kjs/interpreter.cpp: | |
(Interpreter::lock): | |
(Interpreter::unlock): | |
* kjs/interpreter.h: | |
2002-12-04 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by: Darin Adler | |
Set things up so JavaScriptCore builds in PCRE and uses it for | |
regular expressions. This fixes many form validation bugs: | |
- fixed 3103197 - javascript at fidelity.com rejects valid input | |
- fixed 2942552 - form validation at weather.com fails | |
- fixed 3079752 - js always reports textarea is empty | |
- fixed 3079719 - covad.com "check availalbility" fails | |
* Makefile.am: Add pcre subdir. | |
* kjs/config.h: define HAVE_PCREPOSIX to true. | |
* kjs/regexp.h: Don't include pcreposix.h since nothing from there | |
is used. | |
* pcre/.cvsignore: Added. | |
* pcre/ChangeLog: Removed. | |
* pcre/INSTALL: Removed. | |
* pcre/Makefile.am: Added. | |
* pcre/Makefile.in: Removed. | |
* pcre/NEWS: Removed. | |
* pcre/NON-UNIX-USE: Removed. | |
* pcre/README: Removed. | |
* pcre/chartables.c: Added. | |
* pcre/config.guess: Removed. | |
* pcre/config.in: Removed. | |
* pcre/config.sub: Removed. | |
* pcre/configure: Removed. | |
* pcre/configure.in: Removed. | |
* pcre/dll.mk: Removed. | |
* pcre/doc/Tech.Notes: Removed. | |
* pcre/doc/pcre.3: Removed. | |
* pcre/doc/pcre.html: Removed. | |
* pcre/doc/pcre.txt: Removed. | |
* pcre/doc/pcregrep.1: Removed. | |
* pcre/doc/pcregrep.html: Removed. | |
* pcre/doc/pcregrep.txt: Removed. | |
* pcre/doc/pcreposix.3: Removed. | |
* pcre/doc/pcreposix.html: Removed. | |
* pcre/doc/pcreposix.txt: Removed. | |
* pcre/doc/pcretest.1: Removed. | |
* pcre/doc/pcretest.html: Removed. | |
* pcre/doc/pcretest.txt: Removed. | |
* pcre/doc/perltest.txt: Removed. | |
* pcre/install-sh: Removed. | |
* pcre/ltmain.sh: Removed. | |
* pcre/pcre-config.h: Added. | |
* pcre/pcre-config.in: Removed. | |
* pcre/internal.h: Include pcre-config.h instead of config.h | |
* pcre/pcre.c: | |
(ord2utf8): Fix warnings. | |
(pcre_compile): Fix warnings. | |
* pcre/pcre.def: Removed. | |
* pcre/pcre.h: Added. | |
* pcre/pcre.in: Removed. | |
* JavaScriptCore.pbproj/project.pbxproj: Added pcre files to build. | |
* JavaScriptCorePrefix.h: Guard c++ headers with #ifdef __cplusplus. | |
2002-12-04 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by: Richard Williamson | |
* pcre/doc/*: Added. | |
* pcre/testdata/*: Added. | |
2002-12-03 Maciej Stachowiak <mjs@apple.com> | |
Reviewed by: Darin Adler | |
- imported PCRE 3.9 into the tree; this isn't actually compiled or | |
used yet. | |
* pcre/*: Added. | |
== Rolled over to ChangeLog-2002-12-03 == |