wine-commits June 2016

wine-commits@winehq.org

2 participants
592 discussions

Michael Stefaniuc : riched20/tests: Use EqualRect() instead of open coding it.
by Alexandre Julliard 02 Jun '16

02 Jun '16

Module: wine Branch: master Commit: b0104692ac9bffa89452e22ff776fb83056ba59f URL: http://source.winehq.org/git/wine.git/?a=commit;h=b0104692ac9bffa89452e22ff… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jun 2 13:41:21 2016 +0200 riched20/tests: Use EqualRect() instead of open coding it. Signed-off-by: Michael Stefaniuc <mstefani(a)redhat.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/riched20/tests/editor.c | 54 ++++++++++++-------------------------------- 1 file changed, 14 insertions(+), 40 deletions(-) diff --git a/dlls/riched20/tests/editor.c b/dlls/riched20/tests/editor.c index 2fd9c99..92af700 100644 --- a/dlls/riched20/tests/editor.c +++ b/dlls/riched20/tests/editor.c @@ -7050,9 +7050,7 @@ static void test_format_rect(void) expected.left += 1; expected.right -= 1; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7071,8 +7069,7 @@ static void test_format_rect(void) expected.bottom = min(clientRect.bottom, rc.bottom); expected.right = min(clientRect.right, rc.right); SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, + ok(EqualRect(&rc, &expected), "[n=%d] rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", n, rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7082,9 +7079,7 @@ static void test_format_rect(void) SendMessageA(hwnd, EM_SETRECT, 0, (LPARAM)&rc); expected = clientRect; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7094,9 +7089,7 @@ static void test_format_rect(void) ok(options & ECO_SELECTIONBAR, "EM_SETOPTIONS failed to add selectionbar.\n"); expected.left += 8; /* selection bar width */ SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7104,9 +7097,7 @@ static void test_format_rect(void) SendMessageA(hwnd, EM_SETRECT, 0, (LPARAM)&rc); expected = clientRect; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7117,9 +7108,7 @@ static void test_format_rect(void) ok(!(options & ECO_SELECTIONBAR), "EM_SETOPTIONS failed to remove selectionbar.\n"); expected.left -= 8; /* selection bar width */ SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7128,8 +7117,7 @@ static void test_format_rect(void) SendMessageA(hwnd, EM_SETRECT, 0, (LPARAM)&rc); expected = clientRect; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, + ok(EqualRect(&rc, &expected), "[n=%d] rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", n, rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7144,9 +7132,7 @@ static void test_format_rect(void) expected = rc; SendMessageA(hwnd, EM_SETRECT, 1, (LPARAM)&rc); SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7159,9 +7145,7 @@ static void test_format_rect(void) expected = rc; SendMessageA(hwnd, EM_SETRECT, 1, (LPARAM)&rc); SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7176,9 +7160,7 @@ static void test_format_rect(void) expected.left += 1; expected.right -= 1; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7189,9 +7171,7 @@ static void test_format_rect(void) expected.left += 1; expected.right -= 1; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7202,9 +7182,7 @@ static void test_format_rect(void) expected.left += 1; expected.right -= 1; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7223,9 +7201,7 @@ static void test_format_rect(void) expected.top += 1; expected.right -= 1; SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right); @@ -7240,9 +7216,7 @@ static void test_format_rect(void) expected.right += 1; SendMessageA(hwnd, EM_SETRECT, 0, (LPARAM)&rc); SendMessageA(hwnd, EM_GETRECT, 0, (LPARAM)&rc); - ok(rc.top == expected.top && rc.left == expected.left && - rc.bottom == expected.bottom && rc.right == expected.right, - "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", + ok(EqualRect(&rc, &expected), "rect a(t=%d, l=%d, b=%d, r=%d) != e(t=%d, l=%d, b=%d, r=%d)\n", rc.top, rc.left, rc.bottom, rc.right, expected.top, expected.left, expected.bottom, expected.right);

1 0

Alistair Leslie-Hughes : inetcomm: Add missing default properties.
by Alexandre Julliard 02 Jun '16

02 Jun '16

Module: wine Branch: master Commit: 943b376dbae0e9a6b99845948443e24e2e1fb5c6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=943b376dbae0e9a6b99845948… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Jun 1 06:06:30 2016 +0000 inetcomm: Add missing default properties. Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/inetcomm/mimeole.c | 95 ++++++++++++++++++++++++++++++++++++++----------- 1 file changed, 75 insertions(+), 20 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=943b376dbae0e9a6b9984…

1 0

Andrew Eikum : xaudio2: Implement GetVoiceDetails for source and submix voices.
by Alexandre Julliard 02 Jun '16

02 Jun '16

Module: wine Branch: master Commit: e733770416f42789f7b96b74505a9dc71e282ee4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e733770416f42789f7b96b745… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Jun 1 15:09:05 2016 -0500 xaudio2: Implement GetVoiceDetails for source and submix voices. Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/xaudio2_7/compat.c | 447 ++++++++++++++++++++++++++++++++++++++-- dlls/xaudio2_7/tests/xaudio2.c | 68 ++++++ dlls/xaudio2_7/xaudio_dll.c | 23 +++ dlls/xaudio2_7/xaudio_private.h | 8 + include/xaudio2.idl | 116 ++++++++++- 5 files changed, 645 insertions(+), 17 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e733770416f42789f7b96…

1 0

Andrew Eikum : xaudio2: Shut down OpenAL in the correct order.
by Alexandre Julliard 02 Jun '16

02 Jun '16

Module: wine Branch: master Commit: b77ec00f1c68c083c9ab0bb3b9fc26755d8084c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b77ec00f1c68c083c9ab0bb3b… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Jun 1 14:51:50 2016 -0500 xaudio2: Shut down OpenAL in the correct order. Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/xaudio2_7/xaudio_dll.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/xaudio2_7/xaudio_dll.c b/dlls/xaudio2_7/xaudio_dll.c index 1393599..acad6e1 100644 --- a/dlls/xaudio2_7/xaudio_dll.c +++ b/dlls/xaudio2_7/xaudio_dll.c @@ -963,12 +963,12 @@ static void WINAPI XA2M_DestroyVoice(IXAudio2MasteringVoice *iface) IAudioClient_Release(This->aclient); This->aclient = NULL; - alcCloseDevice(This->al_device); - This->al_device = NULL; - alcDestroyContext(This->al_ctx); This->al_ctx = NULL; + alcCloseDevice(This->al_device); + This->al_device = NULL; + LeaveCriticalSection(&This->lock); }

1 0

Hadrien Boizard : winegstreamer: Use BOOL type where appropriate.
by Alexandre Julliard 02 Jun '16

02 Jun '16

Module: wine Branch: master Commit: 68e45dd4d0021bff92fb26951fd30b4828558ba2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=68e45dd4d0021bff92fb26951… Author: Hadrien Boizard <h.boizard(a)laposte.net> Date: Tue May 31 17:41:33 2016 +0200 winegstreamer: Use BOOL type where appropriate. Signed-off-by: Hadrien Boizard <h.boizard(a)laposte.net> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/gstdemux.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c index d71225d..92804e3 100644 --- a/dlls/winegstreamer/gstdemux.c +++ b/dlls/winegstreamer/gstdemux.c @@ -84,7 +84,7 @@ struct GSTOutPin { GstPad *their_src; GstPad *my_sink; GstBufferPool *gstpool; - int isaud, isvid; + BOOL isaud, isvid; AM_MEDIA_TYPE * pmt; HANDLE caps_event; GstSegment *segment; @@ -783,7 +783,7 @@ static void init_new_decoded_pad(GstElement *bin, GstPad *pad, GSTImpl *This) GstPad *mypad; GSTOutPin *pin; int ret; - int isvid = 0, isaud = 0; + BOOL isvid = FALSE, isaud = FALSE; gchar my_name[1024]; TRACE("%p %p %p\n", This, bin, pad); @@ -809,9 +809,9 @@ static void init_new_decoded_pad(GstElement *bin, GstPad *pad, GSTImpl *This) gst_pad_set_query_function(mypad, query_sink_wrapper); if (!strcmp(typename, "audio/x-raw")) { - isaud = 1; + isaud = TRUE; } else if (!strcmp(typename, "video/x-raw")) { - isvid = 1; + isvid = TRUE; } else { FIXME("Unknown type \'%s\'\n", typename); return;

1 0

Francois Gouget : imm32/tests: Spelling fix.
by Alexandre Julliard 02 Jun '16

02 Jun '16

Module: wine Branch: master Commit: 09a401e2a89057ba2660cc9ddac72d67796e27f7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=09a401e2a89057ba2660cc9dd… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Jun 1 16:29:53 2016 +0200 imm32/tests: Spelling fix. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Aric Stewart <aric(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/imm32/tests/imm32.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/imm32/tests/imm32.c b/dlls/imm32/tests/imm32.c index fa6ae62..c7ae168 100644 --- a/dlls/imm32/tests/imm32.c +++ b/dlls/imm32/tests/imm32.c @@ -976,7 +976,7 @@ static DWORD WINAPI test_default_ime_window_cancel_cb(void *arg) WS_OVERLAPPEDWINDOW | visible, CW_USEDEFAULT, CW_USEDEFAULT, 240, 120, NULL, NULL, GetModuleHandleW(NULL), NULL); - ok(hwnd1 == NULL, "creation suceeded, got %p\n", hwnd1); + ok(hwnd1 == NULL, "creation succeeded, got %p\n", hwnd1); ok(!get_ime_window(), "Expected no IME windows\n"); test_phase = CREATE_CANCEL; @@ -984,7 +984,7 @@ static DWORD WINAPI test_default_ime_window_cancel_cb(void *arg) WS_OVERLAPPEDWINDOW | visible, CW_USEDEFAULT, CW_USEDEFAULT, 240, 120, NULL, NULL, GetModuleHandleW(NULL), NULL); - ok(hwnd1 == NULL, "creation suceeded, got %p\n", hwnd1); + ok(hwnd1 == NULL, "creation succeeded, got %p\n", hwnd1); ok(!get_ime_window(), "Expected no IME windows\n"); test_phase = FIRST_WINDOW;

1 0

Huw Davies : riched20: Add support for quoting urls with spaces within '<' '>'.
by Alexandre Julliard 01 Jun '16

01 Jun '16

Module: wine Branch: master Commit: 03fa4cfaffd4c10f56a2f8bfd383f4eaf67d364c URL: http://source.winehq.org/git/wine.git/?a=commit;h=03fa4cfaffd4c10f56a2f8bfd… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Jun 1 12:35:38 2016 +0100 riched20: Add support for quoting urls with spaces within '<' '>'. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/riched20/editor.c | 31 +++++++++++++++++++++++++++---- 1 file changed, 27 insertions(+), 4 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index d4ca570..a07dd7f 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -5038,8 +5038,8 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, ME_Cursor *candidate_min, ME_Cursor *candidate_max) { - ME_Cursor cursor = *start, neutral_end; - BOOL candidateStarted = FALSE; + ME_Cursor cursor = *start, neutral_end, space_end; + BOOL candidateStarted = FALSE, quoted = FALSE; WCHAR c; while (nChars > 0) @@ -5060,9 +5060,11 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, *candidate_min = cursor; candidateStarted = TRUE; neutral_end.pPara = NULL; + space_end.pPara = NULL; cursor.nOffset++; break; } + quoted = (c == '<'); cursor.nOffset++; } } @@ -5074,9 +5076,28 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, { c = str[cursor.nOffset]; if (isspaceW( c )) - goto done; + { + if (quoted && c != '\r') + { + if (!space_end.pPara) + { + if (neutral_end.pPara) + space_end = neutral_end; + else + space_end = cursor; + } + } + else + goto done; + } else if (isurlneutral( c )) { + if (quoted && c == '>') + { + neutral_end.pPara = NULL; + space_end.pPara = NULL; + goto done; + } if (!neutral_end.pPara) neutral_end = cursor; } @@ -5095,7 +5116,9 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, done: if (candidateStarted) { - if (neutral_end.pPara) + if (space_end.pPara) + *candidate_max = space_end; + else if (neutral_end.pPara) *candidate_max = neutral_end; else *candidate_max = cursor;

1 0

Huw Davies : riched20: Fix URL auto-detection to handle 'neutral' characters.
by Alexandre Julliard 01 Jun '16

01 Jun '16

Module: wine Branch: master Commit: ea1e44e4f698b278e3e980914188f414630bfc31 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea1e44e4f698b278e3e980914… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Jun 1 12:35:37 2016 +0100 riched20: Fix URL auto-detection to handle 'neutral' characters. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/riched20/editor.c | 124 ++++++++++++++++++++----------------------- dlls/riched20/tests/editor.c | 77 +++++++++++++++++++++++++++ 2 files changed, 136 insertions(+), 65 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 85395e7..d4ca570 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -5005,10 +5005,24 @@ LRESULT WINAPI REExtendedRegisterClass(void) return result; } -static BOOL isurlspecial(WCHAR c) +static int wchar_comp( const void *key, const void *elem ) { - static const WCHAR special_chars[] = {'.','/','%','@','*','|','\\','+','#',0}; - return strchrW( special_chars, c ) != NULL; + return *(const WCHAR *)key - *(const WCHAR *)elem; +} + +/* neutral characters end the url if the next non-neutral character is a space character, + otherwise they are included in the url. */ +static BOOL isurlneutral( WCHAR c ) +{ + /* NB this list is sorted */ + static const WCHAR neutral_chars[] = {'!','\"','\'','(',')',',','-','.',':',';','<','>','?','[',']','{','}'}; + + /* Some shortcuts */ + if (isalnum( c )) return FALSE; + if (c > neutral_chars[sizeof(neutral_chars) / sizeof(neutral_chars[0]) - 1]) return FALSE; + + return !!bsearch( &c, neutral_chars, sizeof(neutral_chars) / sizeof(neutral_chars[0]), + sizeof(c), wchar_comp ); } /** @@ -5024,87 +5038,67 @@ static BOOL ME_FindNextURLCandidate(ME_TextEditor *editor, ME_Cursor *candidate_min, ME_Cursor *candidate_max) { - ME_Cursor cursor = *start; - BOOL foundColon = FALSE; + ME_Cursor cursor = *start, neutral_end; BOOL candidateStarted = FALSE; - WCHAR lastAcceptedChar = '\0'; + WCHAR c; while (nChars > 0) { - WCHAR *strStart = get_text( &cursor.pRun->member.run, 0 ); - WCHAR *str = strStart + cursor.nOffset; - int nLen = cursor.pRun->member.run.len - cursor.nOffset; - nChars -= nLen; + WCHAR *str = get_text( &cursor.pRun->member.run, 0 ); + int run_len = cursor.pRun->member.run.len; + + nChars -= run_len - cursor.nOffset; - if (~cursor.pRun->member.run.nFlags & MERF_ENDPARA) + /* Find start of candidate */ + if (!candidateStarted) { - /* Find start of candidate */ - if (!candidateStarted) + while (cursor.nOffset < run_len) { - while (nLen) + c = str[cursor.nOffset]; + if (!isspaceW( c ) && !isurlneutral( c )) { - nLen--; - if (isalnumW(*str) || isurlspecial(*str)) - { - cursor.nOffset = str - strStart; - *candidate_min = cursor; - candidateStarted = TRUE; - lastAcceptedChar = *str++; - break; - } - str++; + *candidate_min = cursor; + candidateStarted = TRUE; + neutral_end.pPara = NULL; + cursor.nOffset++; + break; } + cursor.nOffset++; } + } - /* Find end of candidate */ - if (candidateStarted) { - while (nLen) + /* Find end of candidate */ + if (candidateStarted) + { + while (cursor.nOffset < run_len) + { + c = str[cursor.nOffset]; + if (isspaceW( c )) + goto done; + else if (isurlneutral( c )) { - nLen--; - if (*str == ':' && !foundColon) { - foundColon = TRUE; - } else if (!isalnumW(*str) && !isurlspecial(*str)) { - cursor.nOffset = str - strStart; - if (lastAcceptedChar == ':') - ME_MoveCursorChars(editor, &cursor, -1); - *candidate_max = cursor; - return TRUE; - } - lastAcceptedChar = *str++; + if (!neutral_end.pPara) + neutral_end = cursor; } - } - } else { - /* End of paragraph: skip it if before candidate span, or terminates - current active span */ - if (candidateStarted) { - if (lastAcceptedChar == ':') - ME_MoveCursorChars(editor, &cursor, -1); - *candidate_max = cursor; - return TRUE; - } - } + else + neutral_end.pPara = NULL; - /* Reaching this point means no span was found, so get next span */ - if (!ME_NextRun(&cursor.pPara, &cursor.pRun)) { - if (candidateStarted) { - /* There are no further runs, so take end of text as end of candidate */ - cursor.nOffset = str - strStart; - if (lastAcceptedChar == ':') - ME_MoveCursorChars(editor, &cursor, -1); - *candidate_max = cursor; - return TRUE; + cursor.nOffset++; } - *candidate_max = *candidate_min = cursor; - return FALSE; } + cursor.nOffset = 0; + if (!ME_NextRun(&cursor.pPara, &cursor.pRun)) + goto done; } - if (candidateStarted) { - /* There are no further runs, so take end of text as end of candidate */ - if (lastAcceptedChar == ':') - ME_MoveCursorChars(editor, &cursor, -1); - *candidate_max = cursor; +done: + if (candidateStarted) + { + if (neutral_end.pPara) + *candidate_max = neutral_end; + else + *candidate_max = cursor; return TRUE; } *candidate_max = *candidate_min = cursor; diff --git a/dlls/riched20/tests/editor.c b/dlls/riched20/tests/editor.c index fd00487..2fd9c99 100644 --- a/dlls/riched20/tests/editor.c +++ b/dlls/riched20/tests/editor.c @@ -1781,6 +1781,7 @@ static void test_EM_AUTOURLDETECT(void) "This is some text with #X on it", "This is some text with @X on it", "This is some text with \\X on it", + "This is some text with _X on it", }; /* All of these cause the URL detection to be extended by one more byte, thus demonstrating that the tested character is considered as part @@ -1794,6 +1795,33 @@ static void test_EM_AUTOURLDETECT(void) "This is some text with X# on it", "This is some text with X@ on it", "This is some text with X\\ on it", + "This is some text with X_ on it", + }; + /* These delims act as neutral breaks. Whether the url is ended + or not depends on the next non-neutral character. We'll test + with Y unchanged, in which case the url should include the + deliminator and the Y. We'll also test with the Y changed + to a space, in which case the url stops before the + deliminator. */ + const char * templates_neutral_delim[] = { + "This is some text with X-Y on it", + "This is some text with X--Y on it", + "This is some text with X!Y on it", + "This is some text with X[Y on it", + "This is some text with X]Y on it", + "This is some text with X{Y on it", + "This is some text with X}Y on it", + "This is some text with X(Y on it", + "This is some text with X)Y on it", + "This is some text with X\"Y on it", + "This is some text with X;Y on it", + "This is some text with X:Y on it", + "This is some text with X'Y on it", + "This is some text with X?Y on it", + "This is some text with X<Y on it", + "This is some text with X>Y on it", + "This is some text with X.Y on it", + "This is some text with X,Y on it", }; char buffer[1024]; @@ -1974,6 +2002,55 @@ static void test_EM_AUTOURLDETECT(void) } } + for (j = 0; j < sizeof(templates_neutral_delim) / sizeof(const char *); j++) { + char * at_pos, * end_pos; + int at_offset; + int end_offset; + + if (!urls[i].is_url) continue; + + at_pos = strchr(templates_neutral_delim[j], 'X'); + at_offset = at_pos - templates_neutral_delim[j]; + memcpy(buffer, templates_neutral_delim[j], at_offset); + buffer[at_offset] = '\0'; + strcat(buffer, urls[i].text); + strcat(buffer, templates_neutral_delim[j] + at_offset + 1); + + end_pos = strchr(buffer, 'Y'); + end_offset = end_pos - buffer; + + SendMessageA(hwndRichEdit, EM_AUTOURLDETECT, TRUE, 0); + SendMessageA(hwndRichEdit, WM_SETTEXT, 0, (LPARAM)buffer); + + /* This assumes no templates start with the URL itself, and that they + have at least two characters before the URL text */ + ok(!check_CFE_LINK_selection(hwndRichEdit, 0, 1), + "CFE_LINK incorrectly set in (%d-%d), text: %s\n", 0, 1, buffer); + ok(!check_CFE_LINK_selection(hwndRichEdit, at_offset -2, at_offset -1), + "CFE_LINK incorrectly set in (%d-%d), text: %s\n", at_offset -2, at_offset -1, buffer); + ok(!check_CFE_LINK_selection(hwndRichEdit, at_offset -1, at_offset), + "CFE_LINK incorrectly set in (%d-%d), text: %s\n", at_offset -1, at_offset, buffer); + + ok(check_CFE_LINK_selection(hwndRichEdit, at_offset, at_offset +1), + "CFE_LINK not set in (%d-%d), text: %s\n", at_offset, at_offset +1, buffer); + ok(check_CFE_LINK_selection(hwndRichEdit, end_offset -1, end_offset), + "CFE_LINK not set in (%d-%d), text: %s\n", end_offset -1, end_offset, buffer); + ok(check_CFE_LINK_selection(hwndRichEdit, end_offset, end_offset +1), + "CFE_LINK not set in (%d-%d), text: %s\n", end_offset, end_offset +1, buffer); + + *end_pos = ' '; + + SendMessageA(hwndRichEdit, EM_AUTOURLDETECT, TRUE, 0); + SendMessageA(hwndRichEdit, WM_SETTEXT, 0, (LPARAM)buffer); + + ok(check_CFE_LINK_selection(hwndRichEdit, at_offset, at_offset +1), + "CFE_LINK not set in (%d-%d), text: %s\n", at_offset, at_offset +1, buffer); + ok(!check_CFE_LINK_selection(hwndRichEdit, end_offset -1, end_offset), + "CFE_LINK set in (%d-%d), text: %s\n", end_offset -1, end_offset, buffer); + ok(!check_CFE_LINK_selection(hwndRichEdit, end_offset, end_offset +1), + "CFE_LINK set in (%d-%d), text: %s\n", end_offset, end_offset +1, buffer); + } + DestroyWindow(hwndRichEdit); hwndRichEdit = NULL; }

1 0

Huw Davies : include: Add flags for the EM_AUTOURLDETECT message.
by Alexandre Julliard 01 Jun '16

01 Jun '16

Module: wine Branch: master Commit: 0bdf9e5817497784bf9a791c4fe34fc73b7af5f1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0bdf9e5817497784bf9a791c4… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Jun 1 12:35:36 2016 +0100 include: Add flags for the EM_AUTOURLDETECT message. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/richedit.h | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/include/richedit.h b/include/richedit.h index ce2c145..f0ae57e 100644 --- a/include/richedit.h +++ b/include/richedit.h @@ -1070,6 +1070,14 @@ typedef struct _settextex { #define ST_NEWCHARS 0x00000004 #define ST_UNICODE 0x00000008 +/* Flags for EM_AUTOURLDETECT message */ +#define AURL_ENABLEURL 0x00000001 +#define AURL_ENABLEMAILADDR 0x00000002 +#define AURL_ENABLETELNO 0x00000004 +#define AURL_ENABLEEAURLS 0x00000008 +#define AURL_ENABLEDRIVELETTERS 0x00000010 +#define AURL_DISABLEMIXEDLGC 0x00000020 + #ifdef __cplusplus } #endif

1 0

Nikolay Sivov : msxml3/schema: Support element nodes in add().
by Alexandre Julliard 01 Jun '16

01 Jun '16

Module: wine Branch: master Commit: 1140291adc537121422e5d4184a0502d6e368d2d URL: http://source.winehq.org/git/wine.git/?a=commit;h=1140291adc537121422e5d418… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Jun 1 14:33:20 2016 +0300 msxml3/schema: Support element nodes in add(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msxml3/schema.c | 25 ++++++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/dlls/msxml3/schema.c b/dlls/msxml3/schema.c index f92130a..30272cb 100644 --- a/dlls/msxml3/schema.c +++ b/dlls/msxml3/schema.c @@ -1247,7 +1247,30 @@ static HRESULT WINAPI schema_cache_add(IXMLDOMSchemaCollection2* iface, BSTR uri IUnknown_QueryInterface(V_UNKNOWN(&var), &IID_IXMLDOMNode, (void**)&domnode); if (domnode) - doc = xmlNodePtr_from_domnode(domnode, XML_DOCUMENT_NODE)->doc; + { + DOMNodeType type; + + IXMLDOMNode_get_nodeType(domnode, &type); + switch (type) + { + case NODE_ELEMENT: + { + IXMLDOMDocument *domdoc; + VARIANT_BOOL b; + BSTR xml; + + IXMLDOMNode_get_xml(domnode, &xml); + DOMDocument_create(This->version, (void**)&domdoc); + IXMLDOMDocument_loadXML(domdoc, xml, &b); + SysFreeString(xml); + doc = xmlNodePtr_from_domnode((IXMLDOMNode*)domdoc, XML_DOCUMENT_NODE)->doc; + break; + } + default: + doc = xmlNodePtr_from_domnode(domnode, XML_DOCUMENT_NODE)->doc; + break; + } + } if (!doc) {

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits June 2016