About bugfix: For ISO 8601 syntax, Date.parse should accept " " in places where "T" is allowed

LittleQ littleq0903 at gmail.com
Sun Sep 16 21:53:26 PDT 2012


I made a patch for bug-791320 *[1]*, and somebody ask me to get opinions
from this mailing list.

This patch added support for accepting " " in places where "T" is allowed
originally, will it cause any problem.

for example:

"2012-12-12T12:12:12" and the derivation(append millisecond or timezone)
will be accepted.
"2012-12-12 12:12:12" will be accepted after this patch

and these won't be accepted after this modifying:

"2012-12-12A12:12:12" (A could be replaced by a-zA-Z)
"2012-12-12-12:12:12"
"2012-12-12:12:12:12"

any comment for this fixing?

Thanks, this is my first patch for firefox :) any advise will be welcome~

*[1] For ISO 8601 syntax, Date.parse should accept " " in places where "T"
is allowed.: *https://bugzilla.mozilla.org/show_bug.cgi?id=791320

-- 
{
    greeting: "Have a nice day!",
    name: "Colin Su",
    nickname: "LittleQ",
    email: "littleq0903 at gmail.com",
    website: "http://about.me/littleq",
    title: [
        ["Genie Capital", "Web Developer"],
        ["National Chengchi University", "Student", "Computer Science Dept."
]
    ]
}
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.mozilla.org/pipermail/es-discuss/attachments/20120917/8ac6f4a8/attachment.html>


More information about the es-discuss mailing list