Skip to main content

Last Call Review of draft-ietf-cellar-flac-11
review-ietf-cellar-flac-11-genart-lc-enghardt-2023-09-25-00

Request Review of draft-ietf-cellar-flac
Requested revision No specific revision (document currently at 14)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2023-09-27
Requested 2023-09-13
Authors Martijn van Beurden , Andrew Weaver
I-D last updated 2023-09-25
Completed reviews Genart Last Call review of -11 by Reese Enghardt (diff)
Secdir Telechat review of -12 by Robert Sparks (diff)
Assignment Reviewer Reese Enghardt
State Completed
Request Last Call review on draft-ietf-cellar-flac by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/ZWPEXOBbXfOQlgJ716KNR4CZ3YY
Reviewed revision 11 (document currently at 14)
Result Ready w/nits
Completed 2023-09-25
review-ietf-cellar-flac-11-genart-lc-enghardt-2023-09-25-00
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-cellar-flac-11
Reviewer: Reese Enghardt
Review Date: 2023-09-25
IETF LC End Date: 2023-09-27
IESG Telechat date: Not scheduled for a telechat

Summary: The document is very well-written and clear. It is ready for
publication, I just found a few nits.

Major issues: None.

Minor issues:

Section 8.8:
When discussing URIs in absolute or relative form, please consider citing RFC
3986.

Nits/editorial comments:

Section 4.4:

"If indeed most residual samples are close to zero and a suitable Rice
parameter is chosen, this form of coding, a so-called variable-length code,
fewer less bits to store than storing the residual in unencoded form." "fewer
less bits" -> "uses fewer bits"?

Section 6:

"If a frame header refers to the streaminfo metadata block, the file is not
'streamable', see Section 7 for details. In this way, the file is streamable
and the frame header size small for all of the most common forms of audio
data." What does "In this way" refer to here? In the second sentence, "the
file" does not refer to the same file as in the first sentence, or does it?
Please consider rephrasing this part to make it clearer what is being said.