On 2 March 2018 at 11:51, Eric Engestrom <[email protected]> wrote:
> On Friday, 2018-03-02 13:28:28 +0200, Andres Gomez wrote:
>> Found by inspection.
>>
>> Cc: Ian Romanick <[email protected]>
>> Cc: Emil Velikov <[email protected]>
>> Cc: Eric Engestrom <[email protected]>
>> Signed-off-by: Andres Gomez <[email protected]>
>> ---
>>  src/egl/main/eglcontext.c | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/src/egl/main/eglcontext.c b/src/egl/main/eglcontext.c
>> index 18c1bc59acc..3e5d8ad97bf 100644
>> --- a/src/egl/main/eglcontext.c
>> +++ b/src/egl/main/eglcontext.c
>> @@ -594,7 +594,6 @@ _eglInitContext(_EGLContext *ctx, _EGLDisplay *dpy, 
>> _EGLConfig *conf,
>
> For reviewers: the line removed is a duplicate of the line literally
> just above the context being printed here.
>
Thanks Eric - change did raise an eyebrow on quick skim.
I'd suggest "s/redundant/duplicate/" or picking your note for the
commit message.

Either way
Reviewed-by: Emil Velikov <[email protected]>

-Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to