Re: [PATCH v18 0/4] MR6737: comctl32/toolbar: forward WM_NOTIFY CBEN_ENDEDITW

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/comctl32/toolbar.c:
> - case CBEN_ENDEDITW:
> -
>    if (infoPtr->bUnicode)
> -
>    {
> -
>        TRACE("Forwarding CBEN_ENDEDITW (no conversion).\n");
> -
>        return SendMessageW(infoPtr->hwndNotify, WM_NOTIFY, lpnmh->idFrom, (LPARAM)lpnmh);
> -
>    }
> -
>    TRACE("Converting CBEN_ENDEDITW to CBEN_ENDEDITA and forwarding.\n");
> -
>    return COMCTL32_ForwardNotifyToAnsiWindow(infoPtr->hwndNotify, lpnmh, NULL);
> -
> - case CBEN_ENDEDITA:
> -
>    TRACE("Forwarding CBEN_ENDEDITA (no conversion).\n");
> -
>    return SendMessageW(infoPtr->hwndNotify, WM_NOTIFY, lpnmh->idFrom, (LPARAM)lpnmh);
> - default:
> -
>    WARN("Should WM_NOTIFY NMHDR code 0x%x be forwarded?\n", lpnmh->code);
>    return 0;

I don't think we need these traces.