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

[Hermès BÉLUSCA - MAÏTO]

Oh yeah, thank you!
H.

-----Message d'origine-----
De : Ros-dev [mailto:[e-mail]] De la part de Michael
Fritscher
Envoyé : jeudi 19 novembre 2015 07:30
À : ros-dev(a)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(a)reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev