Skip to content

Commit

Permalink
Merge pull request #23 from cbor-wg/ev-iesg-review-1
Browse files Browse the repository at this point in the history
Changes addressing Éric Vyncke's IESG review
  • Loading branch information
cabo committed Oct 25, 2023
2 parents 1d4d8a4 + 0342ffe commit 1c85701
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions draft-ietf-cbor-time-tag.md
Expand Up @@ -162,6 +162,7 @@ informative:
refcontent:
- Fourth Edition
ann: Contents available via <⁠<https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2310.pdf>>
RFC5905: ntp

--- abstract

Expand Down Expand Up @@ -279,8 +280,8 @@ and 1) or text string (major type 3) keys, with the value type
determined by each specific key.
For negative integer keys and text string values of the key,
implementations MUST ignore key/value pairs they do not understand;
these keys are "elective", as the extended time is still
usable if an implementation elects not to implement them.
these keys are "elective", as the extended time as a whole is still
usable without the information they carry if an implementation elects not to implement them.
Conversely, for unsigned integer keys, implementations MUST signal as
an error key/value pairs they do not understand or implement
(these are either "base time" or "critical", see below).
Expand Down Expand Up @@ -428,7 +429,7 @@ If key -1 is not present, the default timescale value 0 is implied.
Additional values can be registered in the Timescale Registry
({{timescale-registry}}); values MUST be integers or text strings.

(Note that there should be no timescales "GPS" or "NTP" — instead,
(Note that there should be no timescales "GPS" or "NTP" {{-ntp}} — instead,
the time should be converted to TAI or UTC using a single addition or subtraction.)

~~~ math-asciitex
Expand Down

0 comments on commit 1c85701

Please sign in to comment.