listview: Fix LISTVIEW_CancelEditLabel to not save the changed text
Nikolay Sivov
bunglehead at gmail.com
Sun Dec 6 04:28:51 CST 2009
Jeff Latimer wrote:
> On 06/12/09 18:56, Nikolay Sivov wrote:
>> Jeff Latimer wrote:
>>> ---
>>> dlls/comctl32/listview.c | 2 +-
>>> dlls/comctl32/tests/listview.c | 2 +-
>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>> Hi, Jeff.
>>
>> This is obviously wrong. Test will fail on native systems after your
>> patch.
> Why do you say that? It tests ok on XP.
No, it doesn't.
> It was not being run on XP due to
> if (!load_v6_module(&ctx_cookie, &hCtx))
Then it's not supposed to run on your system.
> so it looked if it was executing ok. When I forced it to run, the
> original failed.
You broke a logic This test needs v6 module to run.
> In any case the test comment says it is to check that the string
> "test" does not become "test1", however the test is
> ok(strcmp(buff, test1) == 0, "Expected label text not to change\n");
> which is the opposite.
So the comment is wrong, not the test condition. Please send a patch to
change comment.
More information about the wine-devel
mailing list