Re: [PATCH 2/2] Fix mem region parsing regression and add test

Simon Marchi
2017-11-14T21:43:55Z

On 2017-11-09 18:18, Simon Marchi wrote:
> In my patch
>
> Get rid of VEC (mem_region)
> a664f67e50eff30198097d51cec0ec4690abb2a1
>
> I introduced a regression, where the length of the memory region is
> assigned to the "hi" field. It should obviously be computed as "start
> +
> length". To my defense, no test had caught this :). As a penance, I
> wrote one.

I have pushed this patch (2/2), since it's a bug fix and I don't want to
forget it. Patch 1/2 is still up for review, especially the part about
updating the file on the website.

Simon