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
À : [e-mail]
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
[e-mail]
http://www.reactos.org/mailman/listinfo/ros-dev