oledlg: Implement OleUIAddVerbMenuA/W functions [resend]
Vitaly Lipatov
lav at etersoft.ru
Thu Dec 25 08:41:08 CST 2008
В сообщении от 25 декабря 2008 Вы написали:
...
> You could start with testing for some special cases (such as NULL input
> parameters).
If I will add test case for NULL input and returned values, my commit will be
accepted?
> For me your comment looks suspicious:
> -----
>
> + * TODO
> + * Check if OLE object has no verbs
>
> -----
>
> + hr = IOleObject_EnumVerbs(lpOleObj, &pEnumVerbs);
> + if (hr == S_OK) {
> ----
> Will this hr be OLEOBJ_E_NOVERBS when no verbs present?
> If it's so you're actually checking for this.
> This one needs a testcase for example.
Please understand me. I did a part of work. Possible my implementation is
incomplete. But I wish to share result of my work with the world, winehq
community. I agree with test case argument with I fix already existing
function. But I do a new realization and have no enough time and knowledge
to write ideal code. Also I can improve code only from winehq repository.
...
> Does native crash on it or not? I'm not sure.
>
> The same about SetLastError on failure - this should be checked by testing
> it directly.
I will try to add this tests.
--
Vitaly Lipatov, ALT Linux Team
Russia, Saint-Petersburg, www.etersoft.ru
More information about the wine-devel
mailing list