Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(89)

Issue 247373004: Speed up DateTime members, by caching the results. (Closed)

Created:
6 years, 8 months ago by Anders Johnsen
Modified:
6 years, 8 months ago
Reviewers:
srdjan
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Speed up DateTime members, by caching the results. The result is stored in a list of ints. This turned out to be the most efficient way to store it, compared to packing in ints. BUG= R=srdjan@google.com Committed: https://code.google.com/p/dart/source/detail?r=35303

Patch Set 1 #

Patch Set 2 : #

Total comments: 2

Patch Set 3 : Cleanup. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+68 lines, -54 lines) Patch
M runtime/lib/date_patch.dart View 1 2 5 chunks +68 lines, -54 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Anders Johnsen
This speeds up the following: import 'dart:io'; void main() { var date = new DateTime.now(); ...
6 years, 8 months ago (2014-04-22 13:37:53 UTC) #1
srdjan
lgtm https://codereview.chromium.org/247373004/diff/20001/runtime/lib/date_patch.dart File runtime/lib/date_patch.dart (right): https://codereview.chromium.org/247373004/diff/20001/runtime/lib/date_patch.dart#newcode150 runtime/lib/date_patch.dart:150: return _parts[_MILLISECOND_INDEX]; You could write: int get millisecond ...
6 years, 8 months ago (2014-04-22 19:55:46 UTC) #2
Anders Johnsen
https://codereview.chromium.org/247373004/diff/20001/runtime/lib/date_patch.dart File runtime/lib/date_patch.dart (right): https://codereview.chromium.org/247373004/diff/20001/runtime/lib/date_patch.dart#newcode150 runtime/lib/date_patch.dart:150: return _parts[_MILLISECOND_INDEX]; On 2014/04/22 19:55:46, srdjan wrote: > You ...
6 years, 8 months ago (2014-04-23 05:58:54 UTC) #3
Anders Johnsen
6 years, 8 months ago (2014-04-23 06:17:45 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 manually as r35303 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698