March 2017 - Wine-devel - list.winehq.org

Re: [PATCH 1/5] d3d11: Downgrade unknown DWORD FIXMEs() to TRACE() messages.
by Henri Verbeet 22 Mar '17

22 Mar '17

On 22 March 2017 at 09:40, Józef Kucia <jkucia(a)codeweavers.com> wrote: > @@ -727,11 +727,11 @@ void skip_dword_unknown(const char **ptr, unsigned int count) > unsigned int i; > DWORD d; > > - FIXME("Skipping %u unknown DWORDs:\n", count); > + TRACE("Skipping %u unknown DWORDs:\n", count); > for (i = 0; i < count; ++i) > { > read_dword(ptr, &d); > - FIXME("\t0x%08x\n", d); > + TRACE("\t0x%08x\n", d); > } > } > I think WARN would be appropriate.

1 0

Re: [PATCH 1/3] odbccp32: Implement SQLInstallTranslator
by Hans Leidekker 22 Mar '17

22 Mar '17

On Wed, 2017-03-22 at 00:14 +0000, Alistair Leslie-Hughes wrote: > + reg_ret = RegOpenKeyExA(HKEY_LOCAL_MACHINE, "Software\\ODBC\\ODBCINST.INI\\WINE ODBC Translator", 0, KEY_READ, &hkey); > + ok(reg_ret == ERROR_SUCCESS, "RegOpenKeyExW failed\n"); > + if (reg_ret == ERROR_SUCCESS) > + { > + DWORD type, size = MAX_PATH; > + char driverpath[MAX_PATH]; > + > + strcpy(driverpath, syspath); > + strcat(driverpath, "\\sample.dll"); > + > + reg_ret = RegGetValueA(hkey, NULL, "Translator", RRF_RT_REG_SZ, &type, &path, &size); > + ok(reg_ret == ERROR_SUCCESS, "RegGetValueA failed\n"); > + ok(!strcmp(path, driverpath), "invalid path %s\n", path); RegGetValueA is not available on XP. A warning dialog is shown but it's never closed, which can cause failures in tests that follow: https://testbot.winehq.org/JobDetails.pl?Key=29622&scrshot_101=1#k101 I've submitted a patch that avoids RegGetValueA. Please do something similar in these patches.

1 0

Re: [PATCH v2] msvcp100: Add stub for Concurrency::is_current_task_group_canceling
by Piotr Caban 22 Mar '17

22 Mar '17

Hi, Is any application working thanks to this patch (not crashing on next unimplemented function)? Thanks, Piotr

2 3

Outreachy Internships
by Iwona Kotlarska 21 Mar '17

21 Mar '17

Hi everyone, I'm new to contributing to Wine and I would like to apply to Outreachy Internship. I'm generally comfortable with C, but I don't understand Wine's code yet. I tried to download and build Wine on my computer, I followed this: https://wiki.winehq.org/Developer_FAQ#How_do_I_download_and_build_the_very_… and make fails (linking actually) with undefined reference to yywrap in function yylex in file programs/winhlp32/macro.lex.yy.c line 1032. I'm using gcc 5.4.0. Could anyone help me solve this compilation issue? And suggest some small bugs to work on after succesfully compiling wine? Thanks in advance for help! Iwona Kotlarska

4 3

Re: cmd: Adjust CMD_LIST to a tree-like structure and implement command chains
by Marvin 21 Mar '17

21 Mar '17

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=29617 Your paranoid android. === wxppro (32 bit) === Found no trace of a test summary line or of a crash === w2003std (32 bit) === Found no trace of a test summary line or of a crash === wvistau64 (32 bit) === Found no trace of a test summary line or of a crash === w2008s64 (32 bit) === Found no trace of a test summary line or of a crash === w7u (32 bit) === Found no trace of a test summary line or of a crash === w7pro64 (32 bit) === Found no trace of a test summary line or of a crash === w8 (32 bit) === Found no trace of a test summary line or of a crash === w864 (32 bit) === Found no trace of a test summary line or of a crash === w1064 (32 bit) === Found no trace of a test summary line or of a crash === wvistau64 (64 bit) === Found no trace of a test summary line or of a crash === w2008s64 (64 bit) === Found no trace of a test summary line or of a crash === w7pro64 (64 bit) === Found no trace of a test summary line or of a crash === w864 (64 bit) === Found no trace of a test summary line or of a crash === w1064 (64 bit) === Found no trace of a test summary line or of a crash

1 0

Re: [PATCH 3/4] xmllite: Set empty value for end element nodes.
by Nikolay Sivov 21 Mar '17

21 Mar '17

On 21.03.2017 18:19, Jacek Caban wrote: > Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> > --- > dlls/xmllite/reader.c | 1 + > 1 file changed, 1 insertion(+) > Could you make this another node type case in GetValue? Like for XmlNodeType_XmlDeclaration. The goal is to make sure empty static string is returned unconditionally for certain node types, as opposed to heap-allocated empty string.

2 1

Re: [PATCH v2 resend] user32: Don't call IsChild with NULL window in EnableWindow
by Marvin 21 Mar '17

21 Mar '17

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=29610 Your paranoid android. === w7u (32 bit win) === win.c:999: Test failed: wrong dwWindowStatus: 0000 != 0001 active 00A90148 fg 000F0138 in hook HCBT_MINMAX

1 0

Re: [6/7] [resend] version: Add support for FILE_VER_GET_LOCALISED flag.
by Alexandre Julliard 21 Mar '17

21 Mar '17

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > @@ -356,7 +356,10 @@ static BOOL find_pe_resource( HFILE lzfd, DWORD *resLen, DWORD *resOff ) > TRACE("No resid entry found\n" ); > goto done; > } > - resPtr = find_entry_language( resPtr, resDir ); > + if ( flags & FILE_VER_GET_LOCALISED ) > + resPtr = find_entry_language( resPtr, resDir ); > + else > + resPtr = find_entry_default( resPtr, resDir ); This won't do what you want, it will return the first available resource which may not be English. > @@ -642,12 +645,12 @@ DWORD WINAPI GetFileVersionInfoSizeExW( DWORD flags, LPCWSTR filename, LPDWORD h > SetLastError(ERROR_BAD_PATHNAME); > return 0; > } > - if (flags) > - FIXME("flags 0x%x ignored\n", flags); > + if (flags & ~FILE_VER_GET_LOCALISED) > + FIXME("flags 0x%x ignored\n", flags & ~FILE_VER_GET_LOCALISED); > > if ((lzfd = LZOpenFileW( (LPWSTR)filename, &ofs, OF_READ )) != HFILE_ERROR) > { > - magic = find_version_resource( lzfd, &len, &offset ); > + magic = find_version_resource( lzfd, &len, &offset, flags ); > LZClose( lzfd ); > } You'd also need to change the FindResourceW call a few lines below. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: wined3d: A grammar fix in a comment.
by Henri Verbeet 21 Mar '17

21 Mar '17

On 12 March 2017 at 20:47, Francois Gouget <fgouget(a)free.fr> wrote: > @@ -3645,8 +3645,8 @@ static void wined3d_adapter_init_limits(struct wined3d_gl_info *gl_info) > gl_info->gl_ops.gl.p_glGetIntegerv(GL_MAX_COMPUTE_TEXTURE_IMAGE_UNITS, &gl_max); > gl_info->limits.compute_samplers = gl_max; > TRACE("Max compute samplers: %u.\n", gl_info->limits.compute_samplers); > - /* A majority of OpenGL implementations allow to statically partition > - * the set of texture bindings into six separate sets. */ > + /* A majority of OpenGL implementations allow applications to statically > + * partition the set of texture bindings into six separate sets. */ I think that changes the meaning somewhat.

2 2

32-bit dlls in 64-bit process
by Viktor Semykin 21 Mar '17

21 Mar '17

Hello. I'm trying to run Adobe Photoshop Lightroom using Wine. One of the showstoppers was the fact that 64-bit process of Lightroom is loading 32-bit DLLs containing various resources. While Windows allows it [1 <https://support.microsoft.com/en-us/help/282423/list-of-limitations-in-64-b…>] wine does not. I made this simple change to Wine code just to make it work for my local usecase: > --- a/server/mapping.c > +++ b/server/mapping.c > @@ -414,8 +414,6 @@ static unsigned int get_image_params( struct > mapping *mapping, file_pos_t file_s > if (nt.opt.hdr32.Magic != IMAGE_NT_OPTIONAL_HDR32_MAGIC) > return STATUS_INVALID_IMAGE_FORMAT; > break; > case CPU_x86_64: > - if (nt.FileHeader.Machine != IMAGE_FILE_MACHINE_AMD64) return > STATUS_INVALID_IMAGE_FORMAT; > - if (nt.opt.hdr64.Magic != IMAGE_NT_OPTIONAL_HDR64_MAGIC) > return STATUS_INVALID_IMAGE_FORMAT; > break; > case CPU_POWERPC: > if (nt.FileHeader.Machine != IMAGE_FILE_MACHINE_POWERPC) > return STATUS_INVALID_IMAGE_FORMAT; From what I can tell this change looks neigher neat nor correct. My question is, what is /the/ correct way to allow loading 32-bit dll into 64-bit process just for loading its resources. Thank you. [1] https://support.microsoft.com/en-us/help/282423/list-of-limitations-in-64-b… -- best, Viktor.

1 0