Reviewers: Lasse Reichstein,

Message:
Lasse, would you have a look? Thx, Anders



Description:
One element cache for localtime.

Please review this at http://codereview.chromium.org/2023005/show

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/date.js


Index: src/date.js
===================================================================
--- src/date.js (revision 4617)
+++ src/date.js (working copy)
@@ -238,7 +238,15 @@
   return time + DaylightSavingsOffset(time) + local_time_offset;
 }

+
+var ltcache = {
+  key: null,
+  val: null
+};
+
 function LocalTimeNoCheck(time) {
+  var ltc = ltcache;
+  if (time == ltc.key) return ltc.val;
   if (time < -MAX_TIME_MS || time > MAX_TIME_MS) {
     return $NaN;
   }
@@ -252,7 +260,8 @@
   } else {
     var dst_offset = DaylightSavingsOffset(time);
   }
-  return time + local_time_offset + dst_offset;
+  ltc.key = time;
+  return (ltc.val = time + local_time_offset + dst_offset);
 }




--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to