Re: [Cellar] shepherd review of Matroska: part 1

Michael Richardson <mcr+ietf@sandelman.ca> Wed, 06 July 2022 17:45 UTC

Return-Path: <mcr+ietf@sandelman.ca>
X-Original-To: cellar@ietfa.amsl.com
Delivered-To: cellar@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B0AA9C157B5B for <cellar@ietfa.amsl.com>; Wed, 6 Jul 2022 10:45:22 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.705
X-Spam-Level:
X-Spam-Status: No, score=-1.705 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_INVALID=0.1, DKIM_SIGNED=0.1, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=neutral reason="invalid (public key: not available)" header.d=sandelman.ca
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id UdE_8PKk6YAS for <cellar@ietfa.amsl.com>; Wed, 6 Jul 2022 10:45:19 -0700 (PDT)
Received: from tuna.sandelman.ca (tuna.sandelman.ca [209.87.249.19]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id EB547C14792A for <cellar@ietf.org>; Wed, 6 Jul 2022 10:45:18 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by tuna.sandelman.ca (Postfix) with ESMTP id 476C1399E4; Wed, 6 Jul 2022 14:02:21 -0400 (EDT)
Received: from tuna.sandelman.ca ([127.0.0.1]) by localhost (localhost [127.0.0.1]) (amavisd-new, port 10024) with LMTP id stob4b6kGrdX; Wed, 6 Jul 2022 14:02:20 -0400 (EDT)
Received: from sandelman.ca (obiwan.sandelman.ca [209.87.249.21]) by tuna.sandelman.ca (Postfix) with ESMTP id 9F3DE3998C; Wed, 6 Jul 2022 14:02:20 -0400 (EDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=sandelman.ca; s=mail; t=1657130540; bh=F4jWgVLIc/b7IuL0S45ZHvz8LnuPAAVWK5h6niBErwU=; h=From:To:Subject:In-Reply-To:References:Date:From; b=YpdaFplM80cJX38JoDj42mjlninWaTNFWY+lQ3PQ/LPmItwJ78TSm5+xs+Cs6o71J Tc3nzGzuEpvgVC62eVnyh1mW0ZyjiAYbw2mRZlJjdbtgyaKEjlCNaDc737X7O+jkKV ad+LQa6v4W1/eCH6KMFNhpAnI/10Gzykfaemj9IzRa9iQPk0M4UJgDMiIfsrLggkmO 67Wn8Q1IYWY90tsn4oIQSxoO+vib1jwC2Ms2pEihu8xwrjDvUtncWVAiqMJa3PashA l69JwRmSGnq4LpEvSaFXLt3jnpU2LafR0gnyRl3vEhpGsmqXaytz0L34E5RNT/eoOr aIZZ3vZVlW7qg==
Received: from localhost (localhost [IPv6:::1]) by sandelman.ca (Postfix) with ESMTP id 403A15DF; Wed, 6 Jul 2022 13:45:17 -0400 (EDT)
From: Michael Richardson <mcr+ietf@sandelman.ca>
To: Steve Lhomme <slhomme@matroska.org>, cellar@ietf.org
In-Reply-To: <01ddd325-18a0-a9be-f3c1-1995bfa97e8b@matroska.org>
References: <1216608.1654521891@dooku> <a0de8e0c-b163-efbf-3c76-9cfd38faa4bc@matroska.org> <01ddd325-18a0-a9be-f3c1-1995bfa97e8b@matroska.org>
X-Mailer: MH-E 8.6+git; nmh 1.7+dev; GNU Emacs 27.1
X-Face: $\n1pF)h^`}$H>Hk{L"x@)JS7<%Az}5RyS@k9X%29-lHB$Ti.V>2bi.~ehC0; <'$9xN5Ub# z!G,p`nR&p7Fz@^UXIn156S8.~^@MJ*mMsD7=QFeq%AL4m<nPbLgmtKK-5dC@#:k
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-="; micalg="pgp-sha512"; protocol="application/pgp-signature"
Date: Wed, 06 Jul 2022 13:45:17 -0400
Message-ID: <5852.1657129517@localhost>
Archived-At: <https://mailarchive.ietf.org/arch/msg/cellar/nve56MitiUIAzd6Ia4cqDwvlgXM>
Subject: Re: [Cellar] shepherd review of Matroska: part 1
X-BeenThere: cellar@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Codec Encoding for LossLess Archiving and Realtime transmission <cellar.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/cellar>, <mailto:cellar-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/cellar/>
List-Post: <mailto:cellar@ietf.org>
List-Help: <mailto:cellar-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/cellar>, <mailto:cellar-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 06 Jul 2022 17:45:22 -0000

The IDNITS has a few complaints:

https://www6.ietf.org/tools/idnits?url=https://www.ietf.org/archive/id/draft-ietf-cellar-matroska-11.txt

A bunch of downref concerns which are probably all just missing references.
Let me know if I can help track those down.

Miscellaneous warnings:
----------------------------------------------------------------------------

  == The expression 'MAY NOT', while looking like RFC 2119 requirements text,
     is not defined in RFC 2119, and should not be used.  Consider using 'MUST
     NOT' instead (if that is what you mean).

     Found 'MAY NOT' in this paragraph:

     Cluster Elements contain the content for each track, e.g., video
     frames.  A Matroska file SHOULD contain at least one Cluster Element. The
     Cluster Element helps to break up SimpleBlock or BlockGroup Elements and
     helps with seeking and error protection.  Every Cluster Element MUST
     contain a Timestamp Element.  This SHOULD be the Timestamp Element used
     to play the first Block in the Cluster Element.  Cluster Elements contain
     one or more block element, such as BlockGroup or SimpleBlock elements.
     In some situation, a Cluster Element *MAY NOT* contain any block element,
     for example in a live recording when no data has been collected.

I suggest rewording:

     In some situation, a Cluster Element *MAY* contain no block element,
     for example in a live recording when no data has been collected.

--
Michael Richardson <mcr+IETF@sandelman.ca>   . o O ( IPv6 IøT consulting )
           Sandelman Software Works Inc, Ottawa and Worldwide