[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...

Hermès BÉLUSCA - MAÏTO hermes.belusca at sfr.fr
Thu Nov 19 12:55:35 UTC 2015


Oh yeah, thank you!
H.

-----Message d'origine-----
De : Ros-dev [mailto:ros-dev-bounces at reactos.org] De la part de Michael
Fritscher
Envoyé : jeudi 19 novembre 2015 07:30
À : ros-dev at reactos.org
Objet : 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...

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


_______________________________________________
Ros-dev mailing list
Ros-dev at reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev



More information about the Ros-dev mailing list