Re: [PATCH 0/9] MR7835: win32u: Implement generic OpenGL context functions.

[Dmitry Timoshkov (＠dmitry)]

Dmitry Timoshkov (@dmitry) commented about dlls/win32u/opengl.c:
> -
>    if (descr->cRedShift == 8) gl_format = OSMESA_ARGB;
> -
>    else if (descr->cRedShift == 16) gl_format = OSMESA_BGRA;
> -
>    if (descr.cRedShift == 8) gl_format = OSMESA_ARGB;
> -
>    else if (descr.cRedShift == 16) gl_format = OSMESA_BGRA;
>    else gl_format = OSMESA_RGBA;
>    break;
> case 24:
> -
>    gl_format = descr->cRedShift == 16 ? OSMESA_BGR : OSMESA_RGB;
> -
>    gl_format = descr.cRedShift == 16 ? OSMESA_BGR : OSMESA_RGB;
>    break;
> case 16:
>        gl_format = OSMESA_RGB_565;
>        break;
>    default:
> -
>    return NULL;
> -
>    return FALSE;

Why are you changing return value to FALSE? (here and below multiple times)