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

Fix typo #245

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix typo #245

wants to merge 1 commit into from

Conversation

ledoge
Copy link

@ledoge ledoge commented Sep 22, 2024

No description provided.

@ktmf01
Copy link
Collaborator

ktmf01 commented Sep 22, 2024

Currently the document is undergoing intense scrutiny as part by the RFC editor, so I won't make any changes. They'll probably notice this as well, but thanks for bringing it up anyway.

@H2Swine
Copy link

H2Swine commented Nov 16, 2024

@ktmf01 : They will likely have spottted the following, but in case not ... Page references to https://www.ietf.org/archive/id/draft-ietf-cellar-flac-14.txt :

  • Two instances: page 53 and 69 uncapitalized "vorbis". All the other (except in URL) have capital V
  • page 41: "least-sigificant" should be "least-significant"
  • page 8: a "the the" that should only be a single "the"
  • Also but maybe good enough, page 3: "at at". It is "looking back at at most 32", which does make sense as "looking back at" "at most 32", but "looking back at most 32" is less puzzling to the reading eye.

@ktmf01
Copy link
Collaborator

ktmf01 commented Nov 16, 2024

but in case not ...

@H2Swine Then now is too late.

Here is the current state: https://www.rfc-editor.org/authors/rfc9639.html

All those things have been corrected AFAIK. Publication will probably be in about 3 weeks.

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

Successfully merging this pull request may close these issues.

3 participants