From dbd977a8b5afd5ccb0490646bc4e9a22e92dad64 Mon Sep 17 00:00:00 2001 From: Levhita Date: Sun, 15 Sep 2019 13:19:07 -0500 Subject: [PATCH] doc: added leap second behavior to napi_create_date and napi_get_date_value napi_create_date and napi_get_date_value ignore leap seconds as per ECMAScript spec that follows POSIX spec for time, comments added to the documentation where added fo clarify it. fixes: https://github.com/nodejs/node/issues/29439 --- doc/api/n-api.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/doc/api/n-api.md b/doc/api/n-api.md index 8f4eac295799ca..81da3c8ae72afc 100644 --- a/doc/api/n-api.md +++ b/doc/api/n-api.md @@ -1775,6 +1775,9 @@ napi_status napi_create_date(napi_env env, Returns `napi_ok` if the API succeeded. +This API does not observe leap seconds; they are ignored, as +ECMAScript aligns with POSIX time specification. + This API allocates a JavaScript `Date` object. JavaScript `Date` objects are described in @@ -2434,6 +2437,9 @@ napi_status napi_get_date_value(napi_env env, - `[out] result`: Time value as a `double` represented as milliseconds since midnight at the beginning of 01 January, 1970 UTC. +This API does not observe leap seconds; they are ignored, as +ECMAScript aligns with POSIX time specification. + Returns `napi_ok` if the API succeeded. If a non-date `napi_value` is passed in it returns `napi_date_expected`.