Re: [Cellar] AD Review: draft-ietf-cellar-ffv1

Michael Niedermayer <michael@niedermayer.cc> Thu, 09 January 2020 23:29 UTC

Return-Path: <michael@niedermayer.cc>
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 CB09B120128; Thu, 9 Jan 2020 15:29:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.618
X-Spam-Level:
X-Spam-Status: No, score=-2.618 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_NONE=0.001] autolearn=ham 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 Usj4Puh_Kb_1; Thu, 9 Jan 2020 15:29:27 -0800 (PST)
Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E4C9812003E; Thu, 9 Jan 2020 15:29:26 -0800 (PST)
X-Originating-IP: 213.47.68.29
Received: from localhost (213-47-68-29.cable.dynamic.surfer.at [213.47.68.29]) (Authenticated sender: michael@niedermayer.cc) by relay4-d.mail.gandi.net (Postfix) with ESMTPSA id 5DD9EE0004; Thu, 9 Jan 2020 23:29:24 +0000 (UTC)
Date: Fri, 10 Jan 2020 00:29:23 +0100
From: Michael Niedermayer <michael@niedermayer.cc>
To: Adam Roach <adam@nostrum.com>
Cc: Dave Rice <dave@dericed.com>, draft-ietf-cellar-ffv1.all@ietf.org, cellar@ietf.org
Message-ID: <20200109232923.GK1173@michaelspb>
References: <b71e9496-c924-b970-9094-2b29ba173bdd@nostrum.com> <7DDCA42B-8BDA-4EBD-9C04-F0615265E021@dericed.com> <20200109182335.GH1173@michaelspb> <95c9d576-3683-208f-4e0a-0fa0cba70c42@nostrum.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="aqWxf8ydqYKP8htK"
Content-Disposition: inline
In-Reply-To: <95c9d576-3683-208f-4e0a-0fa0cba70c42@nostrum.com>
Archived-At: <https://mailarchive.ietf.org/arch/msg/cellar/n60A-KxmWG3Ip65w_cl98RdkwOo>
Subject: Re: [Cellar] AD Review: draft-ietf-cellar-ffv1
X-BeenThere: cellar@ietf.org
X-Mailman-Version: 2.1.29
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: Thu, 09 Jan 2020 23:29:29 -0000

On Thu, Jan 09, 2020 at 03:14:26PM -0600, Adam Roach wrote:
> On 1/9/20 12:23 PM, Michael Niedermayer wrote:
> >The implementation definedness can be avoided by simply refering to an implementation
> >and not just "C90" or other. not sure "twos complement" is strictly enough but it
> >may be the cleanest way to specify it.
> 
> 
> I don't think "two's complement" clearly indicates sign extension versus
> zero padding. I think you need to clearly specify precisely what happens to
> new high-order bits during a right-shift.

thats what i meant by "strictly enough".
here a gradient from clear to precisse to clarify more what i meant:
A. >> is defined as on common mainstream two's complement architectures
B. >> with signed values is defined with two's complement representation
   and sign extension
C. a >> b with signed a is defined so that the two's complement representation
   of a is shifted toward the less significant bit by b places. The newly
   introduced bits are identical to the previous sign bit of a. The bits
   shifted beyond the least significant bit are discarded. If b is negative
   the behavior is undefined.
   
when reading the text for A., i think its clear to everyone knowing ISO-C
while C. is more precisse it requires more effort to understand that this
is mostly what is done by ISO-C on mainstream architectures.
I guess we probably should combine something like A and C so its both
precisse and still can be immedeatly understood without the need to
disentagle the definition

Thanks
   

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.