August 2017 - Wine-commits - list.winehq.org

Alexandre Julliard : crypt32/tests: Increase buffer size to avoid gcc warnings.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: 3427fbb33e02a52d8fef98402b7150e155ab1224 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3427fbb33e02a52d8fef98402… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Aug 28 18:23:28 2017 +0200 crypt32/tests: Increase buffer size to avoid gcc warnings. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/crypt32/tests/encode.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/crypt32/tests/encode.c b/dlls/crypt32/tests/encode.c index eae1282..8cb384c 100644 --- a/dlls/crypt32/tests/encode.c +++ b/dlls/crypt32/tests/encode.c @@ -476,7 +476,7 @@ static void testTimeEncoding(DWORD dwEncoding, LPCSTR structType, static const char *printSystemTime(const SYSTEMTIME *st) { - static char buf[25]; + static char buf[64]; sprintf(buf, "%02d-%02d-%04d %02d:%02d:%02d.%03d", st->wMonth, st->wDay, st->wYear, st->wHour, st->wMinute, st->wSecond, st->wMilliseconds); @@ -485,7 +485,7 @@ static const char *printSystemTime(const SYSTEMTIME *st) static const char *printFileTime(const FILETIME *ft) { - static char buf[25]; + static char buf[64]; SYSTEMTIME st; FileTimeToSystemTime(ft, &st);

1 0

Andrey Gusev : avrt: Add AvQuerySystemResponsiveness stub.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: dfa1815c02d1661766c8f3a4a290635e7adb4ccf URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfa1815c02d1661766c8f3a4a… Author: Andrey Gusev <andrey.goosev(a)gmail.com> Date: Mon Aug 28 15:48:20 2017 +0300 avrt: Add AvQuerySystemResponsiveness stub. Signed-off-by: Andrey Gusev <andrey.goosev(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/avrt/avrt.spec | 2 +- dlls/avrt/main.c | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/dlls/avrt/avrt.spec b/dlls/avrt/avrt.spec index 640a12a..a7dfc42 100644 --- a/dlls/avrt/avrt.spec +++ b/dlls/avrt/avrt.spec @@ -1,4 +1,4 @@ -@ stub AvQuerySystemResponsiveness +@ stdcall AvQuerySystemResponsiveness(long ptr) @ stdcall AvRevertMmThreadCharacteristics(long) @ stub AvRtCreateThreadOrderingGroup @ stub AvRtCreateThreadOrderingGroupExA diff --git a/dlls/avrt/main.c b/dlls/avrt/main.c index 8a11025..7ef2824 100644 --- a/dlls/avrt/main.c +++ b/dlls/avrt/main.c @@ -83,6 +83,12 @@ HANDLE WINAPI AvSetMmThreadCharacteristicsW(LPCWSTR TaskName, LPDWORD TaskIndex) return (HANDLE)0x12345678; } +BOOL WINAPI AvQuerySystemResponsiveness(HANDLE AvrtHandle, ULONG *value) +{ + FIXME("(%p, %p): stub\n", AvrtHandle, value); + return FALSE; +} + BOOL WINAPI AvRevertMmThreadCharacteristics(HANDLE AvrtHandle) { FIXME("(%p): stub\n", AvrtHandle);

1 0

Paul Gofman : d3d9/tests: Add test for SW shader in MVP mode.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: 0a4d3074ee7a13deea2d737fd35326b979a7f771 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0a4d3074ee7a13deea2d737fd… Author: Paul Gofman <gofmanp(a)gmail.com> Date: Fri Aug 25 12:51:47 2017 +0300 d3d9/tests: Add test for SW shader in MVP mode. Signed-off-by: Paul Gofman <gofmanp(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d9/tests/visual.c | 220 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 220 insertions(+) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=0a4d3074ee7a13deea2d7…

1 0

Miklós Máté : winex11: Improve the desktop resolution list.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: cd4139df3c3cf4eb55143d81c9a7b499bb519712 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cd4139df3c3cf4eb55143d81c… Author: Miklós Máté <mtmkls(a)gmail.com> Date: Fri Aug 25 21:25:40 2017 +0200 winex11: Improve the desktop resolution list. Signed-off-by: Miklós Máté <mtmkls(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winex11.drv/desktop.c | 48 +++++++++++++++++++++++++++++++++++++++------- 1 file changed, 41 insertions(+), 7 deletions(-) diff --git a/dlls/winex11.drv/desktop.c b/dlls/winex11.drv/desktop.c index bf2c463..4df8784 100644 --- a/dlls/winex11.drv/desktop.c +++ b/dlls/winex11.drv/desktop.c @@ -37,9 +37,43 @@ static unsigned int dd_mode_count; static unsigned int max_width; static unsigned int max_height; -static const unsigned int widths[] = {320, 320, 400, 512, 640, 640, 800, 1024, 1152, 1280, 1280, 1400, 1600}; -static const unsigned int heights[] = {200, 240, 300, 384, 400, 480, 600, 768, 864, 960, 1024, 1050, 1200}; -#define NUM_DESKTOP_MODES (sizeof(widths) / sizeof(widths[0])) +static struct screen_size { + unsigned int width; + unsigned int height; +} screen_sizes[] = { + /* 4:3 */ + { 320, 240}, + { 400, 300}, + { 512, 384}, + { 640, 480}, + { 768, 576}, + { 800, 600}, + {1024, 768}, + {1152, 864}, + {1280, 960}, + {1400, 1050}, + {1600, 1200}, + {2048, 1536}, + /* 5:4 */ + {1280, 1024}, + {2560, 2048}, + /* 16:9 */ + {1280, 720}, + {1366, 768}, + {1600, 900}, + {1920, 1080}, + {2560, 1440}, + {3840, 2160}, + /* 16:10 */ + { 320, 200}, + { 640, 400}, + {1280, 800}, + {1440, 900}, + {1680, 1050}, + {1920, 1200}, + {2560, 1600} +}; +#define NUM_DESKTOP_MODES (sizeof(screen_sizes) / sizeof(struct screen_size)) #define _NET_WM_STATE_REMOVE 0 #define _NET_WM_STATE_ADD 1 @@ -56,13 +90,13 @@ static void make_modes(void) X11DRV_Settings_AddOneMode(screen_width, screen_height, 0, 60); for (i=0; i<NUM_DESKTOP_MODES; i++) { - if ( (widths[i] <= max_width) && (heights[i] <= max_height) ) + if ( (screen_sizes[i].width <= max_width) && (screen_sizes[i].height <= max_height) ) { - if ( ( (widths[i] != max_width) || (heights[i] != max_height) ) && - ( (widths[i] != screen_width) || (heights[i] != screen_height) ) ) + if ( ( (screen_sizes[i].width != max_width) || (screen_sizes[i].height != max_height) ) && + ( (screen_sizes[i].width != screen_width) || (screen_sizes[i].height != screen_height) ) ) { /* only add them if they are smaller than the root window and unique */ - X11DRV_Settings_AddOneMode(widths[i], heights[i], 0, 60); + X11DRV_Settings_AddOneMode(screen_sizes[i].width, screen_sizes[i].height, 0, 60); } } }

1 0

Nikolay Sivov : wsdapi: Fix use-after-free when checking for unique prefix (Valgrind).
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: 3cb938fd4f3f16b52022e109a2ededb40371765b URL: http://source.winehq.org/git/wine.git/?a=commit;h=3cb938fd4f3f16b52022e109a… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Aug 26 22:12:59 2017 +0300 wsdapi: Fix use-after-free when checking for unique prefix (Valgrind). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/xml.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/wsdapi/xml.c b/dlls/wsdapi/xml.c index 145c394..824b7ff 100644 --- a/dlls/wsdapi/xml.c +++ b/dlls/wsdapi/xml.c @@ -529,6 +529,7 @@ static HRESULT WINAPI IWSDXMLContextImpl_AddNamespace(IWSDXMLContext *iface, LPC if (setNewPrefix) { WSDFreeLinkedMemory((void *)ns->PreferredPrefix); + ns->PreferredPrefix = NULL; if ((newPrefix != NULL) && (is_prefix_unique(This->namespaces, newPrefix))) {

1 0

Nikolay Sivov : wsdapi/tests: Use constants for test strings when possible.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: b491635f0ada4fb9501610270282505bf59d68f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b491635f0ada4fb9501610270… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Aug 26 22:38:44 2017 +0300 wsdapi/tests: Use constants for test strings when possible. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/tests/xml.c | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/dlls/wsdapi/tests/xml.c b/dlls/wsdapi/tests/xml.c index d6a7294..633124f 100644 --- a/dlls/wsdapi/tests/xml.c +++ b/dlls/wsdapi/tests/xml.c @@ -35,7 +35,7 @@ static void BuildAnyForSingleElement_tests(void) WSDXML_NAME name; WSDXML_NAMESPACE ns; WCHAR nameText[] = {'E','l','1',0}; - WCHAR text[] = {'H','e','l','l','o',0}; + static const WCHAR text[] = {'H','e','l','l','o',0}; static const WCHAR uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','t','e','s','t','/',0}; static const WCHAR prefix[] = {'t',0}; HRESULT hr; @@ -277,8 +277,8 @@ static void GetValueFromAny_tests(void) WCHAR child1NameText[] = {'T','i','m',0}; WCHAR child2NameText[] = {'B','o','b',0}; WCHAR child3NameText[] = {'J','o','e',0}; - WCHAR child1Value[] = {'V','1',0}; - WCHAR child2Value[] = {'V','2',0}; + static const WCHAR child1Value[] = {'V','1',0}; + static const WCHAR child2Value[] = {'V','2',0}; static const WCHAR uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','t','e','s','t','/',0}; static const WCHAR uri2[] = {'h','t','t','p',':','/','/','t','e','s','t','2','.','t','e','s','t','/',0}; static const WCHAR prefix[] = {'t',0}; @@ -379,15 +379,15 @@ static void GetValueFromAny_tests(void) static void XMLContext_AddNamespace_tests(void) { - WCHAR ns1Uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','t','e','s','t',0}; - WCHAR ns2Uri[] = {'h','t','t','p',':','/','/','w','i','n','e','.','r','o','c','k','s',0}; - WCHAR ns3Uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','a','g','a','i','n',0}; - WCHAR ns4Uri[] = {'h','t','t','p',':','/','/','o','n','e','.','m','o','r','e',0}; - WCHAR prefix1[] = {'t','s','t',0}; - WCHAR prefix2[] = {'w','i','n','e',0}; - WCHAR unPrefix0[] = {'u','n','0',0}; - WCHAR unPrefix1[] = {'u','n','1',0}; - WCHAR unPrefix2[] = {'u','n','2',0}; + static const WCHAR ns1Uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','t','e','s','t',0}; + static const WCHAR ns2Uri[] = {'h','t','t','p',':','/','/','w','i','n','e','.','r','o','c','k','s',0}; + static const WCHAR ns3Uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','a','g','a','i','n',0}; + static const WCHAR ns4Uri[] = {'h','t','t','p',':','/','/','o','n','e','.','m','o','r','e',0}; + static const WCHAR prefix1[] = {'t','s','t',0}; + static const WCHAR prefix2[] = {'w','i','n','e',0}; + static const WCHAR unPrefix0[] = {'u','n','0',0}; + static const WCHAR unPrefix1[] = {'u','n','1',0}; + static const WCHAR unPrefix2[] = {'u','n','2',0}; IWSDXMLContext *context; WSDXML_NAMESPACE *ns1 = NULL, *ns2 = NULL; @@ -506,12 +506,12 @@ static void XMLContext_AddNamespace_tests(void) static void XMLContext_AddNameToNamespace_tests(void) { - WCHAR ns1Uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','t','e','s','t',0}; - WCHAR ns2Uri[] = {'h','t','t','p',':','/','/','w','i','n','e','.','r','o','c','k','s',0}; - WCHAR prefix2[] = {'w','i','n','e',0}; - WCHAR unPrefix0[] = {'u','n','0',0}; - WCHAR name1Text[] = {'B','o','b',0}; - WCHAR name2Text[] = {'T','i','m',0}; + static const WCHAR ns1Uri[] = {'h','t','t','p',':','/','/','t','e','s','t','.','t','e','s','t',0}; + static const WCHAR ns2Uri[] = {'h','t','t','p',':','/','/','w','i','n','e','.','r','o','c','k','s',0}; + static const WCHAR prefix2[] = {'w','i','n','e',0}; + static const WCHAR unPrefix0[] = {'u','n','0',0}; + static const WCHAR name1Text[] = {'B','o','b',0}; + static const WCHAR name2Text[] = {'T','i','m',0}; IWSDXMLContext *context; WSDXML_NAMESPACE *ns2 = NULL; WSDXML_NAME *name1 = NULL, *name2 = NULL;

1 0

Alexandre Julliard : gdi32: Pass the brush origin explicitly to the brush backend functions.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: 97a9db43d256b85fae84fd63fcadd3d064602b4b URL: http://source.winehq.org/git/wine.git/?a=commit;h=97a9db43d256b85fae84fd63f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Aug 28 09:07:31 2017 +0200 gdi32: Pass the brush origin explicitly to the brush backend functions. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/dibdrv/bitblt.c | 10 ++++++---- dlls/gdi32/dibdrv/dibdrv.h | 2 +- dlls/gdi32/dibdrv/graphics.c | 6 ++++-- dlls/gdi32/dibdrv/objects.c | 9 ++++----- 4 files changed, 15 insertions(+), 12 deletions(-) diff --git a/dlls/gdi32/dibdrv/bitblt.c b/dlls/gdi32/dibdrv/bitblt.c index 79cd629..f4efc8f 100644 --- a/dlls/gdi32/dibdrv/bitblt.c +++ b/dlls/gdi32/dibdrv/bitblt.c @@ -737,7 +737,8 @@ static DWORD create_tmp_dib( const dib_info *copy, int width, int height, dib_in } static DWORD execute_rop( dibdrv_physdev *pdev, const RECT *dst_rect, dib_info *src, - const RECT *src_rect, const struct clipped_rects *clipped_rects, DWORD rop ) + const RECT *src_rect, const struct clipped_rects *clipped_rects, + const POINT *brush_org, DWORD rop ) { dib_info *dibs[3], *result = src, tmp; RECT rects[3]; @@ -784,10 +785,10 @@ static DWORD execute_rop( dibdrv_physdev *pdev, const RECT *dst_rect, dib_info * break; case OP_ARGS(PAT,DST): pdev->brush.rects( pdev, &pdev->brush, dibs[DST], clipped_rects->count, clipped_rects->rects, - OP_ROP(*opcode) ); + brush_org, OP_ROP(*opcode) ); break; case OP_ARGS(PAT,SRC): - pdev->brush.rects( pdev, &pdev->brush, dibs[SRC], 1, &rects[SRC], OP_ROP(*opcode) ); + pdev->brush.rects( pdev, &pdev->brush, dibs[SRC], 1, &rects[SRC], brush_org, OP_ROP(*opcode) ); break; } } @@ -1026,7 +1027,8 @@ DWORD dibdrv_PutImage( PHYSDEV dev, HRGN clip, BITMAPINFO *info, mask_rect( &pdev->dib, &dst->visrect, &src_dib, &src->visrect, &clipped_rects, rop2 ); } else - ret = execute_rop( pdev, &dst->visrect, &src_dib, &src->visrect, &clipped_rects, rop ); + ret = execute_rop( pdev, &dst->visrect, &src_dib, &src->visrect, &clipped_rects, + &dc->brush_org, rop ); free_clipped_rects( &clipped_rects ); } if (tmp_rgn) DeleteObject( tmp_rgn ); diff --git a/dlls/gdi32/dibdrv/dibdrv.h b/dlls/gdi32/dibdrv/dibdrv.h index adf16c4..ce34c0d 100644 --- a/dlls/gdi32/dibdrv/dibdrv.h +++ b/dlls/gdi32/dibdrv/dibdrv.h @@ -75,7 +75,7 @@ typedef struct dib_brush rop_mask_bits masks; struct brush_pattern pattern; BOOL (*rects)(struct dibdrv_physdev *pdev, struct dib_brush *brush, dib_info *dib, - int num, const RECT *rects, INT rop); + int num, const RECT *rects, const POINT *brush_org, INT rop); } dib_brush; struct intensity_range diff --git a/dlls/gdi32/dibdrv/graphics.c b/dlls/gdi32/dibdrv/graphics.c index 98b4dcf..a39f1e2 100644 --- a/dlls/gdi32/dibdrv/graphics.c +++ b/dlls/gdi32/dibdrv/graphics.c @@ -74,7 +74,7 @@ static BOOL brush_rect( dibdrv_physdev *pdev, dib_brush *brush, const RECT *rect if (!get_clipped_rects( &pdev->dib, rect, clip, &clipped_rects )) return TRUE; ret = brush->rects( pdev, brush, &pdev->dib, clipped_rects.count, clipped_rects.rects, - dc->ROPmode ); + &dc->brush_org, dc->ROPmode ); free_clipped_rects( &clipped_rects ); return ret; } @@ -1174,6 +1174,7 @@ BOOL dibdrv_PatBlt( PHYSDEV dev, struct bitblt_coords *dst, DWORD rop ) { dibdrv_physdev *pdev = get_dibdrv_pdev(dev); dib_brush *brush = &pdev->brush; + DC *dc = get_physdev_dc( dev ); int rop2 = get_rop2_from_rop( rop ); struct clipped_rects clipped_rects; DWORD and = 0, xor = 0; @@ -1196,7 +1197,8 @@ BOOL dibdrv_PatBlt( PHYSDEV dev, struct bitblt_coords *dst, DWORD rop ) case R2_NOP: break; default: - ret = brush->rects( pdev, brush, &pdev->dib, clipped_rects.count, clipped_rects.rects, rop2 ); + ret = brush->rects( pdev, brush, &pdev->dib, clipped_rects.count, clipped_rects.rects, + &dc->brush_org, rop2 ); break; } free_clipped_rects( &clipped_rects ); diff --git a/dlls/gdi32/dibdrv/objects.c b/dlls/gdi32/dibdrv/objects.c index a36e03f..736e526 100644 --- a/dlls/gdi32/dibdrv/objects.c +++ b/dlls/gdi32/dibdrv/objects.c @@ -1773,7 +1773,7 @@ BOOL fill_with_pixel( DC *dc, dib_info *dib, DWORD pixel, int num, const RECT *r * Fill a number of rectangles with the solid brush */ static BOOL solid_brush(dibdrv_physdev *pdev, dib_brush *brush, dib_info *dib, - int num, const RECT *rects, INT rop) + int num, const RECT *rects, const POINT *brush_org, INT rop) { DC *dc = get_physdev_dc( &pdev->dev ); DWORD color = get_pixel_color( dc, &pdev->dib, brush->colorref, TRUE ); @@ -2005,9 +2005,8 @@ static BOOL select_pattern_brush( dibdrv_physdev *pdev, dib_brush *brush, BOOL * * FIXME: Should we insist l < r && t < b? Currently we assume this. */ static BOOL pattern_brush(dibdrv_physdev *pdev, dib_brush *brush, dib_info *dib, - int num, const RECT *rects, INT rop) + int num, const RECT *rects, const POINT *brush_org, INT rop) { - DC *dc = get_physdev_dc( &pdev->dev ); BOOL needs_reselect = FALSE; if (rop != brush->rop) @@ -2043,14 +2042,14 @@ static BOOL pattern_brush(dibdrv_physdev *pdev, dib_brush *brush, dib_info *dib, } } - dib->funcs->pattern_rects( dib, num, rects, &dc->brush_org, &brush->dib, &brush->masks ); + dib->funcs->pattern_rects( dib, num, rects, brush_org, &brush->dib, &brush->masks ); if (needs_reselect) free_pattern_brush( brush ); return TRUE; } static BOOL null_brush(dibdrv_physdev *pdev, dib_brush *brush, dib_info *dib, - int num, const RECT *rects, INT rop) + int num, const RECT *rects, const POINT *brush_org, INT rop) { return TRUE; }

1 0

Alexandre Julliard : gdi32: Use the correct brush origin when the blit operation requires an intermediate bitmap.
by Alexandre Julliard 28 Aug '17

28 Aug '17

Module: wine Branch: master Commit: 1c56b3b5168844eb72e0dd22cbe98713c1da7b4f URL: http://source.winehq.org/git/wine.git/?a=commit;h=1c56b3b5168844eb72e0dd22c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Aug 28 09:10:45 2017 +0200 gdi32: Use the correct brush origin when the blit operation requires an intermediate bitmap. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/dibdrv/bitblt.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/dlls/gdi32/dibdrv/bitblt.c b/dlls/gdi32/dibdrv/bitblt.c index f4efc8f..0acb23f 100644 --- a/dlls/gdi32/dibdrv/bitblt.c +++ b/dlls/gdi32/dibdrv/bitblt.c @@ -742,6 +742,7 @@ static DWORD execute_rop( dibdrv_physdev *pdev, const RECT *dst_rect, dib_info * { dib_info *dibs[3], *result = src, tmp; RECT rects[3]; + POINT origin; int width = dst_rect->right - dst_rect->left; int height = dst_rect->bottom - dst_rect->top; const BYTE *opcode = BITBLT_Opcodes[(rop >> 16) & 0xff]; @@ -788,7 +789,10 @@ static DWORD execute_rop( dibdrv_physdev *pdev, const RECT *dst_rect, dib_info * brush_org, OP_ROP(*opcode) ); break; case OP_ARGS(PAT,SRC): - pdev->brush.rects( pdev, &pdev->brush, dibs[SRC], 1, &rects[SRC], brush_org, OP_ROP(*opcode) ); + /* offset the brush origin to match the final dest rectangle */ + origin.x = brush_org->x + rects[DST].left - rects[SRC].left; + origin.y = brush_org->y + rects[DST].top - rects[SRC].top; + pdev->brush.rects( pdev, &pdev->brush, dibs[SRC], 1, &rects[SRC], &origin, OP_ROP(*opcode) ); break; } }

1 0

Rosanne DiMesio : Fix submitTime field
by Jeremy Newman 28 Aug '17

28 Aug '17

Module: appdb Branch: master Commit: c2d2c7ad817828cafcf6e967b451fbdf009f268d URL: http://source.winehq.org/git/appdb.git/?a=commit;h=c2d2c7ad817828cafcf6e967… Author: Rosanne DiMesio <dimesio(a)earthlink.net> Date: Fri Aug 25 10:24:23 2017 -0500 Fix submitTime field Fixes a regression created by 4a1506322966340d3ad42151ffa31f551e7e9f4c where the submitTime field was not being filled in on new test reports. Signed-off-by: Rosanne DiMesio <dimesio(a)earthlink.net> Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com> --- include/testData.php | 7 +++---- tables/alter.sql | 4 ++++ tables/testResults.sql | 2 +- 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/include/testData.php b/include/testData.php index 49cfa69..98f1c09 100644 --- a/include/testData.php +++ b/include/testData.php @@ -84,11 +84,11 @@ class testData{ "testedRelease, staging, installs, runs,". "usedWorkaround, workarounds,". "testedRating, comments,". - "submitTime, submitterId, state,". + "submitterId, state,". "gpuMfr, graphicsDriver)". "VALUES('?', '?', '?', '?', '?', '?', '?',". "'?', '?', '?', '?', '?',". - "'?', '?', '?', '?', '?', '?', '?')", + "'?', '?', '?', '?', '?', '?')", $this->iVersionId, $this->shWhatWorks, $this->shWhatDoesnt, @@ -102,8 +102,7 @@ class testData{ $this->sUsedWorkaround, $this->shWorkarounds, $this->sTestedRating, - $this->sComments, - "NOW()", + $this->sComments, $_SESSION['current']->iUserId, $this->sState, $this->sGpuMfr, diff --git a/tables/alter.sql b/tables/alter.sql index 457f8d9..8aa4423 100644 --- a/tables/alter.sql +++ b/tables/alter.sql @@ -1,2 +1,6 @@ /* append to this file when changes are required to the live db */ /* it will be cleared when the changes go live */ +ALTER TABLE `testResults` CHANGE `submitTime` `submitTime` DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP; + +/* to fix the incorrect entries*/ +UPDATE testResults SET submitTime = testedDate WHERE submitTime = '0000-00-00 00:00:00' diff --git a/tables/testResults.sql b/tables/testResults.sql index 511c115..be80b9d 100644 --- a/tables/testResults.sql +++ b/tables/testResults.sql @@ -21,7 +21,7 @@ create table testResults ( workarounds text default NULL, testedRating enum('Platinum','Gold','Silver','Bronze','Garbage') NOT NULL, comments text, - submitTime datetime NOT NULL, + submitTime datetime NOT NULL default CURRENT_TIMESTAMP, submitterId int(11) NOT NULL default '0', state enum('accepted','queued','rejected','pending','deleted') NOT NULL default 'accepted', gpuMfr enum('AMD', 'Intel', 'Nvidia', 'Other', 'Unknown') default NULL,

1 0

Jeremy Newman : Revert "Fix submitTime field"
by Jeremy Newman 28 Aug '17

28 Aug '17

Module: appdb Branch: master Commit: 69991e4b2e9a211a043c3650249ba2017e77ac5e URL: http://source.winehq.org/git/appdb.git/?a=commit;h=69991e4b2e9a211a043c3650… Author: Jeremy Newman <jnewman(a)codeweavers.com> Date: Mon Aug 28 13:36:26 2017 -0500 Revert "Fix submitTime field" This reverts commit c2d2c7ad817828cafcf6e967b451fbdf009f268d. --- include/testData.php | 7 ++++--- tables/alter.sql | 4 ---- tables/testResults.sql | 2 +- 3 files changed, 5 insertions(+), 8 deletions(-) diff --git a/include/testData.php b/include/testData.php index 98f1c09..49cfa69 100644 --- a/include/testData.php +++ b/include/testData.php @@ -84,11 +84,11 @@ class testData{ "testedRelease, staging, installs, runs,". "usedWorkaround, workarounds,". "testedRating, comments,". - "submitterId, state,". + "submitTime, submitterId, state,". "gpuMfr, graphicsDriver)". "VALUES('?', '?', '?', '?', '?', '?', '?',". "'?', '?', '?', '?', '?',". - "'?', '?', '?', '?', '?', '?')", + "'?', '?', '?', '?', '?', '?', '?')", $this->iVersionId, $this->shWhatWorks, $this->shWhatDoesnt, @@ -102,7 +102,8 @@ class testData{ $this->sUsedWorkaround, $this->shWorkarounds, $this->sTestedRating, - $this->sComments, + $this->sComments, + "NOW()", $_SESSION['current']->iUserId, $this->sState, $this->sGpuMfr, diff --git a/tables/alter.sql b/tables/alter.sql index 8aa4423..457f8d9 100644 --- a/tables/alter.sql +++ b/tables/alter.sql @@ -1,6 +1,2 @@ /* append to this file when changes are required to the live db */ /* it will be cleared when the changes go live */ -ALTER TABLE `testResults` CHANGE `submitTime` `submitTime` DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP; - -/* to fix the incorrect entries*/ -UPDATE testResults SET submitTime = testedDate WHERE submitTime = '0000-00-00 00:00:00' diff --git a/tables/testResults.sql b/tables/testResults.sql index be80b9d..511c115 100644 --- a/tables/testResults.sql +++ b/tables/testResults.sql @@ -21,7 +21,7 @@ create table testResults ( workarounds text default NULL, testedRating enum('Platinum','Gold','Silver','Bronze','Garbage') NOT NULL, comments text, - submitTime datetime NOT NULL default CURRENT_TIMESTAMP, + submitTime datetime NOT NULL, submitterId int(11) NOT NULL default '0', state enum('accepted','queued','rejected','pending','deleted') NOT NULL default 'accepted', gpuMfr enum('AMD', 'Intel', 'Nvidia', 'Other', 'Unknown') default NULL,

1 0