IETF Logo Mail Archive

Re: [Cellar] FFV1 nits

Dave Rice <dave@dericed.com> Wed, 07 June 2017 23:51 UTC

Return-Path: <dave@dericed.com>
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 4C7CD131491 for <cellar@ietfa.amsl.com>; Wed, 7 Jun 2017 16:51:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.119
X-Spam-Level:
X-Spam-Status: No, score=-1.119 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, SPF_NEUTRAL=0.779, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id svFt5DaYqm2C for <cellar@ietfa.amsl.com>; Wed, 7 Jun 2017 16:51:06 -0700 (PDT)
Received: from s172.web-hosting.com (s172.web-hosting.com [68.65.122.110]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 47E9E1314A4 for <cellar@ietf.org>; Wed, 7 Jun 2017 16:51:06 -0700 (PDT)
Received: from cpe-104-162-86-103.nyc.res.rr.com ([104.162.86.103]:34736 helo=[10.0.1.5]) by server172.web-hosting.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.87) (envelope-from <dave@dericed.com>) id 1dIkj9-001086-Sc; Wed, 07 Jun 2017 19:51:05 -0400
From: Dave Rice <dave@dericed.com>
Message-Id: <7F342F81-40F1-4AF6-9083-5D556BFD447E@dericed.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_80D44179-C414-49EA-ACE0-ED5C0739F249"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
Date: Wed, 07 Jun 2017 19:51:00 -0400
In-Reply-To: <CADK0Wux=z7UEp2Q8Lto-tkJ9tjYF=ZyOB-Q9L5eMwZFgF=rETw@mail.gmail.com>
Cc: Codec Encoding for LossLess Archiving and Realtime transmission <cellar@ietf.org>
To: Tessa Fallon <tessa.fallon@gmail.com>
References: <CADK0Wux=z7UEp2Q8Lto-tkJ9tjYF=ZyOB-Q9L5eMwZFgF=rETw@mail.gmail.com>
X-Mailer: Apple Mail (2.3273)
X-OutGoing-Spam-Status: No, score=-2.9
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - server172.web-hosting.com
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - dericed.com
X-Get-Message-Sender-Via: server172.web-hosting.com: authenticated_id: dave@dericed.com
X-Authenticated-Sender: server172.web-hosting.com: dave@dericed.com
X-Source:
X-Source-Args:
X-Source-Dir:
X-From-Rewrite: unmodified, already matched
Archived-At: <https://mailarchive.ietf.org/arch/msg/cellar/jWDzRLxs6Vncebs_GXSpCBg9JHA>
Subject: Re: [Cellar] FFV1 nits
X-BeenThere: cellar@ietf.org
X-Mailman-Version: 2.1.22
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, 07 Jun 2017 23:51:08 -0000

Hi Tessa,

> On Jun 7, 2017, at 12:37 PM, Tessa Fallon <tessa.fallon@gmail.com> wrote:
> 
> Generated by running nits from https://datatracker.ietf.org/doc/draft-niedermayer-cellar-ffv1/ <https://datatracker.ietf.org/doc/draft-niedermayer-cellar-ffv1/> 
> 
> idnits 2.14.02 
> 
> /tmp/draft-niedermayer-cellar-ffv1-02.txt:
[…]
>   ** The document seems to lack an IANA Considerations section.  (See Section
>      2.2 of http://www.ietf.org/id-info/checklist <http://www.ietf.org/id-info/checklist> for how to handle the case
>      when there are no actions for IANA.)
AFAIK there are no IANA considerations for FFV1. Anyone suggest otherwise? I added a pull request at github.com/FFmpeg/FFV1/pull/71 <https://github.com/FFmpeg/FFV1/pull/71> to just say "This document has no IANA actions."

[…]
>   == Missing Reference: 'L-l' is mentioned on line 383, but not defined
> 
>   == Missing Reference: 'T-t' is mentioned on line 384, but not defined

There is a sentence preceding the use of these terms that defines them as quantized sample differences. There’s an active pull request working on these lines. If the warning still exists after that pr I may try to adjust the formatting of the code block to silence these warnings.

Best Regards,
Dave Rice

v2.23.0 | Report a Bug | By Email