Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ABNF as2-partial-time accept time-secfrac without time-second #488

Open
1 of 5 tasks
artlog opened this issue Jul 29, 2018 · 2 comments
Open
1 of 5 tasks

ABNF as2-partial-time accept time-secfrac without time-second #488

artlog opened this issue Jul 29, 2018 · 2 comments

Comments

@artlog
Copy link

artlog commented Jul 29, 2018

Please Indicate One:

  • Editorial
  • Question
  • Feedback
  • Blocking Issue
  • Non-Blocking Issue

Please Describe the Issue:

for ABNF in Date and Time :

as2-partial-time = time-hour ":" time-minute [":" time-second] [time-secfrac]

shouldn't it be

as2-partial-time = time-hour ":" time-minute [":" time-second [time-secfrac] ]

since time-secfrac has few interest if time-second is not provided.

@gobengo
Copy link
Contributor

gobengo commented Sep 12, 2018

Good eye.
Though conceptually you can add a fraction of a second to a minute without first needing to add a second.

I don't think there are any inconsistencies with the way it is that would warrant a backward-incompatible change or adding to errata.

Again, though, good eye.

@evanp
Copy link
Collaborator

evanp commented Oct 11, 2023

This seems like an error to me. I added an erratum here:

#550

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants