On Mon, Nov 6, 2017 at 4:41 PM, Eric Engestrom
<[email protected]> wrote:
> On Friday, 2017-11-03 21:38:51 +0100, Giuseppe Bilotta wrote:
>> This fixes some “Conditional jump depends on uninitialized value(s)”
>> errors spotted by valgrind.
>>
>> Signed-off-by: Giuseppe Bilotta <[email protected]>
>
> Reviewed-by: Eric Engestrom <[email protected]>
>
> Although the rest of the array is 0-initialised, so this only works
> because NoSymbol is also 0.
Indeed. I considered clarifying this in a comment, and even just
making it all 0-initialized with {}. Would that be better?
--
Giuseppe "Oblomov" Bilotta
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
