Re: [PATCH v13 0/5] MR10673: oleaut32: Fix Null handling in three-valued logical ops.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/oleaut32/variant.c:
> -            switch(rightvt)
> -            {
> -            case VT_I1:   if (V_I1(right)) resvt = VT_NULL; break;
> -            case VT_UI1:  if (V_UI1(right)) resvt = VT_NULL; break;
> -            case VT_I2:   if (V_I2(right)) resvt = VT_NULL; break;
> -            case VT_UI2:  if (V_UI2(right)) resvt = VT_NULL; break;
> -            case VT_I4:   if (V_I4(right)) resvt = VT_NULL; break;
> -            case VT_UI4:  if (V_UI4(right)) resvt = VT_NULL; break;
> -            case VT_I8:   if (V_I8(right)) resvt = VT_NULL; break;
> -            case VT_UI8:  if (V_UI8(right)) resvt = VT_NULL; break;
> -            case VT_INT:  if (V_INT(right)) resvt = VT_NULL; break;
> -            case VT_UINT: if (V_UINT(right)) resvt = VT_NULL; break;
> -            case VT_BOOL: if (V_BOOL(right)) resvt = VT_NULL; break;
> -            case VT_R4:   if (V_R4(right)) resvt = VT_NULL; break;
> -            case VT_R8:   if (V_R8(right)) resvt = VT_NULL; break;
> -            case VT_CY:
> Why is this change needed? `resvt == VT_NULL` is what we need here anyway.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10673#note_138750