util,test: Use consistent Date representation for util/inspect by Xotic750 · Pull Request #4318 · nodejs/node (original) (raw)

Re: #4314
Inspect formats dates in two different ways, depending on
whether it has properties or not.

No properties and it uses toString.
https://github.com/nodejs/node/blob/master/lib/util.js#L272

The toString() method always returns a string representation of
the date in American English.

While when properties are present it uses toUTCString.
https://github.com/nodejs/node/blob/master/lib/util.js#L393

The format of the return value may vary according to the platform.
The most common return value is a RFC-1123 formatted date stamp,
which is a slightly updated version of RFC-822 date stamps.

I am wondering why two different representations are used?

And being 2015, it would seem sensible to use toISOString.

The toISOString() method returns a string in simplified
extended ISO format (ISO 8601), which is always 24 characters
long: YYYY-MM-DDTHH:mm:ss.sssZ. The timezone is always zero
UTC offset, as denoted by the suffix "Z".

./configure
make test
Total errors found: 0