January 2016 - Wine-Devel - list.winehq.org

Re: [6/6] shell32/tests: Add server-side DDE checks.
by Marvin Jan. 29, 2016

Jan. 29, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20064 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2433: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2435: Test failed: failed: rc=29 err=32 shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2433: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2435: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2433: Test failed: ShlexecVar expected 'Present', but key not found or empty shlexec.c:2435: Test failed: failed: rc=29 err=32 shlexec.c:2447: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt209B.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt209B.tmp\test file.sde")]' shlexec.c:2435: Test failed: failed: rc=29 err=2 shlexec.c:2435: Test failed: failed: rc=29 err=2

1 0

Re: [4/6] shell32/tests: Check for failures in the child process.
by Marvin Jan. 29, 2016

Jan. 29, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20062 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2408: Test failed: failed: rc=29 err=2 shlexec.c:2408: Test failed: failed: rc=29 err=32 shlexec.c:2408: Test failed: failed: rc=29 err=2 shlexec.c:2408: Test failed: failed: rc=29 err=32 === w1064 (64 bit shlexec) === shlexec.c:2408: Test failed: failed: rc=29 err=2 shlexec.c:2408: Test failed: failed: rc=29 err=32 shlexec.c:2408: Test failed: failed: rc=29 err=2 shlexec.c:2408: Test failed: failed: rc=29 err=32

1 0

Re: [3/6] shell32/tests: Check the child process exit code and close the process handle.
by Marvin Jan. 28, 2016

Jan. 28, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20061 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2394: Test failed: failed: rc=29 err=2 shlexec.c:2394: Test failed: failed: rc=29 err=2 shlexec.c:2394: Test failed: failed: rc=29 err=2 shlexec.c:2394: Test failed: failed: rc=29 err=2 === w1064 (64 bit shlexec) === shlexec.c:2394: Test failed: failed: rc=29 err=2 shlexec.c:2394: Test failed: failed: rc=29 err=2 shlexec.c:2394: Test failed: failed: rc=29 err=2 shlexec.c:2394: Test failed: failed: rc=29 err=2

1 0

Re: [2/6] shell32/tests: Declare childPrintf() as a printf-style function.
by Marvin Jan. 28, 2016

Jan. 28, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20060 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 === w1064 (64 bit shlexec) === shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2401: Test failed: ddeExec expected '[ifexec("C:\Users\winetest\AppData\Local\Temp\wt8341.tmp\test file.sde")]', got '[open("C:\Users\winetest\AppData\Local\Temp\wt8341.tmp\test file.sde")]' shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2

1 0

Re: [1/6] shell32/tests: Reorder the functions to avoid forward declarations.
by Marvin Jan. 28, 2016

Jan. 28, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20059 Your paranoid android. === w1064 (32 bit shlexec) === shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 === w1064 (64 bit shlexec) === shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2 shlexec.c:2389: Test failed: failed: rc=29 err=2

1 0

Re: [1/3] scrrun: Implement filesys_DriveExists. (resend)
by Alexandre Julliard Jan. 28, 2016

Jan. 28, 2016

Joachim Priesner <joachim.priesner(a)web.de> writes: > - return E_NOTIMPL; > + if (!pfExists) return E_POINTER; > + *pfExists = VARIANT_FALSE; > + len = SysStringLen(DriveSpec); > + if (len >= 1 && DriveSpec[0] >= 'A' && DriveSpec[0] <= 'Z' > + && (len < 2 || DriveSpec[1] == ':') > + && (len < 3 || DriveSpec[2] == '\\')) { Surely lower-case drives should work too. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH 2/2] shell32: Set the stream and storage attributes where needed.
by Marvin Jan. 28, 2016

Jan. 28, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20039 Your paranoid android. === w1064 (32 bit shlfolder) === shlfolder.c:1591: Test failed: IShellFolder_GetDisplayNameOf(NULL) failed! hr = 80070057

2 1

Re: ndis.sys: add NdisAllocateSpinLock stub
by Alexandre Julliard Jan. 28, 2016

Jan. 28, 2016

Austin English <austinenglish(a)gmail.com> writes: > From 5450fedec076c6d2f2c10c2e5cfa8f79578803a0 Mon Sep 17 00:00:00 2001 > From: Austin English <austinenglish(a)gmail.com> > Date: Tue, 5 Jan 2016 17:24:12 -0600 > Subject: [PATCH] ndis.sys: add NdisAllocateSpinLock stub > > Signed-off-by: Austin English <austinenglish(a)gmail.com> > --- > dlls/ndis.sys/main.c | 5 +++++ > include/ddk/ndis.h | 7 +++++++ > 2 files changed, 12 insertions(+) You forgot the spec file. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 6/7] wined3d: Validate stream_idx in wined3d_device_set_stream_source_freq().
by Henri Verbeet Jan. 28, 2016

Jan. 28, 2016

On 28 January 2016 at 00:17, Józef Kucia <jkucia(a)codeweavers.com> wrote: > @@ -1412,6 +1412,12 @@ HRESULT CDECL wined3d_device_set_stream_source_freq(struct wined3d_device *devic > return WINED3DERR_INVALIDCALL; > } > > + if (stream_idx >= MAX_STREAMS) > + { > + FIXME("Stream index %u out of range.\n", stream_idx); > + return WINED3DERR_INVALIDCALL; > + } This would have to be a WARN instead of a FIXME, but in general we don't do this in wined3d unless we have to.

1 0

Re: [PATCH 2/7] d3d9/tests: Add test for invalid render states.
by Henri Verbeet Jan. 28, 2016

Jan. 28, 2016

On 28 January 2016 at 00:17, Józef Kucia <jkucia(a)codeweavers.com> wrote: > Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> > --- > A similar test crashes in d3d8. > --- > dlls/d3d9/tests/device.c | 40 ++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 40 insertions(+) > This probably just corrupts memory on Windows, if e.g. SetTextureStageState() is any indication.

1 0

Re: [PATCH 1/7] wined3d: Validate render state in wined3d_device_set_render_state().
by Henri Verbeet Jan. 28, 2016

Jan. 28, 2016

On 28 January 2016 at 00:17, Józef Kucia <jkucia(a)codeweavers.com> wrote: > + if (state > WINEHIGHEST_RENDER_STATE) > + { > + WARN("Unhandled render state %#x.\n", state); > + return; > + } It may make sense to do this for wined3d_device_get_render_state() as well.

1 0

Re: [PATCH v2] ntoskrnl.exe: Implement IoGetDeviceProperty for DevicePropertyPhysicalDeviceObjectName
by Alexandre Julliard Jan. 28, 2016

Jan. 28, 2016

Aric Stewart <aric(a)codeweavers.com> writes: > + switch (device_property) > + { > + case DevicePropertyPhysicalDeviceObjectName: > + { > + OBJECT_NAME_INFORMATION *name = HeapAlloc(GetProcessHeap(), 0, buffer_length + (sizeof(OBJECT_NAME_INFORMATION))); > + > + status = NtQueryObject(device->Reserved, ObjectNameInformation, name, buffer_length + sizeof(OBJECT_NAME_INFORMATION), result_length); > + if (status == STATUS_SUCCESS) > + memcpy(property_buffer, name->Name.Buffer, name->Name.Length); > + HeapFree(GetProcessHeap(), 0, name); > + break; result_length is not handled correctly. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] d3dx9: Signature isn't part of the DDS header
by Matteo Bruni Jan. 28, 2016

Jan. 28, 2016

2016-01-27 10:29 GMT+01:00 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com>: > Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> > --- > dlls/d3dx9_36/surface.c | 78 ++++++++++++++++++++++++++----------------------- > 1 file changed, 41 insertions(+), 37 deletions(-) > > diff --git a/dlls/d3dx9_36/surface.c b/dlls/d3dx9_36/surface.c > index 4fa2a76..a7bf9b7 100644 > --- a/dlls/d3dx9_36/surface.c > +++ b/dlls/d3dx9_36/surface.c > @@ -125,7 +125,6 @@ struct dds_pixel_format > > struct dds_header > { > - DWORD signature; > DWORD size; > DWORD flags; > DWORD height; > @@ -142,6 +141,12 @@ struct dds_header > DWORD reserved2; > }; > > +struct DDS Probably better to pick a lowercase struct name. > +{ > + DWORD signature; > + struct dds_header header; > +}; > + > static D3DFORMAT dds_fourcc_to_d3dformat(DWORD fourcc) > { > unsigned int i; > @@ -355,7 +360,7 @@ static UINT calculate_dds_file_size(D3DFORMAT format, UINT width, UINT height, U > } > > file_size *= faces; > - file_size += sizeof(struct dds_header); > + file_size += sizeof(struct DDS); > return file_size; > } > > @@ -379,37 +384,37 @@ static HRESULT get_image_info_from_dds(const void *buffer, UINT length, D3DXIMAG > { > UINT faces = 1; > UINT expected_length; > - const struct dds_header *header = buffer; > + const struct DDS *dds = buffer; I'd keep the *header variable, initializing it to &dds->header... > > - if (length < sizeof(*header) || !info) > + if (length < sizeof(*dds) || !info) > return D3DXERR_INVALIDDATA; > > - if (header->pixel_format.size != sizeof(header->pixel_format)) > + if (dds->header.pixel_format.size != sizeof(dds->header.pixel_format)) > return D3DXERR_INVALIDDATA; ... so that you can leave this and the other similar lines below unchanged. > > - info->Width = header->width; > - info->Height = header->height; > + info->Width = dds->header.width; > + info->Height = dds->header.height; > info->Depth = 1; > - info->MipLevels = header->miplevels ? header->miplevels : 1; > + info->MipLevels = dds->header.miplevels ? dds->header.miplevels : 1; > > - info->Format = dds_pixel_format_to_d3dformat(&header->pixel_format); > + info->Format = dds_pixel_format_to_d3dformat(&dds->header.pixel_format); > if (info->Format == D3DFMT_UNKNOWN) > return D3DXERR_INVALIDDATA; > > TRACE("Pixel format is %#x\n", info->Format); > > - if (header->caps2 & DDS_CAPS2_VOLUME) > + if (dds->header.caps2 & DDS_CAPS2_VOLUME) > { > - info->Depth = header->depth; > + info->Depth = dds->header.depth; > info->ResourceType = D3DRTYPE_VOLUMETEXTURE; > } > - else if (header->caps2 & DDS_CAPS2_CUBEMAP) > + else if (dds->header.caps2 & DDS_CAPS2_CUBEMAP) > { > DWORD face; > faces = 0; > for (face = DDS_CAPS2_CUBEMAP_POSITIVEX; face <= DDS_CAPS2_CUBEMAP_NEGATIVEZ; face <<= 1) > { > - if (header->caps2 & face) > + if (dds->header.caps2 & face) > faces++; > } > info->ResourceType = D3DRTYPE_CUBETEXTURE; > @@ -437,8 +442,8 @@ static HRESULT load_surface_from_dds(IDirect3DSurface9 *dst_surface, const PALET > { > UINT size; > UINT src_pitch; > - const struct dds_header *header = src_data; > - const BYTE *pixels = (BYTE *)(header + 1); > + const struct DDS *dds = src_data; > + const BYTE *pixels = (BYTE *)(dds + 1); I guess you could instead make the src_data parameter BYTE * and compute "pixels" as src_data + sizeof(struct dds) (or however you want to call the new struct). It's mostly a matter of taste though so it's fine to me either way.

2 1

Re: [PATCH 4/7] ddraw/tests: Add tests for invalid render states.
by Marvin Jan. 28, 2016

Jan. 28, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20023 Your paranoid android. === build (build) === Missing build status line The build timed out

2 1

Re: [PATCH v2] winspool.drv: If the call to cupsPrintFile fails, print the cups error message.
by Huw Davies Jan. 28, 2016

Jan. 28, 2016

On Wed, Jan 27, 2016 at 08:58:26AM -0600, Jeremy White wrote: > diff --git a/dlls/winspool.drv/info.c b/dlls/winspool.drv/info.c > index cebf6f7..89966ce 100644 > --- a/dlls/winspool.drv/info.c > +++ b/dlls/winspool.drv/info.c > @@ -781,6 +781,7 @@ static void *cupshandle; > DO_FUNC(cupsPrintFile) > #define CUPS_OPT_FUNCS \ > DO_FUNC(cupsGetNamedDest); \ > + DO_FUNC(cupsLastErrorString); \ > DO_FUNC(cupsGetPPD3) Could we keep these in alphabetical order please? Otherwise, looks good. Huw.

1 0

Re: [PATCH 5/7] ddraw/tests: Move other SetRenderState() tests to test_set_render_state().
by Marvin Jan. 28, 2016

Jan. 28, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20024 Your paranoid android. === w1064 (32 bit d3d) === d3d.c:2301: Test failed: Unexpected destination texture level pixels; 2031 differences at 0 level d3d.c:2301: Test failed: Unexpected destination texture level pixels; 518 differences at 1 level d3d.c:2301: Test failed: Unexpected destination texture level pixels; 138 differences at 2 level d3d.c:2301: Test failed: Unexpected destination texture level pixels; 46 differences at 3 level d3d.c:2485: Test failed: Unexpected destination texture level pixels; 257 differences d3d.c:2632: Test failed: Unexpected destination texture level pixels; 256 differences at 2 level d3d.c:2632: Test failed: Unexpected destination texture level pixels; 64 differences at 3 level d3d.c:2632: Test failed: Unexpected destination texture level pixels; 16 differences at 4 level d3d.c:2632: Test failed: Unexpected destination texture level pixels; 4 differences at 5 level === w1064 (32 bit ddraw7) === ddraw7.c:2254: Test failed: Expected {0, 0, 512, 384}, got {0, 0, 1024, 768}.

1 0

Re: [PATCH 6/6] d3d8/tests: Test the creation of render targets with invalid multisample settings.
by Marvin Jan. 27, 2016

Jan. 27, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20021 Your paranoid android. === w1064 (32 bit device) === device.c:2727: Test failed: Expected message 0x7e for window 0, but didn't receive it. device.c:2732: Test failed: The focus window is active. device.c:2740: Test failed: Got unexpected hr 0x88760869. device.c:2746: Test failed: Got unexpect screen size 800x600. device.c:2761: Test failed: Expected message 0x47 for window 0, but didn't receive it, i=1. device.c:2770: Test failed: Got unexpect screen size 800x600. device.c:2783: Test failed: Expected message 0x7e for window 0, but didn't receive it. device.c:2791: Test failed: Got unexpect width 640. device.c:2792: Test failed: Got unexpect height 480.

1 0

Re: [PATCH 5/6] d3d8/tests: Port test_checkdevicemultisampletype() from d3d9.
by Marvin Jan. 27, 2016

Jan. 27, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20020 Your paranoid android. === w1064 (32 bit device) === device.c:2668: Test failed: Expected message 0x7e for window 0, but didn't receive it. device.c:2673: Test failed: The focus window is active. device.c:2681: Test failed: Got unexpected hr 0x88760869. device.c:2687: Test failed: Got unexpect screen size 800x600. device.c:2702: Test failed: Expected message 0x47 for window 0, but didn't receive it, i=1. device.c:2711: Test failed: Got unexpect screen size 800x600. device.c:2724: Test failed: Expected message 0x7e for window 0, but didn't receive it. device.c:2732: Test failed: Got unexpect width 640. device.c:2733: Test failed: Got unexpect height 480.

1 0

Re: [PATCH 6/6] jscript: Added JSON tests.
by Marvin Jan. 27, 2016

Jan. 27, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20011 Your paranoid android. === wxppro (32 bit) === No test summary line found === w2003std (32 bit) === No test summary line found === wvistau64 (32 bit) === No test summary line found === w2008s64 (32 bit) === No test summary line found === w7u (32 bit) === No test summary line found === w7pro64 (32 bit) === No test summary line found === w8 (32 bit) === No test summary line found === w864 (32 bit) === No test summary line found === w1064 (32 bit) === No test summary line found === wvistau64 (64 bit) === No test summary line found === w2008s64 (64 bit) === No test summary line found === w7pro64 (64 bit) === No test summary line found === w864 (64 bit) === No test summary line found === w1064 (64 bit) === No test summary line found

1 0

Re: [PATCH 3/6] jscript: Added JSON object stub implementation.
by Marvin Jan. 27, 2016

Jan. 27, 2016

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 https://testbot.winehq.org/JobDetails.pl?Key=20010 Your paranoid android. === build (build) === Missing build status line The build timed out

1 0

Re: [PATCH] cmd: Consistently wrap lines to 75 characters.
by Francois Gouget Jan. 27, 2016

Jan. 27, 2016

On Fri, 3 Jul 2015, Alex Henrie wrote: > 75 characters is the maximum length of a line before gettext breaks it, > so this convention will cause each translatable string to fit snugly > into the po files. It's also about what we were doing anyway, just not > consistently. I think this patch and particularly its justification does not make any sense. The command line tool messages were wrapped to the typical width of a terminal: 80 characters. So with this change the messages now take more lines and lose in readability for the user. So now we gettext does not wrap the messages as much in the PO files. But that's totally irrelevant and invisible to the users. It's also totally irrelevant and invisible to a translator as soon as he uses a tool like poedit or a web GUI like Pootle. So what we have here is a change made solely to make things look a bit nicer to the few developers who look at PO files, at the cost of making them slightly worse for our users. Not a good tradeoff in my book and never a good justification for a patch. > I removed Hebrew "translations" of cmd.rc that were just copies of > the English strings, There was a reason for these. Has anything changed? -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ The last time religion ruled, it was called the dark ages.

2 5

New Wiki
by Jeremy Newman Jan. 27, 2016

Jan. 27, 2016

Hey all. We've migrated our Wiki to a new backend. At this point we've ported most of the English pages over. BIG BIG thanks to Rosanne for doing this migration work. Right now, the Wiki is locked down so only approved editors can make changes. So, if you need edit access, send an email to web-admin(a)winehq.org until we have a better system for approving editors in place. The old wiki is temporarily available at http://oldwiki.winehq.org If you need some content from there. I'm sure there will be plenty of things to fix. I also want to give serious kudos to Dimi and Lattica for previously hosting the Wiki. Thanks man! -N

5 7

Re: [PATCH] ntdll: Filling in memory counters under OS X.
by Ken Thomases Jan. 27, 2016

Jan. 27, 2016

Hi, Thanks for the submission. See my comments below. On Jan 27, 2016, at 10:19 AM, Snorri Sturluson <snorri.sturluson(a)ccpgames.com> wrote: > > +#ifdef __APPLE__ > +#include <mach/mach.h> > +#endif We're not totally consistent about this, but probably better to conditionalize this (and the code below) on HAVE_MACH_MACH_H instead of __APPLE__. > +#if defined(__APPLE__) > + > +void fill_VM_COUNTERS(VM_COUNTERS* pvmi) This should be "static" (in both branches of the #if). > +{ > + struct mach_task_basic_info info; > + mach_msg_type_number_t infoCount = MACH_TASK_BASIC_INFO_COUNT; > + if(task_info(mach_task_self(), MACH_TASK_BASIC_INFO, (task_info_t)&info, &infoCount) == KERN_SUCCESS) You need to conditionalize this on MACH_TASK_BASIC_INFO being defined. It is not defined in the 10.5 SDK and Wine needs to compile against that. (Well, Alexandre may have finally given up on 10.5, but it's also not in the 10.6 SDK.) If it's not defined, you can fall back to TASK_BASIC_INFO_64 / struct task_basic_info_64, although it doesn't have as much information. > + { > + pvmi->VirtualSize = info.resident_size + info.virtual_size; > + pvmi->PagefileUsage = info.virtual_size; > + pvmi->WorkingSetSize = info.resident_size; pvmi->PeakWorkingSetSize can be set from info.resident_size_max, right? The other two Peak* fields could be set based on the current info.virtual_size (possibly combined with resident_size_max), at least. You could also track the highest value seen by this code, which is not the same as the real peak but at least never decreases. You would have to be careful to do it in a thread-safe manner, probably with atomic compare-and-swap in a loop. > + } You can get PageFaultCount from TASK_EVENTS_INFO / struct task_events_info.faults. Regards, Ken

1 0

Releasing wine-1.8.1 Soon
by Michael Stefaniuc Jan. 27, 2016

Jan. 27, 2016

Hello guys, now that two proper devel versions are out I plan to release wine-1.8.1 next week on Monday or Tuesday (2nd or 3rd February). A list of included and queued patches can be found on https://static-winehq.rhcloud.com/stable/ Or directly via git from https://github.com/mstefani/wine-stable . I still need to pull in two bugs marked for 1.8.x (https://bugs.winehq.org/buglist.cgi?bug_status=RESOLVED&bug_status=CLOSED&l…) If you guys know other bugs or commits that should go into 1.8.1 please let me know (set 1.8.x in bugzilla, email, irc). The rules for acceptable patches for Stable are on https://wiki.winehq.org/Stable_Rules 1.8.x bugs deferred for 1.8.2 or later: - #35187 - Jade Empire Special Edition configuration tool always resetting video settings - #39838 - Wreckfest Server: Crashes at startup, unimplemented function KERNEL32.dll.GetConsoleScreenBufferInfoEx & SetConsoleScreenBufferInfoEx Out of scope for 1.8.x: - D3D10 / D3D11 as cherry-picking single commits makes no sense. Many thanks to the people that already provided me with input on patches for Stable. bye michael

1 0

crypt32:chain test failures
by Francois Gouget Jan. 27, 2016

Jan. 27, 2016

crypt32:chain has started having the following failures on the 21st on essentially all the VMs it runs on: https://test.winehq.org/data/tests/crypt32:chain.html chain.c:3875: Test failed: didn't expect CERT_TRUST_IS_NOT_VALID_FOR_USAGE, got 10 chain.c:3897: Test failed: didn't expect CERT_TRUST_IS_NOT_VALID_FOR_USAGE And yet it has not been modified recently. Based on the source it looks like these tests are running against Google's certificate. Could it be that that certificate changed recently thus breaking the tests? -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Indifference will certainly be the downfall of mankind, but who cares?

1 0