Re: [ros-dev] [ros-diffs] [hbelusca] 69936: [USER32] Implement now-documented MB_GetString. See: https://msdn.microsoft.com/en-us/library/windows/desktop/dn910915(v=vs.85).aspx and: http://undoc.airesoft.co.uk/user32.dll/MB...

[Michael Fritscher]

Hi,

>   +
>  +    /* The allowable IDs are between IDOK (0) and IDCONTINUE (11)
>  inclusive */
>  +    if (wBtn >= IDCONTINUE)
>  +        return NULL;
>  +
>  +    LoadStringW(User32Instance, wBtn, (LPWSTR)&btnStr, 0);
>  +    return btnStr; 
If IDCONTINUE is inclusive then the check in the next line should be >,
shouldn't it?

Best regards,
Michael