[FFmpeg-devel] [PATCH] doc/developer: Add terse documentation of assumed C implementation defined behavior

[wm4]

On Fri, 28 Apr 2017 02:50:42 +0200
Michael Niedermayer <michael at niedermayer.cc> wrote:

> Suggested-by: "Ronald S. Bultje" <rsbultje at gmail.com>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  doc/developer.texi | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/developer.texi b/doc/developer.texi
> index dbe1f5421f..a948113792 100644
> --- a/doc/developer.texi
> +++ b/doc/developer.texi
> @@ -131,6 +131,11 @@ designated struct initializers (@samp{struct s x = @{ .i = 17 @};});
>  
>  @item
>  compound literals (@samp{x = (struct s) @{ 17, 23 @};}).
> +
> + at item
> +Implementation defined behavior for signed integers is assumed to match the
> +expected for Twos complement. Non representable values in integer casts are binary

Patch is ok, but "the expected for Twos complement" sounds a bit weird.
Maybe "expected behavior"? Also "two's complement".

> +truncated. Shift right of signed values uses sign extension.
>  @end itemize
>  
>  These features are supported by all compilers we care about, so we will not