Clarify that date parsing follows Safari. (issue 11085006)

7 views
Skip to first unread message

ross...@chromium.org

unread,
Oct 8, 2012, 7:58:47 AM10/8/12
to ul...@chromium.org, v8-...@googlegroups.com
Reviewers: ulan,

Description:
Clarify that date parsing follows Safari.


R=ul...@chromium.org
BUG=145198


Please review this at https://codereview.chromium.org/11085006/

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

Affected files:
M src/dateparser-inl.h


Index: src/dateparser-inl.h
diff --git a/src/dateparser-inl.h b/src/dateparser-inl.h
index
a5c7143bdd37743a0b334696ec6e83ecd181b96e..3cb36fa433998e96dbb9acadae999a791cc89277
100644
--- a/src/dateparser-inl.h
+++ b/src/dateparser-inl.h
@@ -62,7 +62,8 @@ bool DateParser::Parse(Vector<Char> str,
// sss is in the range 000..999,
// hh is in the range 00..23,
// mm, ss, and sss default to 00 if missing, and
- // timezone defaults to Z if missing.
+ // timezone defaults to Z if missing
+ // (following Safari, ISO actually demands local time).
// Extensions:
// We also allow sss to have more or less than three digits (but at
// least one).


Reply all
Reply to author
Forward
0 new messages