[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