April 2012 - Wine-Devel - list.winehq.org

Re: GDI+ pageoffset
by Stepa Nick April 11, 2012

April 11, 2012

> > Run with `WINEDEBUG=loaddll wine my.exe` and ensure that builtin gdiplus > is being used. I run `WINEDEBUG=loaddll wine my.exe` and get such output: > trace:loaddll:load_builtin_dll Loaded L"KERNEL32.dll" at 0x7b810000: builtin > trace:loaddll:load_native_dll Loaded L"Z:\\home\\user\\tmp\\test\\CalculateCPP.exe" at 0x400000: native > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\msvcrt.dll" at 0x72540000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\advapi32.dll" at 0x68550000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\gdi32.dll" at 0x684b0000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\version.dll" at 0x6e960000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\user32.dll" at 0x68380000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\rpcrt4.dll" at 0x70d00000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\ole32.dll" at 0x685c0000: builtin > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\gdiplus.dll" at 0x4ec50000: native > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\MSVCR100.dll" at 0x78aa0000: native > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\shlwapi.dll" at 0x686c0000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\comctl32.dll" at 0x68720000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\msimg32.dll" at 0x68810000: builtin > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\mfc100u.dll" at 0x785f0000: native > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\imm32.dll" at 0x68bc0000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\winex11.drv" at 0x68980000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\uxtheme.dll" at 0x68c10000: builtin > trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\dwmapi.dll" at 0x68c40000: builtin > trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\MFC100ENU.DLL" at 0x5d360000: native > fixme:heap:HeapSetInformation (nil) 1 (nil) 0 > fixme:win:EnumDisplayDevicesW ((null),0,0x32f744,0x00000000), stub! So, non-native gdiplus.dll used?! Hmm... Can I see into gdiplus.dll anyway? I really need to understand how pixeloffset works.

2 1

Re: ws2_32/tests: Update hostent struct tests
by Austin English April 11, 2012

April 11, 2012

On Tue, Apr 10, 2012 at 21:13, Bruno Jesus <00cpxxx(a)gmail.com> wrote: > * Added a final required test for the h->h_addr_list[0] to show we > must put it in the right place. > * Removed the cached use of older gethostbyname result since the > memory is static and turned into garbage in windows if the current > call fails. > * Added a skip if we hit older OS allowing we to avoid using broken() > on each test. + if(h->h_addr_list == addr.mem) /* <= W2K */ + { + skip("Skipping hostent tests since this OS is unsupported\n"); + return; + } + You should use win_skip so that the tests will only be skipped on windows (we'd want the failure to be exposed on wine). -- -Austin

2 2

Re: gdi32/dibdrv: don't add_join if points are coincident in wide_line_segments
by Alexandre Julliard April 11, 2012

April 11, 2012

Daniel Lehman <dlehman(a)esri.com> writes: > This fixes a crash in PolyPolyline if points are in the same spot > > There were timeouts in the Testbot but were unrelated to anything I changed > > From 23edf2c2b89b98ef18849806e74b5d77f7889160 Mon Sep 17 00:00:00 2001 > From: Daniel Lehman <dlehman(a)esri.com> > Date: Wed, 25 Jan 2012 13:47:31 -0800 > Subject: gdi32/dibdrv: don't add_join if points are coincident in wide_line_segments > > if 2 points coincide (dx == dy == 0), wide_line_segments returns immediately > in that case, face_1 is not changed. it is not initialized so junk values on the > stack are used. when passed to add_join they create a region that causes a huge > amount of memory to be allocated. depending on the values, all available/rlimit > memory is exhausted and the program crashes. reverting to 1.3.26 works fine > > the test case submitted is a drastically reduced, simplest possible form of what our > internal test does, based on real map data You should integrate the test with the other graphics tests, to reuse all the infrastructure and also verify the graphics results. Also there is probably the same problem with the first and last points. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: d3drm: Some includes are useless
by Alexandre Julliard April 11, 2012

April 11, 2012

Nozomi Kodama <nozomi.kodama(a)yahoo.com> writes: > From 9772529c63a9c11bc169545b6d68722308591a2f Mon Sep 17 00:00:00 2001 > From: Nozomi Kodama <nozomi.kodama(a)yahoo.com> > Date: Tue, 10 Apr 2012 10:50:31 +0200 > Subject: Some includes are useless That sort of change is not useful, the headers will be included anyway. -- Alexandre Julliard julliard(a)winehq.org

1 0

GDI+ pageoffset
by Stepa Nick April 11, 2012

April 11, 2012

Hello. I have following code: > Bitmap * bmp = new Bitmap(500, 500, PixelFormat32bppARGB); > { > Graphics gr(bmp); > > //FIRST LINE > PointF* local1PointF = new PointF(17.823631, 89.429169); > PointF* local2PointF = new PointF(17.823631, 87.577080); > > //SECOND LINE > PointF* local3PointF = new PointF(14.933444, 89.429169); > PointF* local4PointF = new PointF(14.933444, 87.577080); > Pen * pen = new Pen(Color(255, 0, 0), 0.1); > gr.SetPageUnit(UnitMillimeter); > > gr.SetPixelOffsetMode(PixelOffsetModeHalf); //PROBLEM! > > gr.DrawLine(pen,(*local1PointF),(*local2PointF)); > gr.DrawLine(pen,(*local3PointF),(*local4PointF)); > } When I run it without SetPixelOffsetMode i get equal line, but with it line are not identical(first line is longer). I explore "dlls/gdiplus/graphics.c" and i find this comment: "/* FIXME: Pixel offset mode is not used anywhere except the getter/setter. */". So, I what to understand why I have different images(how pixeloffset affect on drawing lines)? I try to run windbg, run `WINEDEBUG=gdiplus,gdi,ddraw,graphics wine ./my.exe`, but analyzing output didn`t give my any results. Thanks!

2 1

Re: [PATCH 3/3] comctl32: Truncate the info tip text when using ANSI version messages.
by Marvin April 11, 2012

April 11, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17802 Your paranoid android. === WNT4WSSP6 (32 bit tooltips) === tooltips.c:755: Test failed: lpszText has 0 chars

1 0

Re: [PATCH 2/3] comctl32/tests: Add tests for long info tip texts.
by Marvin April 11, 2012

April 11, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17801 Your paranoid android. === WNT4WSSP6 (32 bit tooltips) === tooltips.c:755: Test failed: lpszText has 0 chars

1 0

Re: [PATCH 1/2] server: Add mechanism to retrieve the connect time of a socket (try 2).
by Dmitry Timoshkov April 11, 2012

April 11, 2012

"Erich E. Hoover" <ehoover(a)mines.edu> wrote: > +/* Get information about a socket */ > +DECL_HANDLER(get_socket_info) > +{ > + struct sock *sock; > + > + sock=(struct sock *)get_handle_obj( current->process, req->handle, FILE_WRITE_ATTRIBUTES, &sock_ops ); Shouldn't the required access be FILE_READ_ATTRIBUTES or FILE_READ_PROPERTIES? Why write access is requied in order to read the socket connect time? Also please avoid including auto generated code in the patch. -- Dmitry.

2 1

Re: [PATCH 1/6] msvcrt: Rewrite asctime function
by Alexandre Julliard April 11, 2012

April 11, 2012

Piotr Caban <piotr(a)codeweavers.com> writes: > @@ -48,7 +49,7 @@ static const int MonthLengths[2][12] = > > static inline int IsLeapYear(int Year) > { > - return Year % 4 == 0 && (Year % 100 != 0 || Year % 400 == 0); > + return (Year % 4 == 0 && (Year % 100 != 0 || Year % 400 == 0)) ? 1 : 0; Maybe you should do something like: return (((Year % 4 == 0 && (Year % 100 != 0 || Year % 400 == 0)) ? 1 : 0) ? 1 : 0) ? 1 : 0; to make really sure you get 0 or 1, you never know what might happen ;-) -- Alexandre Julliard julliard(a)winehq.org

3 2

Re: [1/2] comctl32/tests: Added tests for mouse events handling (try 5)
by Marvin April 10, 2012

April 10, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17777 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:1894: Test failed: right click (parent): the msg sequence is not complete: expected 0000 - actual 0133 treeview.c:1895: Test failed: right click (tree): the msg sequence is not complete: expected 0000 - actual 000f treeview.c:1902: Test failed: right drag (parent): the msg sequence is not complete: expected 0000 - actual 0133 treeview.c:1903: Test failed: right drag (tree): the msg sequence is not complete: expected 0000 - actual 000f

1 0

Re: [1/2] comctl32/tests: Added tests for mouse events handling (try 4)
by Marvin April 10, 2012

April 10, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17772 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:1892: Test failed: right click (parent): in msg 0x004e expecting id 0xfffffffb got 0xfffffff4 treeview.c:1892: Test failed: right click (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1893: Test failed: right click (tree): the msg 0x000f should NOT have been sent by DefWindowProc treeview.c:1893: Test failed: right click (tree): the msg sequence is not complete: expected 0000 - actual 000f treeview.c:1900: Test failed: right drag (parent): in msg 0x004e expecting id 0xfffffe68 got 0xfffffff4 treeview.c:1900: Test failed: right drag (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1901: Test failed: right drag (tree): the msg 0x000f should NOT have been sent by DefWindowProc treeview.c:1901: Test failed: right drag (tree): the msg sequence is not complete: expected 0000 - actual 000f

1 0

Re: [PATCH 1/6] msvcrt: Rewrite asctime function (try3)
by Alexandre Julliard April 10, 2012

April 10, 2012

Piotr Caban <piotr(a)codeweavers.com> writes: > + || !MSVCRT_CHECK_PMT(asctime_buf(time, mstm) != NULL)) { You don't want to use MSVCRT_CHECK_PMT this way, the error won't make sense. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/2] server: Add mechanism to retrieve the connect time of a socket (resend 2, deferred, rebased).
by Alexandre Julliard April 10, 2012

April 10, 2012

"Erich E. Hoover" <ehoover(a)mines.edu> writes: > Real Name: > Erich Hoover > > Description: > This patch implements storing the connect time of a socket for > retrieval with the SO_CONNECT_TIME socket option (part 2). > > Changelog: > server: Add mechanism to retrieve the connect time of a socket. You should make it a generic get_info request, and return the time in standard server format. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: msvcrt: Add bsearch_s implementation by reusing code and tests from ntdll (try 4)
by Alexandre Julliard April 10, 2012

April 10, 2012

André Hentschel <nerv(a)dawncrow.de> writes: > /********************************************************************* > + * bsearch_s (msvcrt.@) > + */ > +void* CDECL MSVCRT_bsearch_s(const void *key, const void *base, > + MSVCRT_size_t nmemb, MSVCRT_size_t size, > + int (__cdecl *compare)(void *, const void *, const void *), void *ctx) > +{ > + ssize_t min = 0; > + ssize_t max; > + > + if (!MSVCRT_CHECK_PMT(key != NULL) || !MSVCRT_CHECK_PMT(base != NULL) || > + !MSVCRT_CHECK_PMT(size != 0) || !MSVCRT_CHECK_PMT(compare != NULL)) > + { > + *MSVCRT__errno() = MSVCRT_EINVAL; > + return NULL; > + } > + > + if (!MSVCRT_CHECK_PMT(nmemb != 0) || !MSVCRT_CHECK_PMT(ctx != NULL)) > + { > + *MSVCRT__errno() = 0; > + return NULL; > + } That looks even more suspicious... -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 6/6] msvcrt/tests: Added more strftime tests
by Marvin April 10, 2012

April 10, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17760 Your paranoid android. === WNT4WSSP6 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 78013981 === W2KPROSP4 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 7801124A === WXPPROSP3 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 77C4900F === W2K3R2SESP2 (32 bit time) === time.c:605: Test failed: errno = -559038737 time: unhandled exception c0000005 at 77BD9F8B

1 0

Re: [PATCH 1/6] msvcrt: Rewrite asctime function
by Marvin April 10, 2012

April 10, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17759 Your paranoid android. === WNT4WSSP6 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737 === W2KPROSP4 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737 === WXPPROSP3 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737 === W2K3R2SESP2 (32 bit time) === time.c:646: Test failed: asctime returned time.c:647: Test failed: errno = -559038737 time.c:658: Test failed: asctime returned Wed Feb 30 02:58:41 1979 time.c:659: Test failed: errno = -559038737

1 0

'Pending' patches state
by Dmitry Timoshkov April 10, 2012

April 10, 2012

Hello, http://source.winehq.org/patches has the following legend for the Pending patch status: Pending * The patch is not obviously correct at first glance. Making a more convincing argument, preferably in the form of a test case, may help. * Waiting for feedback from the main developer in that area. It's very confusing, and absolutely not clear what is required from the patch submitter, especially since *there is no any feedback on the patch*. 'Rejected' at least requies some sort of feedback, while 'Pending' doesn't. To me 'Pending' looks like a silent case of 'Reject', but without any obligation to explain why. I find myself on somewhat shaky ground when I see a bunch of my patches in the pending state, especially if they already contain the tests and main developer in the area I'm changing is Alexandre :). Is it possible to get at least some feedback for pending patches? Pretty please? -- Dmitry.

6 13

RFC: KUSER_SHARED_DATA update patch to fix bug 29168
by Joey Yandle April 10, 2012

April 10, 2012

Hi everybody, As originally discovered by Carsten Juttner, Star Wars: The Old Republic uses the time values in KUSER_SHARED_DATA to trigger network send buffers. wine does not currently update these values, so TOR does not make it past an initial server handshake. I'm attaching my current patch which fixes this. It updates the values every 15600000 nanoseconds as does Windows 7, and presumably all Windows versions. The words of the timers are written in the correct order for applications to detect clock tick. It uses a separate thread which does nothing else; a previous attempt at an NtTimer based solution did not work, presumably due to irregularity in the exact callback times. Attempts at setting thread priority were also abandoned after many people were unable to run the code. The patch does not check the return code of nanosleep; I could add code to nanosleep again for the remaining time in the case of an interrupt, but didn't think it was necessary. I could also change it to use clock_nanosleep, which would allow for more absolute timing. Please let me know what you think. cheers, Joey

9 34

Re: [PATCH 2/2] wininet/tests: Added checks for bad return values and skip tests when it happens. (try 2)
by Alexandre Julliard April 9, 2012

April 9, 2012

Morten Rønne <morten.roenne(a)tdcadsl.dk> writes: > diff --git a/dlls/wininet/tests/urlcache.c b/dlls/wininet/tests/urlcache.c > index 02d8d28..1bf3f23 100644 > --- a/dlls/wininet/tests/urlcache.c > +++ b/dlls/wininet/tests/urlcache.c > @@ -60,7 +60,20 @@ static void test_find_url_cache_entriesA(void) > hEnumHandle = FindFirstUrlCacheEntry(NULL, NULL, &cbCacheEntryInfo); > ok(!hEnumHandle, "FindFirstUrlCacheEntry should have failed\n"); > ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "FindFirstUrlCacheEntry should have set last error to ERROR_INSUFFICIENT_BUFFER instead of %d\n", GetLastError()); > + if (GetLastError() != ERROR_INSUFFICIENT_BUFFER) > + { > + skip("Find*UrlCacheEntry tests skipped, due to wrong return value\n"); > + return; > + } You don't need to add skip() for that sort of thing, it's not supposed to happen, and if it does there will already be a test failure in the report. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: [1/3] shell32: Add test for GetCurrentProcessExplicitAppUserModelID.
by Alexandre Julliard April 9, 2012

April 9, 2012

Vincent Povirk <madewokherd(a)gmail.com> writes: > @@ -3,6 +3,7 @@ IMPORTS = shell32 ole32 oleaut32 user32 advapi32 > > C_SRCS = \ > appbar.c \ > + appusermodel.c \ > assoc.c \ > autocomplete.c \ > brsfolder.c \ I don't think you need a new file for that. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: msvcrt: Add bsearch_s implementation by reusing code and tests from ntdll (try 3)
by Alexandre Julliard April 9, 2012

April 9, 2012

André Hentschel <nerv(a)dawncrow.de> writes: > @@ -124,6 +124,39 @@ void* CDECL _lsearch(const void* match, void* start, > } > > /********************************************************************* > + * bsearch_s (msvcrt.@) > + */ > +void* CDECL MSVCRT_bsearch_s(const void *key, const void *base, > + MSVCRT_size_t nmemb, MSVCRT_size_t size, > + int (__cdecl *compare)(void *, const void *, const void *), void *ctx) > +{ > + ssize_t min = 0; > + ssize_t max; > + > + if (!MSVCRT_CHECK_PMT(key != NULL) || !MSVCRT_CHECK_PMT(base != NULL) || > + !MSVCRT_CHECK_PMT(nmemb != 0) || !MSVCRT_CHECK_PMT(size != 0) || > + !MSVCRT_CHECK_PMT(compare != NULL) || !MSVCRT_CHECK_PMT(ctx != NULL)) > + { Some of these look suspicious, you need more tests. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: jscript: Replace only the first match for non-regexp pattern in String.replace
by Marvin April 9, 2012

April 9, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17730 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: ws2_32/tests: Add hostent struct tests for gethostbyname
by Marvin April 8, 2012

April 8, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17727 Your paranoid android. === WXPPROSP3 (32 bit sock) === sock.c:509: Test failed: oob_server (1ac): unexpectedly at the OOB mark: 0 sock.c:519: Test failed: oob_server (1ac): unexpectedly at the OOB mark: 0

3 4

Re: [2/2] comctl32: Fix message sequence on right mouse click
by Marvin April 7, 2012

April 7, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17723 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:1886: Test failed: right click (parent): the msg 0x004e was expected, but got msg 0x0133 instead treeview.c:1886: Test failed: right click (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1887: Test failed: right click (tree): the msg 0x0215 was expected, but got msg 0x000f instead treeview.c:1887: Test failed: right click (tree): the msg sequence is not complete: expected 0000 - actual 0215 treeview.c:1894: Test failed: right drag (parent): the msg 0x004e was expected, but got msg 0x0133 instead treeview.c:1894: Test failed: right drag (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1895: Test failed: right drag (tree): the msg 0x0215 was expected, but got msg 0x000f instead treeview.c:1895: Test failed: right drag (tree): the msg sequence is not complete: expected 0000 - actual 0215

1 0

Re: [1/2] comctl32/tests: Added tests for mouse events handling
by Marvin April 7, 2012

April 7, 2012

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=17721 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:1888: Test failed: right click (parent): the msg 0x004e was expected, but got msg 0x0133 instead treeview.c:1889: Test failed: right click (tree): the msg 0x0215 was expected, but got msg 0x000f instead treeview.c:1896: Test failed: right drag (parent): the msg 0x004e was expected, but got msg 0x0133 instead treeview.c:1896: Test failed: right drag (parent): the msg sequence is not complete: expected 0000 - actual 004e treeview.c:1897: Test failed: right drag (tree): the msg 0x0215 was expected, but got msg 0x000f instead treeview.c:1897: Test failed: right drag (tree): the msg sequence is not complete: expected 0000 - actual 0215

1 0