- wine-devel - winehq.org

Re: Does auto-apt work to get wine dependencies in source code?
by Susan Cragin 26 Aug '13

26 Aug '13

1 0

Re: Use tcdrain() instead of tcflush() to implement FlushFileBuffers() for a COM port.
by Alexandre Julliard 26 Aug '13

26 Aug '13

Wolfgang Walter <wine(a)stwm.de> writes: > @@ -2749,19 +2752,46 @@ NTSTATUS WINAPI NtFlushBuffersFile( HANDLE hFile, IO_STATUS_BLOCK* IoStatusBlock > { > NTSTATUS ret; > HANDLE hEvent = NULL; > - > - SERVER_START_REQ( flush_file ) > - { > - req->handle = wine_server_obj_handle( hFile ); > - ret = wine_server_call( req ); > - hEvent = wine_server_ptr_handle( reply->event ); > + enum server_fd_type type; > + unsigned int options; > + int needs_close; > + int unix_handle; > + > + ret = server_get_unix_fd( hFile, FILE_WRITE_DATA, &unix_handle, > + &needs_close, &type, &options ); > + if (ret) return ret; You probably don't want to fail just because there's no unix fd. > + if (ret == STATUS_SUCCESS && IoStatusBlock) { > + IoStatusBlock->u.Status = ret; > } This is an unrelated change and should be a separate patch, with tests (also for failures cases), -- Alexandre Julliard julliard(a)winehq.org

2 3

Re: [PATCH 2/7] wined3d: Use PBOs for dynamic volumes (try 2)
by Matteo Bruni 26 Aug '13

26 Aug '13

2013/8/26 Stefan Dösinger <stefan(a)codeweavers.com>: > Try 2: > *) Require only GPU access for buffers > *) Reduce number of checkGLcall invocations > *) Remove a return from an ERR case > --- > dlls/wined3d/utils.c | 1 + > dlls/wined3d/volume.c | 168 ++++++++++++++++++++++++++++++++++++----- > dlls/wined3d/wined3d_private.h | 7 +- > 3 files changed, 157 insertions(+), 19 deletions(-) > > + case WINED3D_LOCATION_BUFFER: > + if (!volume->pbo || !(volume->flags & WINED3D_VFLAG_PBO)) > + ERR("Trying to load WINED3D_LOCATION_BUFFER without setting it up first.\n"); > + > + if (volume->locations & WINED3D_LOCATION_DISCARDED) > + { > + TRACE("Volume previously discarded, nothing to do.\n"); > + wined3d_volume_invalidate_location(volume, WINED3D_LOCATION_DISCARDED); > + } > + else if (volume->locations & WINED3D_LOCATION_TEXTURE_RGB) > + { > + struct wined3d_bo_address data = {volume->pbo, NULL}; > + volume_bind_and_dirtify(volume, context); > + wined3d_volume_download_data(volume, context, &data); > + } > + else > + { > + FIXME("Implement WINED3D_LOCATION_SYSMEM loading from %s.\n", > + wined3d_debug_location(volume->locations)); > + return; > + } > + wined3d_volume_validate_location(volume, WINED3D_LOCATION_BUFFER); > + break; > + Not really an actual review, but that FIXME looks wrong (copy-paste remnant probably).

2 1

questions to your commit "wineps.drv: Allow for vertical text printing!"
by Wolfgang Walter 26 Aug '13

26 Aug '13

Hello Aric, I read commit 745e7c93c9042f62460f181daaa1f05645560b41 (wineps.drv: Allow for vertical text printing.) and I have some questions: 1) You added the parameter vertical to PSDRV_WriteSetFont() in ps.c but it remains unused. I think it isn't needed. 2) In PSDRV_WriteSetDownloadFont() in download.c you call PSDRV_WriteSetFont() with (lf.lfFaceName[0] == '@') for this new parameter vertical. I can't see why. 3) You added the parameter vertical to get_download_name() in download.c You use it to add "_vertical" to its name if vertical is true. I can't see why the font needs a different name. This will cause PSDRV_WriteSetDownloadFont() to download a font which is used vertically and horizontally twice. I can't see any reason why this could be necessary. I think that * PSDRV_WriteSetFont() should be changed back and of course the callers, too (in builtin.c and download.c) * get_download_name() should also be changed back (and its call in PSDRV_WriteSetDownloadFont()) Regards, -- Wolfgang Walter Studentenwerk München

2 1

Re: [patch fix]for 98152 and 98151,the encoding type was wrong.
by Akihiro Sagawa 26 Aug '13

26 Aug '13

On Sun, 25 Aug 2013 23:13:53 +0900, matyapiro31 wrote: > The first one is > 0001-add-japanese-man-page-of-wine.patch > the second one is > 0001-last-change-of-my-translation-of-man-page.patch > Please cancel 98152 and 98151and try those two. These translations are outdated. The file seems to be based on wine 1.4 release. Makefile.in has a lot of formatting errors. You'd also refine it before resubmitting. Regards, Akihiro Sagawa

1 0

Re: [2/3] server: Use tcdrain() instead of tcflush() to implement FlushFileBuffers() for a COM port.
by Alexandre Julliard 26 Aug '13

26 Aug '13

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > MSDN for FlushFileBuffers says: > Flushes the buffers of a specified file and causes all buffered data to be > written to a file. > > Linux man page says: > tcdrain() waits until all output written to the object referred to by fd has > been transmitted. It's a blocking call, you can't do that on the server side. -- Alexandre Julliard julliard(a)winehq.org

3 3

Re: [PATCH 1/2] advapi32/tests: Add tests for hkcr handle mask
by Nikolay Sivov 24 Aug '13

24 Aug '13

On 8/24/2013 07:11, George Stephanos wrote: > +#define IS_HKCR(hk) ((UINT_PTR)hk > 0 && ((UINT_PTR)hk & 3) == 2) > + Why is that important to check? In other words what depends on particular value pattern?

1 0

Re: [PATCH 4/6] wined3d: Enforce volume mapping restrictions
by Alexandre Julliard 23 Aug '13

23 Aug '13

Stefan Dösinger <stefan(a)codeweavers.com> writes: > --- > dlls/wined3d/volume.c | 7 +++++++ > 1 file changed, 7 insertions(+) It breaks the tests: ../../../tools/runtest -q -P wine -M d3dx9_36.dll -T ../../.. -p d3dx9_36_test.exe.so volume.c && touch volume.ok volume.c:187: Test succeeded inside todo block: D3DXLoadVolumeFromMemory returned 0x8876086c, expected 0x8876086c make: *** [volume.ok] Error 1 -- Alexandre Julliard julliard(a)winehq.org

1 0

Wiki is down
by Rosanne DiMesio 23 Aug '13

23 Aug '13

I'm getting an "Unable to connect" message from Firefox and "Connection closed by remote server" from Opera. -- Rosanne DiMesio <dimesio(a)earthlink.net>

3 3

winmm: Increase mmdevapi buffer sizes.
by Joerg-Cyril.Hoehle＠t-systems.com 23 Aug '13

23 Aug '13

Hi, Andrew Eikum argued: -/* buffer size = 10 * 100000 (100 ns) = 0.1 seconds */ +/* buffer size = 100 * 100000 (100 ns) = 1 second */ >The tiny buffer size caused audio capture glitches on OSX, where the OS >often returns audio in half-second chunks. That may be good enough for winmm, but that does not solve the problem in general. Since Vista, apps can access mmdevapi directly, thus they won't benefit from this increase in winmm buffer size. I believe that instead and in general, Wine should decouple WinAPI usage from idiosyncrasies of the underlying OS. IOW, apps requesting a mmdevapi capture buffer of a size that typically works on native should work in Wine too. If 100-200ms is that typical size, then it must be made to work with Wine's back-ends, perhaps by using a hidden buffer of at least 1s *internally* in winecoreaudio (and presumably winealsa for when Jack is in use). Somehow, Wine must bridge the impedance mismatch between native's 100-200ms buffers and the observed >0.5s buffer requirements on OSX. BTW, does that imply that input signal processing will see data with 0.5s latency on OSX? Would there be a 0.5s delay in chat applications? Regards, Jörg Höhle

1 0

Re: [PATCH 5/6] wined3d: Use PBOs for dynamic volumes
by Henri Verbeet 23 Aug '13

23 Aug '13

On 22 August 2013 23:22, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > @@ -145,6 +158,9 @@ static DWORD volume_access_from_location(DWORD location) > case WINED3D_LOCATION_TEXTURE_RGB: > return WINED3D_RESOURCE_ACCESS_GPU; > > + case WINED3D_LOCATION_BUFFER: > + return WINED3D_RESOURCE_ACCESS_CPU | WINED3D_RESOURCE_ACCESS_GPU; > + Why does loading into a PBO require CPU access? > + case WINED3D_LOCATION_BUFFER: > + if (!volume->pbo || !(volume->flags & WINED3D_VFLAG_PBO)) > + { > + ERR("Trying to load WINED3D_LOCATION_BUFFER without setting it up first.\n"); > + return; > + } I think there should never be a PBO without WINED3D_VFLAG_PBO being set. There's also something to be said for leaving out the return, since that would at least in theory allow the compiler to drop the extra checks when compiled without debug messages. That probably applies to ERRs in general, since they're either never supposed to happen, or not supposed to be an ERR. > +/* Context activation is done by the caller. */ > +static void wined3d_volume_prepare_pbo(struct wined3d_volume *volume, struct wined3d_context *context) > +{ > + const struct wined3d_gl_info *gl_info = context->gl_info; > + > + if (volume->pbo) > + return; > + > + GL_EXTCALL(glGenBuffersARB(1, &volume->pbo)); > + checkGLcall("glGenBuffersARB(1, &volume->pbo)"); > + GL_EXTCALL(glBindBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB, volume->pbo)); > + checkGLcall("glBindBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB, volume->pbo)"); > + GL_EXTCALL(glBufferDataARB(GL_PIXEL_UNPACK_BUFFER_ARB, volume->resource.size, NULL, GL_STREAM_DRAW_ARB)); > + checkGLcall("glBufferDataARB(GL_PIXEL_UNPACK_BUFFER_ARB, volume->resource.size, NULL, GL_STREAM_DRAW_ARB"); > + GL_EXTCALL(glBindBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB, 0)); > + checkGLcall("glBindBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB, 0)"); > + > + TRACE("Created PBO %u for volume %p.\n", volume->pbo, volume); > +} I think one checkGLcall() per function is enough in general. That's even more true these days with ARB_debug_output that will usually give more specific error messages than checkGLcall() anyway. Applies to a couple of other places as well.

2 5

Re: Permissions bug with Acrobat X Standard
by Alexey Loukianov 23 Aug '13

23 Aug '13

Neil, thanks for reporting. Could you please re-report this bug to a place where reports should be posted - i.e. to Wine's Bugzilla which could be found at http://bugs.winehq.com/ ? Thanks in advance. --- Best regards, Alexey Loukianov HPC support and maintenance engineer, RSC Technologies http://rscgroup.ru/en/

1 0

Re: comsvcs: Add dll comsvcs (try 3)
by Nikolay Sivov 23 Aug '13

23 Aug '13

On 8/23/2013 13:18, Alistair Leslie-Hughes wrote: > Hi, > Fix build issue > > +#include "rpcproxy.h" I don't think you need this. > +HINSTANCE COMSVCS_hInstance = NULL; This should be static (and without initializer after that).

1 0

Re: server: Store full sized completion information on the server side.
by Alexandre Julliard 23 Aug '13

23 Aug '13

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > @@ -152,7 +152,7 @@ struct completion *get_completion_obj( struct process *process, obj_handle_t han > } > > void add_completion( struct completion *completion, apc_param_t ckey, apc_param_t cvalue, > - unsigned int status, unsigned int information ) > + apc_param_t information, unsigned int status ) There's no reason to change the parameter order. That's only adding noise to the patch, and potential bugs. -- Alexandre Julliard julliard(a)winehq.org

1 0

Request to delete some attachments
by Ken Sharp 22 Aug '13

22 Aug '13

Could a Bugzilla admin please delete the attachments in http://bugs.winehq.org/show_bug.cgi?id=24867 ? I don't know if metatester.exe can be freely distributed.

2 1

Re: wined3d: Recognize Nvidia GeForce GTX 770
by Henri Verbeet 22 Aug '13

22 Aug '13

On 22 August 2013 03:31, Gediminas Jakutis <gediminas(a)varciai.lt> wrote: > + {"GTX 770", CARD_NVIDIA_GEFORCE_GTX770}, /* Geforce 700 - highend */ > {"GTX 770M", CARD_NVIDIA_GEFORCE_GTX770M}, /* Geforce 700 - midend high mobile */ That doesn't work, it will always match "GTX 770" now, and never "GTX 770M".

1 0

Re: [PATCH] winemac.drv: Support the public UTF-16 type for Unicode text. (try 2)
by Ken Thomases 22 Aug '13

22 Aug '13

On Aug 22, 2013, at 12:07 AM, Charles Davis wrote: > Try 2: Eliminiate dead store. Sort export functions by name. Looks OK to me. -Ken

1 0

Permissions bug with Acrobat X Standard
by Neil Hellfeldt 22 Aug '13

22 Aug '13

err:msi:cabinet_copy_file failed to create L"C:\\users\\Public\\Application Data\\Adobe\\Adobe PDF\\Settings\\Standard.joboptions" (error 3) err:msi:extract_cabinet FDICopy failed err:msi:ACTION_InstallFiles Failed to extract cabinet: L"Data1.cab" err:msi:ITERATE_Actions Execution halted, action L"InstallFiles" returned 1603 err:msi:ITERATE_Actions Execution halted, action L"ExecuteAction" returned 1603 Sysinfo: lubuntu 3.04 64bit WINEARCH=win32 When I go to the parent directory I find this location: ~/.local/share/wineprefixes/acrobatxstd/drive_c/users/Public/Application Data/Adobe drwxrwxr-x 3 hellfeldt hellfeldt 4.0K Aug 21 17:12 Acrobat/ dr-xr-xr-x 3 hellfeldt hellfeldt 4.0K Aug 21 17:12 Adobe PDF/ ~/.local/share/wineprefixes/acrobatxstd/drive_c/users/Public/Application Data/Adobe$ cd Adobe\ PDF/ ~/.local/share/wineprefixes/acrobatxstd/drive_c/users/Public/Application Data/Adobe/Adobe PDF$ mkdir Settings mkdir: cannot create directory ‘Settings’: Permission denied I tried changing the directory permissions and I tried creating the directory but acrobat delete the Adobe PDF directory and recreates it with the bad permissions again.

1 0

Re: [PATCH] winemac.drv: Support the public UTF-16 type for Unicode text.
by Ken Thomases 22 Aug '13

22 Aug '13

On Aug 21, 2013, at 9:42 PM, Charles Davis wrote: > @@ -90,6 +93,9 @@ static CFDataRef export_hdrop_to_filenames(HANDLE data); > static CFDataRef export_oemtext_to_utf8(HANDLE data); > static CFDataRef export_text_to_utf8(HANDLE data); > static CFDataRef export_unicodetext_to_utf8(HANDLE data); > +static CFDataRef export_oemtext_to_utf16(HANDLE data); > +static CFDataRef export_text_to_utf16(HANDLE data); > +static CFDataRef export_unicodetext_to_utf16(HANDLE data); Please keep the export_* functions sorted (although I'm fine with utf8 coming before utf16). This applies to the function definitions, too. (Yes, this is an exceedingly minor issue. Sorry.) > +static HANDLE import_utf16_to_unicodetext(CFDataRef data) > +{ > + const WCHAR *src; > + unsigned long data_len; > + unsigned long new_lines = 0; > + LPWSTR dst; > + unsigned long i, j; > + HANDLE unicode_handle = NULL; This is an unnecessary initialization / dead store, which is frowned upon. Thanks! -Ken

2 2

Re: [PATCH 6/6] wined3d: Implement basic volume location management (try 2)
by Henri Verbeet 22 Aug '13

22 Aug '13

On 22 August 2013 14:57, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > + if (volume->locations & WINED3D_LOCATION_DISCARDED) > + { > + TRACE("Volume previously discarded, nothing to do.\n"); > + wined3d_volume_invalidate_location(volume, WINED3D_LOCATION_DISCARDED); > + } I don't think the location invalidation here is strictly required, although it probably doesn't hurt much in practice. (I.e., consider the sequence DISCARD, load SYSMEM, load TEXTURE_RGB. There's no real reason the RGB load needs to do an upload from SYSMEM in that case, although it's perhaps not something that's going to happen much in practice either way.) > + wined3d_volume_validate_location(volume, WINED3D_LOCATION_SYSMEM); You could probably just have a wined3d_volume_validate_location(volume, location); at the end of the function and return from the default case instead. That's not something I feel strongly about though.

2 1

Re: oleaut32/tests: Some tests for LoadRegTypeLib() with activated context
by Alexandre Julliard 22 Aug '13

22 Aug '13

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > Some tests for LoadRegTypeLib() with activated context > > From 1ea3d8c82dd77ac14f2cdec748c0284032958b2f Mon Sep 17 00:00:00 2001 > From: Nikolay Sivov <nsivov(a)codeweavers.com> > Date: Wed, 21 Aug 2013 12:34:47 +0400 > Subject: [PATCH 5/6] Some tests for LoadRegTypeLib() with activated context It doesn't work here: ../../../tools/runtest -q -P wine -M oleaut32.dll -T ../../.. -p oleaut32_test.exe.so tmarshal.c && touch tmarshal.ok tmarshal.c:999: Test failed: LoadRegTypeLib failed with error 0x8002801d tmarshal.c:1232: Test failed: IKindaEnumWidget_Next failed with error 0x8007000e wine: Unhandled page fault on read access to 0x00000000 at address 0x68640714 (thread 003a), starting debugger... -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH 5/5] wined3d: Implement basic volume location management
by Henri Verbeet 22 Aug '13

22 Aug '13

On 22 August 2013 00:30, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > +const char *wined3d_debug_location(DWORD location) > +{ > + char buf[238]; That number looks a bit arbitrary, although I suppose it's at least large enough. > + if (volume->resource.pool == WINED3D_POOL_SYSTEM_MEM > + || volume->resource.pool == WINED3D_POOL_SCRATCH) > + { > + if (location & ~WINED3D_LOCATION_SYSMEM) > + { > + ERR("Trying to load a sysmem or scratch volume into %s.\n", > + wined3d_debug_location(location)); > + return; > + } > + } In principle we have resource_access_from_location() and the access_flags field in struct wined3d_resource for this. It's only a warning in surface_load_location() because we load e.g. default pool surfaces into sysmem for CPU fallbacks, but that doesn't apply to volumes. The right way for surfaces would probably be to use a staging resource for CPU fallbacks instead, or something along those lines. > + if (count_bits(location) > 1) > + FIXME("Implement loading two locations with one call\n"); > + I don't think we really want that. > +static void wined3d_volume_update_location(struct wined3d_volume *volume, DWORD location, BOOL keep) > +{ > + if (keep) > + { > + TRACE("Volume %p, invalidating %s.\n", volume, > + wined3d_debug_location(volume->locations & ~location)); > + volume->locations = location; > + } > + else > + { > + TRACE("Volume %p, invalidating %s.\n", volume, wined3d_debug_location(location)); > + volume->locations &= ~location; > + } > +} I'm a bit unhappy with the "persistent" flag to surface_modify_location(), and this is essentially the same thing. I think it would be better to just have two separate functions, wined3d_volume_invalidate_location() and wined3d_volume_validate_location(). Perhaps wined3d_volume_update_location() can work too, but in that case I think it should take two location masks, one to set and one to clear.

2 8

Re: [4/4] oledb32: A basic test for IRowPosition_Initialize()
by Marvin 22 Aug '13

22 Aug '13

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=26813 Your paranoid android. === WINEBUILD (build) === Make failed

2 1

Re: gdi32: some fix of errors writing Arabic on entry boxs
by Yaron Shahrabani 22 Aug '13

22 Aug '13

Ahlan Ya Mas3ab! How can I test it with Hebrew? Yaron Shahrabani <Hebrew translator> On Wed, Aug 21, 2013 at 11:19 PM, مصعب الزعبي <moceap(a)hotmail.com> wrote: > Written By Aric Stewart , fix about 50 percent of Bug 33794 > > --- > dlls/gdi32/font.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > > >

1 0

Re: msvcrt: _[w]access_s returns an error code.
by Marvin 21 Aug '13

21 Aug '13

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=26809 Your paranoid android. === WXPX64 (32 bit msvcr90) === No test summary line found

1 0

RFC: winedev guide multimedia chapter patch
by Frédéric Delanoy 21 Aug '13

21 Aug '13

Hi guys, I've been updating the wine multimedia paths of the wine dev guide, but I'm not really familiar with that area of Wine, so could you please have a look at the attached patch to check if it seems correct? I checked the code a bit but I'm not sure how outdated the guide is. For easier "testing", just clone http://source.winehq.org/git/docs.git, apply the patch and run ./configure, then 'make winedev-guide.html' in the "en/" subdirectory. Thanks, Frédéric Delanoy

2 1

Re: [PATCH] winemac.drv: Use the public UTF-16 type for Unicode text.
by Ken Thomases 21 Aug '13

21 Aug '13

On Aug 20, 2013, at 10:49 PM, Charles Davis wrote: > In the Windows world, "Unicode" almost universally means "UTF-16". So, > use the well-known UTF-16 type instead of making up our own. > > I have to wonder if there was a good reason Ken didn't use this > initially. Please hold this patch while I review it. I think there is a good reason why I didn't use it, but I have to figure it out again. :-/ It has to do with a promised pasteboard type (what Microsoft calls delayed rendering) and the conversions to the other text types. I think we need to use a custom type to distinguish between being asked for promised data vs. being asked for a conversion. -Ken

2 2

Re: mscoree: Attempt to load the assembly from the application directory (try 2)
by Vincent Povirk 21 Aug '13

21 Aug '13

+ /* Assume the namespace is the Assembly name */ Uh, what? We should only be passing an assembly name to get_file_from_strongname, not the class name. You shouldn't even have access to the class name. I think what you're actually doing here is skipping the assembly name and using the assembly version, which won't work.

1 0

Re: wined3d: Add a function for allocating aligned resource, memory (try 3)
by Alexandre Julliard 21 Aug '13

21 Aug '13

Stefan Dösinger <stefan(a)codeweavers.com> writes: > This supersedes patch 98009. It renames the allocation and free > functions as suggested by Henri. It doesn't work: ../../../tools/runtest -q -P wine -M d3dx9_36.dll -T ../../.. -p d3dx9_36_test.exe.so texture.c && touch texture.ok texture.c:555: Tests skipped: No D3DUSAGE_AUTOGENMIPMAP support for volume textures wine: Unhandled page fault on write access to 0x00000000 at address 0x7bc4a5cf (thread 003c), starting debugger... Unhandled exception: page fault on write access to 0x00000000 in 32-bit code (0x7bc4a5cf). Register dump: CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b EIP:7bc4a5cf ESP:0032f79c EBP:0032f898 EFLAGS:00010202( R- -- I - - - ) EAX:00000000 EBX:7bcbc474 ECX:00164268 EDX:68d795a0 ESI:00110014 EDI:001642c8 Stack dump: 0x0032f79c: 7bc37d5d 00000050 00000002 00164278 0x0032f7ac: 00000000 00000000 00000000 00000000 0x0032f7bc: 687c05d8 00000001 00000000 00000000 0x0032f7cc: 4d430003 00000000 00000000 00000000 0x0032f7dc: 00000000 00000000 4d430001 00164268 0x0032f7ec: 00000010 00000000 00000000 00000000 Backtrace: =>0 0x7bc4a5cf HEAP_CreateFreeBlock+0x36f(subheap=0x110014, ptr=0x164268, size=0x60) [/home/julliard/wine/wine/dlls/ntdll/../../include/wine/list.h:101] in ntdll (0x0032f898) 1 0x7bc4a878 HEAP_MakeInUseBlockFree+0x157(subheap=0x110014, pArena=<is not available>) [/home/julliard/wine/wine/dlls/ntdll/heap.c:656] in ntdll (0x0032f938) 2 0x7bc4b8e2 RtlFreeHeap+0xd1(heap=<couldn't compute location>, flags=<couldn't compute location>, ptr=<couldn't compute location>) [/home/julliard/wine/wine/dlls/ntdll/heap.c:1771] in ntdll (0x0032f9a8) 3 0x68e0642e wined3d_resource_free_sysmem+0x2d(mem=0x1642c0) [/home/julliard/wine/wine/dlls/wined3d/resource.c:361] in wined3d (0x0032f9d8) 4 0x68e06503 resource_cleanup+0xc2(resource=0x1651e0) [/home/julliard/wine/wine/dlls/wined3d/resource.c:168] in wined3d (0x0032fa28) 5 0x68e6e671 wined3d_volume_decref+0xb0(volume=<couldn't compute location>) [/home/julliard/wine/wine/dlls/wined3d/volume.c:142] in wined3d (0x0032fa68) 6 0x68e5eb42 texture3d_sub_resource_cleanup+0x31(sub_resource=0x1651e0) [/home/julliard/wine/wine/dlls/wined3d/texture.c:1125] in wined3d (0x0032fa98) 7 0x68e5ea8e wined3d_texture_cleanup+0x4d(texture=0x164c70) [/home/julliard/wine/wine/dlls/wined3d/texture.c:126] in wined3d (0x0032fae8) 8 0x68e5fba9 wined3d_texture_decref+0x48(texture=<couldn't compute location>) [/home/julliard/wine/wine/dlls/wined3d/texture.c:449] in wined3d (0x0032fb28) 9 0x68d6701b d3d9_texture_3d_Release+0x5a() in d3d9 (0x0032fb88) 10 0x686a82ea func_texture+0x24d9() [/home/julliard/wine/wine/dlls/d3dx9_36/tests/texture.c:1595] in d3dx9_36_test (0x0032fd38) 11 0x6862c3d7 main+0x386(argc=<is not available>, argv=<is not available>) [/home/julliard/wine/wine/dlls/d3dx9_36/tests/../../../include/wine/test.h:570] in d3dx9_36_test (0x0032fe08) 12 0x686adcc0 __wine_spec_exe_entry+0x7f(peb=<couldn't compute location>) [/home/julliard/wine/wine/dlls/winecrt0/exe_entry.c:36] in d3dx9_36_test (0x0032fe58) 13 0x7b85eb8c call_process_entry+0xb() in kernel32 (0x0032fe78) 14 0x7b85fd6b start_process+0x6a(peb=<couldn't compute location>) [/home/julliard/wine/wine/dlls/kernel32/process.c:1084] in kernel32 (0x0032feb8) 15 0x7bc7a170 call_thread_func_wrapper+0xb() in ntdll (0x0032fed8) 16 0x7bc7d07d call_thread_func+0x7c(entry=0x7b85fd00, arg=0x7ffdf000, frame=0x32ffc8) [/home/julliard/wine/wine/dlls/ntdll/signal_i386.c:2602] in ntdll (0x0032ffa8) 17 0x7bc7a14e call_thread_entry_point+0x11() in ntdll (0x0032ffc8) 18 0x7bc4fade start_process+0x1d(kernel_start=0x7b85fd00) [/home/julliard/wine/wine/dlls/ntdll/loader.c:2694] in ntdll (0x0032ffe8) 19 0x680311ad wine_call_on_stack+0x1c() in libwine.so.1 (0x00000000) 20 0x6803126b wine_switch_to_stack+0x2a(func=0x7bc4fac0, arg=0x7b85fd00, stack=0x330000) [/home/julliard/wine/wine/libs/wine/port.c:59] in libwine.so.1 (0xffd46e68) 21 0x7bc55617 LdrInitializeThunk+0x3a6(kernel_start=<couldn't compute location>, unknown2=<couldn't compute location>, unknown3=<couldn't compute location>, unknown4=<couldn't compute location>) [/home/julliard/wine/wine/dlls/ntdll/loader.c:2750] in ntdll (0xffd46ed8) 22 0x7b8662e0 __wine_kernel_init+0xbbf() [/home/julliard/wine/wine/dlls/kernel32/process.c:1256] in kernel32 (0xffd47de8) 23 0x7bc55ce3 __wine_process_init+0x182() [/home/julliard/wine/wine/dlls/ntdll/loader.c:2959] in ntdll (0xffd47e78) 24 0x6802edf2 wine_init+0x2a1(argc=0x3, argv=0xffd483b4, error="", error_size=0x400) [/home/julliard/wine/wine/libs/wine/loader.c:847] in libwine.so.1 (0xffd47ed8) 25 0x7bf00d3b main+0x7a(argc=<is not available>, argv=<is not available>) [/home/julliard/wine/wine/loader/main.c:237] in <wine-loader> (0xffd48318) 26 0x682518c5 __libc_start_main+0xf4() in libc.so.6 (0x00000000) 0x7bc4a5cf HEAP_CreateFreeBlock+0x36f [/home/julliard/wine/wine/dlls/ntdll/../../include/wine/list.h:101] in ntdll: movl %edx,0x0(%eax) 101 elem->prev->next = elem->next; -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: ieframe: Use winehq snapshot instead of the main page in tests.
by Marvin 21 Aug '13

21 Aug '13

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=26794 Your paranoid android. === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2631: Test failed: expected GetOverridesKeyPath webbrowser.c:2636: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2637: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3079: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 004033D1 === WXPPROSP2 (32 bit webbrowser) === webbrowser.c:792: Test failed: ReadyState = 1, expected 4

1 0

Re: [PATCH 2/2] msvcrt: Fix lookahead buffer usage on pipes (try2)
by Piotr Caban 21 Aug '13

21 Aug '13

Please ignore this patch, I'll send a better version.

1 0

Re: [5/6] ntdll: Make NtReadFile always initialize a being returned IO_STATUS_BLOCK for regular files.
by Alexandre Julliard 21 Aug '13

21 Aug '13

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > @@ -748,6 +748,11 @@ err: > } > else > { > + if (type == FD_TYPE_FILE) > + { > + io_status->u.Status = status; > + io_status->Information = total; > + } I don't see why this would have to be specific to files. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: [PATCH 2/2] msvcrt: Fix lookahead buffer usage on pipes
by Alexandre Julliard 21 Aug '13

21 Aug '13

Piotr Caban <piotr(a)codeweavers.com> writes: > --- > dlls/msvcrt/file.c | 15 +++++++++++++-- > 1 file changed, 13 insertions(+), 2 deletions(-) It doesn't work: ../../../tools/runtest -q -P wine -M msvcrt.dll -T ../../.. -p msvcrt_test.exe.so file.c && touch file.ok file.c:2042: Test failed: read returned 2, expected 1 make[1]: *** [file.ok] Error 1 -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [6/6] server: Store full sized completion information on the server side.
by Marvin 21 Aug '13

21 Aug '13

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=26791 Your paranoid android. === W2KPROSP4 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPPROSP2 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPX64 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === W2K3R2SESP2 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === W2K8SE (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPX64 (64 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011

1 0

Re: [5/6] ntdll: Make NtReadFile always initialize a being returned IO_STATUS_BLOCK for regular files.
by Marvin 21 Aug '13

21 Aug '13

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=26790 Your paranoid android. === W2KPROSP4 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPPROSP2 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPX64 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === W2K3R2SESP2 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === W2K8SE (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPX64 (64 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011

1 0

Re: [4/6] ntdll/tests: Mark the NtReadFile call which returns STATUS_SUCCESS for an overlapped IO call as broken, and don't
by Marvin 21 Aug '13

21 Aug '13

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=26789 Your paranoid android. === W2KPROSP4 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPPROSP2 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPX64 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === W2K3R2SESP2 (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === W2K8SE (32 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011 === WXPX64 (64 bit file) === file.c:785: Test failed: expected STATUS_PENDING, got 0xc0000011

1 0

Re: oledb32: Add IDataSourceLocator support (try 6)
by Nikolay Sivov 21 Aug '13

21 Aug '13

On 8/21/2013 10:33, Alistair Leslie-Hughes wrote: > +typedef struct DataSourceImpl > +{ > + IDataSourceLocator IDataSourceLocator_iface; > + LONG ref; > + > +} DataSourceImpl; All good, except this one.

1 0

Re: [4/6] kernel32: Make serial tests self-contained.
by Saulius Krasuckas 21 Aug '13

21 Aug '13

* On Thu, 15 Aug 2013, Dmitry Timoshkov wrote: > dlls/kernel32/tests/comm.c | 284 ++++++++++++++++++++++++++------------------- > 1 file changed, 167 insertions(+), 117 deletions(-) ... > --- a/dlls/kernel32/tests/comm.c > +++ b/dlls/kernel32/tests/comm.c > @@ -1486,7 +1588,7 @@ static void test_WaitRing(HANDLE hcom) > success, err, evtmask, after-before, after1-before); > > ok(evtmask & EV_RING, "Failed to detect EV_RING: 0x%08x, expected 0x%08x\n", > - evtmask, EV_RING); > + evtmask, EV_CTS); Is everything OK with the constants here? S.

1 0

Re: oledb32: Add IDataSourceLocator support (try 5)
by Nikolay Sivov 21 Aug '13

21 Aug '13

On 8/21/2013 10:16, Alistair Leslie-Hughes wrote: > Hi, > Updated naming. You updated method names only, what's left : > +typedef struct DataSourceImpl > +HRESULT create_data_source(IUnknown *outer, void **obj) > +static const IDataSourceLocatorVtbl DataSourceVtbl = Also you most likely don't need these: > +#include "wine/unicode.h" > +#include "wine/list.h" > > > Changelog: > oledb32: Add IDataSourceLocator support > > > Best Regards > Alistair Leslie-Hughes > > >

1 0

Re: oledb32: Add IDataSourceLocator support (try 4)
by Nikolay Sivov 21 Aug '13

21 Aug '13

On 8/21/2013 09:15, Alistair Leslie-Hughes wrote: > Hi, > Use helper functions for memory > > > Changelog: > oledb32: Add IDataSourceLocator support > > > Best Regards > Alistair Leslie-Hughes It looks okay, but I suggest to fix a naming: > +typedef struct DataSourceImpl > +{ > + IDataSourceLocator IDataSourceLocator_iface; > + LONG ref; > + > +} DataSourceImpl; > +static HRESULT WINAPI datasource_QueryInterface(IDataSourceLocator *iface, REFIID riid, void **ppvoid) etc. IDataSource is another interface, and it's confusing to name locator after it - dslocator_* naming used in CF is short enough and descriptive to use it everywhere.

1 0

Willing to join Wine
by Priyank Bhatt 20 Aug '13

20 Aug '13

I want to work for Wine . I'm a third year undergraduate Student , I have done my elimentary course on machine learning algorithm . I am also familiar with C/C++ and some what of Python . Please suggest what all do I need to learn or read so as to start contributing to the open source community of mlpack. As an amateur and as per your free slots what project form the idea's page should I work on ?

2 1

Re: winmm: fix WOD_Open() WAVE_MAPPER no-device case.
by Andrew Eikum 20 Aug '13

20 Aug '13

On Tue, Aug 20, 2013 at 05:57:34AM +0200, Fabrice Bauzac wrote: > Hello, > > This patch intends to fix the bug: > http://bugs.winehq.org/show_bug.cgi?id=34305 > > The bug: basically, when there is no sound device and the user tries > to WOD_Open() the WAVE_MAPPER pseudodevice, the code calls read_map() > with device index 0 without first checking whether the number of > devices is greater than 0. > Looks good to me, thanks. I tested on Windows 7, and the error code is correct. By the way, the waveIn path (WID_Open) has the same bug. Feel free to fix that, too, if you want. Or I'll submit a patch myself later. Andrew

2 2

Re: [PATCH 2/4] oleaut32: Implement ICreateTypeInfo::SetTypeDescAlias
by Piotr Caban 20 Aug '13

20 Aug '13

On 08/20/13 16:57, Andrew Eikum wrote: > @@ -10530,8 +10541,28 @@ static HRESULT WINAPI ICreateTypeInfo2_fnSetTypeDescAlias(ICreateTypeInfo2 *ifac > + This->cbSizeInstance = VARIANT_DataSize(This->tdescAlias->vt, This->pTypeLib->ptr_size); > + if(!This->cbSizeInstance){ > + FIXME("Wrong size for variant type! 0x%x\n", This->tdescAlias->vt); > + This->cbSizeInstance = 4; > + } > + > + This->cbAlignment = 4; The This->cbAlignment should be also depending on alias type (e.g. it's 1 for VT_I1).

1 0

Re: Sort the AUTHORS file by their last names.
by Ken Sharp 20 Aug '13

20 Aug '13

On 16/08/13 14:46, Tae Wong wrote:> The AUTHORS file have all developers from the GIT log which were > sorted on their first names. > > This patch will switch the sort order from first to last names. This isn't how the AUTHORS file is generated. http://source.winehq.org/git/wine.git/commit/5da31c8817293ff98b4defceb962f7…

5 14

Re: [PATCH 1/5] wined3d: Add a function for allocating aligned resource memory (try 2)
by Henri Verbeet 20 Aug '13

20 Aug '13

On 20 August 2013 12:07, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > +void *wined3d_alloc_resource_mem(SIZE_T size) > +{ > + void **p; > + SIZE_T align = RESOURCE_ALIGNMENT - 1 + sizeof(*p); > + void *mem; > + > + if (!(mem = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, size + align))) > + return NULL; > + > + p = (void **)(((ULONG_PTR)mem + align) & ~(RESOURCE_ALIGNMENT - 1)) - 1; > + *p = mem; > + > + return ++p; > +} I think this should either be called wined3d_resource_allocate_sysmem() and live in resource.c, or accept any np2 alignment and be called something along the lines of wined3d_memalign(). The former probably makes the most sense at this point. Somewhat related, I don't think there's really a reason to have both "allocatedMemory" and "heap_memory" anymore. It should probably just be called "sysmem". That's a cleanup that doesn't necessarily belong in this patch though.

2 2

Re: kernel32: Fix required buffer length for locale values from registry
by Alexandre Julliard 20 Aug '13

20 Aug '13

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > http://bugs.winehq.org/show_bug.cgi?id=32169 > > Locale info to override is stored as REG_SZ and supposed to be > null-terminated, They are supposed to, but it's not guaranteed. The code should be able to cope with non-terminated strings. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: oledb32: Implement IErrorRecord AddErrorRecord
by Nikolay Sivov 20 Aug '13

20 Aug '13

On 8/20/2013 13:21, Alistair Leslie-Hughes wrote: > Hi, > Yes we do just take the pointer (pErrorInfo) passed in. MSDN states > that the caller responsible for this memory. It means caller could free it at any time (or it could be a stack variable), so you have to make a deep copy. > > > Changelog: > oledb32: Implement IErrorRecord AddErrorRecord > > > Best Regards > Alistair Leslie-Hughes > >

2 1

Re: riched20: Use \ulnone instead of \ul0 for no underline.
by Huw Davies 20 Aug '13

20 Aug '13

On Mon, Aug 19, 2013 at 05:38:13PM -0500, Vincent Povirk wrote: > Testing on Windows (by saving rtf documents with wordpad and viewing > them with notepad) shows that it uses \ulnone instead of \ul0 to end > underline. Wine and Windows are both able to read either one. I think these are symonyms and indeed Windows seems to use \ulnone, so this looks good to me. Huw.

1 0

Re: Could you update/rerun that report on bugs without AppDB links?
by Dan Kegel 19 Aug '13

19 Aug '13

I don't seem to have saved the (crappy) script. There is something similar at https://code.google.com/p/winezeug/source/browse/#svn%2Ftrunk%2Fappdb-mashup and it wouldn't be too hard to whip something up again for somebody who is motivated. Please do throttle the scraping script to avoid overloading the databases if you try. On Sun, Aug 18, 2013 at 12:11 PM, Rosanne DiMesio <dimesio(a)earthlink.net> wrote: > Dan, > > I don't know if you've been following the "PLEASE add bug links!" thread on wine-devel, so I figured I'd just contact you directly. Do you remember that report on unlinked bugs you ran about 5 years ago? http://kegel.com/wine/unlinked.html > > Could you update/run it again against current bugzilla? If we post a link to the results on the forum, we could ask users to help clear the backlog. > > -- > Rosanne DiMesio <dimesio(a)earthlink.net>

2 1

Re: [website] Added templates/zh-cn/cvs.template translation
by Hin-Tak Leung 19 Aug '13

19 Aug '13

------------------------------ On Sat, Aug 17, 2013 09:02 BST Frédéric Delanoy wrote: >On Sat, Aug 17, 2013 at 5:57 AM, xulixin <xulixin1990(a)gmail.com> wrote: >> Should I change my name's spelling to Lixin Xu, or LiXin Xu, or other >> format? > >Not sure, most people (at least non-Chinese) use firstname lastname >convention, but I guess you should use the same convention as other >Chinese-speaking committers. > >Here's a list I extracted from AUTHORS file that *look* Chinese to me >(don't know if they're really Chinese, or Korean, or from another >Asian country): >Chae Jong Bin >Cheer Xiao >Chia-I Wu >Hwang YunSong (황윤성) >Jau-Horng Chen >Jay Yang >Qian Hong >Qingchuan Wang >Qingdoa Daoo >Shi Quan He >Wei-Lun Chao >Weisheng Li >Xiang Li >XueFeng Chang >Yong Chi >Yuxi Zhang >Zhangrong Huang >Zhan Jianyu > >Hope this helps, > >Frédéric Delanoy Many had replied but few had touched the _main_ issue: it is a matter of policy that patches of an anonymous origin to Wine are categorically rejected. The Linux kernel also has a similar policy. This policy is due to past threats of copyright/intellectual-property infringements and possible cause of contamination from people who may have privileged access to proprietary technology. So a "real name" is needed for any patches. As already discussed, "xulixin" is a real name - as it appears on official documents, passports, etc. Although for the benefit of Western custom, "Xu Li-Xin", or "Xu Lixin", or "Lixin Xu" or "Li-Xin Xu" might make it more obvious, and make some people happier. Ultimately it is Alexandre's call, what is considered a real name. Anyway, the git command to set a default user name for generating patches is, e.g. for mine: git config --global user.name "Hin-Tak Leung" (this writes to ~/.gitconfig, which is a text files that one can edit by hand as well). See "man git-config" for details. Without --global, these config's are set per repository. e.g. you may use different names and e-mail addresses for internal (job-wise) work and external (open) work. I also have a few other global configs of my own; the equivalent of these two are probably useful to most people: git config --global ui.color true and git config --global core.editor emacs (the former sets colorization on for "git log", "git diff", etc; I find it useful of having diff's colorized - that assumes that you do all your development work on a terminal that's color-capable (most recent linux consoles/terminals are); the latter sets the default editor to emacs, when composing commit messages. You may want to set it to your favorite editor, whichever that is.).

1 0

Re: [PATCH 2/5] wined3d: Wrap resource->heapMemory into a structure
by Henri Verbeet 19 Aug '13

19 Aug '13

On 19 August 2013 10:52, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > +struct wined3d_heap_memory *wined3d_alloc_resource_mem(UINT size) > +{ > + struct wined3d_heap_memory *ret; > + > + ret = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*ret)); > + if (!ret) > + return NULL; > + > + ret->alloc_ptr = HeapAlloc(GetProcessHeap(), 0, size + RESOURCE_ALIGNMENT); > + if (!ret->alloc_ptr) > + { > + HeapFree(GetProcessHeap(), 0, ret); > + return NULL; > + } > + ret->ptr = (BYTE *)(((ULONG_PTR)ret->alloc_ptr > + + (RESOURCE_ALIGNMENT - 1)) & ~(RESOURCE_ALIGNMENT - 1)); > + return ret; > +} > + I don't think this is a good idea. How about something like the following? The corresponding free function would be trivial. void *wined3d_memalign(SIZE_T alignment, SIZE_T size) { SIZE_T align; void *mem; void **p; align = max(sizeof(*p), alignment - 1); if (!(mem = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, size + align))) return NULL; p = (void **)(((ULONG_PTR)mem + align) & ~(alignment - 1)) - 1; *p = mem; return ++p; } (Untested, there's probably some room for improvement.)

3 6

Re: [PATCH 10/11] widl: Set pointer_size according to host's pointer-size.
by Alexandre Julliard 19 Aug '13

19 Aug '13

Kai Tietz <ktietz70(a)googlemail.com> writes: > diff --git a/tools/widl/widl.c b/tools/widl/widl.c > index 5df99fc..6d8ec87 100644 > --- a/tools/widl/widl.c > +++ b/tools/widl/widl.c > @@ -139,7 +139,7 @@ int line_number = 1; > > static FILE *idfile; > > -unsigned int pointer_size = 0; > +unsigned int pointer_size = sizeof(void*); > syskind_t typelib_kind = sizeof(void*) == 8 ? SYS_WIN64 : SYS_WIN32; This is wrong, the host pointer size shouldn't have any influence on the generated code. -- Alexandre Julliard julliard(a)winehq.org

2 3

re: open bugs / multi thread / recommendation on new system
by Dan Kegel 19 Aug '13

19 Aug '13

Damian wrote: > 1) ATI or nVidia for new GPU? Does SLI/Crossfire work with wine? Don't know. I just use nvidia cards I find in the trash :-) But then I don't actually play games, so I don't care about 3d performance. > 2) FX 8320 or intel same price class CPU? As the AMD has far inferior > single thread performance but far superior multi thread performance - > does wine run a win32 single thread app as multi thread? does wine > make use of multithread in general? For what it's worth, when I was paying attention to such things, the command for cpu in `seq 0 3` do; sudo cpufreq-set -g performance -c $cpu; done is needed to get reasonable performance on my A8, see http://comments.gmane.org/gmane.comp.emulators.wine.devel/86659 Perhaps this also applies to more capable amd chips and current linux. wine just maps threads one to one, there is no magic. If the game uses multiple threads, then wine uses multiple threads. Rather than ask abstract questions here, you should probably go ask for framerates on specific games on the wine forum.

1 0

Re: msxml3: Defer the conversion of XPath expressions to UTF-8. (try 4)
by Alexandre Julliard 19 Aug '13

19 Aug '13

John Chadwick <johnwchadwick(a)gmail.com> writes: > This patch was posted a bit ago without any objections, but seemingly > slipped through the cracks. Very slight revisions were made since try > 3 to improve the conciseness of code. > > From 8157b8a3d347c47f7f21e74d621632b4a9f50fd6 Mon Sep 17 00:00:00 2001 > From: John Chadwick <johnwchadwick(a)gmail.com> > Date: Sun, 21 Jul 2013 14:41:37 -0400 > Subject: msxml3: Defer the conversion of XPath expressions to UTF-8. > > This patch changes the create_selection interface to accept a wide string > instead of an xmlChar string. This is a transitional change, which will be > needed as libxml2's XPath parser is phased out. > > The implementation of tagName_to_XPath also had to be changed as it was using > libxml2's UTF-8 string API. libxml2 contained a strcat that performed > reallocation, so it was replaced with a new function that does the same with > heap_realloc and wide strings (heap_strcatW in msxml_private.h.) That's a very inefficient way to build a string. Adding this as a helper is even worse since it will encourage other places to use it too. Please try to allocate a large enough buffer from the start, or if not possible grow it quadratically. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 5/5] wined3d: Store the mipmap level in the volume
by Henri Verbeet 19 Aug '13

19 Aug '13

On 19 August 2013 10:52, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > +void wined3d_volume_set_texture_level(struct wined3d_volume *volume, GLint level) > +{ > + volume->texture_level = level; > +} > I think that's a step in the wrong direction. I think this should be set on volume creation. We do have a construction like this for surfaces, but the reasons for that don't apply to volumes, and I'd like to get rid of it for surfaces as well if at all possible.

2 2

Re: [PATCH 3/5] wined3d: Properly allocate heap memory for volumes
by Henri Verbeet 19 Aug '13

19 Aug '13

On 19 August 2013 10:52, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > @@ -184,7 +184,16 @@ HRESULT CDECL wined3d_volume_map(struct wined3d_volume *volume, > volume, map_desc, box, flags); > > if (!volume->resource.allocatedMemory) > - volume->resource.allocatedMemory = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, volume->resource.size); Can this ever happen?

2 1

Re: [PATCH 4/5] wined3d: Move volume booleans into a flags field
by Henri Verbeet 19 Aug '13

19 Aug '13

On 19 August 2013 10:52, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > + volume->flags |= WINED3D_VFLAG_LOCKABLE; /* FIXME */ You'll probably want to fix that :-) I think you can just leave the flag out until you rename it to WINED3D_VFLAG_MAPPABLE in one of the later patches.

1 0

Re: [PATCH 1/5] wined3d: Keep GL_UNPACK_CLIENT_STORAGE_APPLE off by default
by Stefan Dösinger 19 Aug '13

19 Aug '13

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Henri, Matteo, When reviewing those patches, please have a look at the attached archive. It contains the rest of the volume cleanup patches and some tests. Cheers, Stefan -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJSEd58AAoJEN0/YqbEcdMwGygP/3UsMheQPv6ZmMMGdX+QXltj 0LfSKw7wJiNFFc2+rR/NAP8s1+iGtXKvABLvTINv5ZXwngPxUc10g/5MveV1MRzQ lgU2ebRjFnFFCv0ytcSeb5dutH5EMnIplxYQEDvPR+ojkjwS5bvRENtdh4/BZPGc gh0KoKk4YwsBl3GyWwM6hM8NlEFrwvtWeEU3UdXzZiIYg3WfT5CaXGR3A4o4/G+i WBYar5nGocKHs+5Wgfe4BMC4XNuFD/RFT9R7gB7nrIYQgPJ6j+J46Zkt4ux0ivtH eBHQ5QQNB4SdVoCwPf9H56cBYhtmC6iEFYJg4wEGukZPIhaZ0Je2KHUIXuXumVfB ciesPUN+7DhuZCLSFm5eJUO7aTF+WHOD+uyQKZkLN1oK3JkXy2wflE2d8O2BQKTo qCZPtcA8vmE0DIvk9Q6j8Xt2nAmTnnqIh26ZCsCsj9vZvPn+pNlXT1lduwoF3x2H oKZufHdCdgRKLmRLpVnmPdwKTqwmlSZa9u1RVhDAypLVv20VqykOLAh6mDJC9zl7 k/2ozgsfYsl0kYMTVw5o6dx3f3pJGfgGjsw4vuJHN08aeZmVl1EFY8nMlqzrT8Wa cRjSHKxqk+W3BxTsQz9Zi1u0J3v8nE4KM79IpLgfjm8M9Rk6eTaLkwCe5XpFuiSw MJbpYdZZGZfZuP71LMmm =gRPu -----END PGP SIGNATURE-----

1 0

Re: [website] Added templates/zh-cn/cvs.template translation
by Frédéric Delanoy 19 Aug '13

19 Aug '13

On Sat, Aug 17, 2013 at 5:57 AM, xulixin <xulixin1990(a)gmail.com> wrote: > Should I change my name's spelling to Lixin Xu, or LiXin Xu, or other > format? Not sure, most people (at least non-Chinese) use firstname lastname convention, but I guess you should use the same convention as other Chinese-speaking committers. Here's a list I extracted from AUTHORS file that *look* Chinese to me (don't know if they're really Chinese, or Korean, or from another Asian country): Chae Jong Bin Cheer Xiao Chia-I Wu Hwang YunSong (황윤성) Jau-Horng Chen Jay Yang Qian Hong Qingchuan Wang Qingdoa Daoo Shi Quan He Wei-Lun Chao Weisheng Li Xiang Li XueFeng Chang Yong Chi Yuxi Zhang Zhangrong Huang Zhan Jianyu Hope this helps, Frédéric Delanoy

2 2

PLEASE add bug links!
by Ken Sharp 18 Aug '13

18 Aug '13

There's little point in my continually asking users to add bug links to the AppDB if maintainers and/or administrators don't bother themselves. It takes longer to create a bug report than it does to link it to the database. It really isn't that hard. I've added hundreds over the past few weeks and this is not the first time I have said it, nor am I the first one to do so. It's even on the Wiki. http://wiki.winehq.org/Bugs

8 20

Re: mshtml: Use winehq snapshot instead of the main page in tests.
by Marvin 18 Aug '13

18 Aug '13

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=26726 Your paranoid android. === WXPPROSP2 (32 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(00289218) != exmon(002683C0) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(00289218) != exmon(002683C0) htmldoc.c:5715: Test failed: mon(00289218) != exmon(002683C0) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(002CFA10) != exmon(002854D0) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(002CFA10) != exmon(002854D0) htmldoc.c:5715: Test failed: mon(002CFA10) != exmon(002854D0) === WXPX64 (32 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(0067F130) != exmon(00653D78) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(0067F130) != exmon(00653D78) htmldoc.c:5715: Test failed: mon(0067F130) != exmon(00653D78) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(006A5D60) != exmon(0069E610) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(006A5D60) != exmon(0069E610) htmldoc.c:5715: Test failed: mon(006A5D60) != exmon(0069E610) === WVISTAX64 (32 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(006F6E18) != exmon(006F6018) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(006F6E18) != exmon(006F6018) htmldoc.c:5715: Test failed: mon(006F6E18) != exmon(006F6018) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(006F6518) != exmon(006F5F18) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(006F6518) != exmon(006F5F18) htmldoc.c:5715: Test failed: mon(006F6518) != exmon(006F5F18) === W7PROX64 (32 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(00708F30) != exmon(00708670) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(00708F30) != exmon(00708670) htmldoc.c:5715: Test failed: mon(00708F30) != exmon(00708670) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(00708C30) != exmon(007081B0) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(00708C30) != exmon(007081B0) htmldoc.c:5715: Test failed: mon(00708C30) != exmon(007081B0) === WXPX64 (64 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(0000000000297C90) != exmon(000000000027FD00) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(0000000000297C90) != exmon(000000000027FD00) htmldoc.c:5715: Test failed: mon(0000000000297C90) != exmon(000000000027FD00) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(00000000002D1AC0) != exmon(00000000002FC160) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(00000000002D1AC0) != exmon(00000000002FC160) htmldoc.c:5715: Test failed: mon(00000000002D1AC0) != exmon(00000000002FC160) === WVISTAX64 (64 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(000000000089DD50) != exmon(0000000000823490) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(000000000089DD50) != exmon(0000000000823490) htmldoc.c:5715: Test failed: mon(000000000089DD50) != exmon(0000000000823490) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(000000000089E1D0) != exmon(0000000000823A90) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(000000000089E1D0) != exmon(0000000000823A90) htmldoc.c:5715: Test failed: mon(000000000089E1D0) != exmon(0000000000823A90) === W7PROX64 (64 bit htmldoc) === htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(00000000007621D0) != exmon(0000000000761750) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:2507: Test failed: unexpected call UpdateUI htmldoc.c:2809: Test failed: unexpected call Exec_UPDATECOMMANDS htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(00000000007621D0) != exmon(0000000000761750) htmldoc.c:5715: Test failed: mon(00000000007621D0) != exmon(0000000000761750) htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3139: Test failed: unexpected call Invoke_AMBIENT_SILENT htmldoc.c:3124: Test failed: unexpected state L"uninitialized", expected 1 htmldoc.c:3124: Test failed: VT_I4(out)=0, expected 1 htmldoc.c:3136: Test failed: unexpected call Invoke_AMBIENT_OFFLINEIFNOTCONNECTED htmldoc.c:3204: Test failed: unexpected call Invoke_OnReadyStateChange_Loading htmldoc.c:2861: Test failed: mon(00000000046C9830) != exmon(0000000000761C50) htmldoc.c:2460: Test failed: unexpected call GetHostInfo htmldoc.c:3412: Test failed: dwFlags = 8 htmldoc.c:7326: Test failed: mon(00000000046C9830) != exmon(0000000000761C50) htmldoc.c:5715: Test failed: mon(00000000046C9830) != exmon(0000000000761C50)

2 1

Re: msxml3: Defer the conversion of XPath expressions to UTF-8. (try 4)
by John Chadwick 17 Aug '13

17 Aug '13

Nikolay, does this patch look good to you? I was advised on #winehackers to get some acknowledgement that the patch is OK. Hopefully doing so can get things moving along.

2 1

open bugs / multi thread / recommendation on new system
by Damian Ivanov 17 Aug '13

17 Aug '13

Hi guys, As you have a track of open bugs: 1) ATI or nVidia for new GPU? Does SLI/Crossfire work with wine? 2) FX 8320 or intel same price class CPU? As the AMD has far inferior single thread performance but far superior multi thread performance - does wine run a win32 single thread app as multi thread? does wine make use of multithread in general? Thanks in advance :) Damian

1 0

Re: [PATCH 4/6] oleaut32: TKIND_ALIAS improvements
by Piotr Caban 17 Aug '13

17 Aug '13

On 08/16/13 21:40, Andrew Eikum wrote: > @@ -8502,7 +8502,7 @@ static HRESULT WINAPI ICreateTypeLib2_fnCreateTypeInfo(ICreateTypeLib2 *iface, > info->cbSizeInstance = 2; > break; > case TKIND_ALIAS: > - info->cbSizeInstance = -0x75; > + info->cbSizeInstance = 4; > break; > default: > FIXME("unrecognized typekind %d\n", info->typekind); > cbSizeInstance and cbAligment value depends on alias type. As long as alias type is not set cbSizeInstance stores some strange values (-0x75 is matching what one of native versions of dll stores there). Both of these values should be set in SetTypeDescAlias function.

1 0

Re: imm32/tests: Fix a trailing '\n' in an ok() call.
by Qian Hong 17 Aug '13

17 Aug '13

> - ok(ic != NULL, "ImmLockIMC failed\n!"); > + ok(ic != NULL, "ImmLockIMC failed!\n"); Oh, good catch... Thanks! -- Regards, Qian Hong - http://www.winehq.org

1 0

Re: Re: [PATCH 1/6] msvcrt: Use LOCK_ENV locking in _chdrive and do validatation of input argument.
by Marvin 16 Aug '13

16 Aug '13

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=26722 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [website] Added templates/zh-cn/cvs.template translation
by Frédéric Delanoy 16 Aug '13

16 Aug '13

On Fri, Aug 16, 2013 at 8:13 AM, xulixin <xulixin1990(a)gmail.com> wrote: > From 860262aed3d58f42012e6dd5264eb29bd29deffb Mon Sep 17 00:00:00 2001 > From: xulixin <xulixin1990(a)gmail.com> > Date: Fri, 16 Aug 2013 13:56:36 +0800 > Subject: add templates/zh-cn/cvs.template > > --- > templates/zh-cn/cvs.template | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/templates/zh-cn/cvs.template b/templates/zh-cn/cvs.template > index ba10a25..a9bc5d1 100644 > --- a/templates/zh-cn/cvs.template > +++ b/templates/zh-cn/cvs.template > @@ -13,9 +13,9 @@ > > <h1 id="sourcetree">通过 Web 方式浏览源码树</h1> > > -<p>The source code of the entire Wine tree is (<a > href="http://lxr.linux.no/">LXR</a>-based) > - cross-referenced and available for browsing, including a search engine. > - This cross-references source is based on the snapshot-releases and is > made available at:</p> > +<p>Wine tree的源码 (<a href="http://lxr.linux.no/">LXR</a>-based) > + 是相互参照且可供浏览的，包括一个搜索引擎。 > + 这交叉参考的源码是基于snapshot-releases版本，可从下面网站获得：</p> > <ul> > <li><a href="http://source.winehq.org/">source.winehq.org</a></li> > </ul> Hi, the real name is required when sending patches. Also, lxr.linux.no isn't used anymore, so the original file would need to be updated beforehand. -- Frédéric Delanoy

3 5

Re: [PATCH 1/6] msvcrt: Use LOCK_ENV locking in _chdrive and do validatation of input argument.
by Alexandre Julliard 16 Aug '13

16 Aug '13

Kai Tietz <ktietz70(a)googlemail.com> writes: > diff --git a/dlls/msvcrt/dir.c b/dlls/msvcrt/dir.c > index 44802b3..3b9e3c7 100644 > --- a/dlls/msvcrt/dir.c > +++ b/dlls/msvcrt/dir.c > @@ -365,14 +365,25 @@ int CDECL MSVCRT__chdrive(int newdrive) > { > WCHAR buffer[] = {'A', ':', 0}; > > + if (newdrive < 1 || newdrive > 31) > + { > + *MSVCRT___doserrno () = ERROR_INVALID_DRIVE; > + *MSVCRT__errno () = MSVCRT_EACCES; > + return -1; > + } > buffer[0] += newdrive - 1; > - if (!SetCurrentDirectoryW( buffer )) > + > + _lock (_ENV_LOCK); > + > + if (!SetCurrentDirectoryW (buffer)) > { > - msvcrt_set_errno(GetLastError()); > - if (newdrive <= 0) > - *MSVCRT__errno() = MSVCRT_EACCES; > + msvcrt_set_errno (GetLastError ()); > + _unlock (_ENV_LOCK); > return -1; > } > + > + _unlock (_ENV_LOCK); > + > return 0; > } Please add test cases at least for the error values. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: ntdll: empty APC queue before starting an application
by Alexandre Julliard 16 Aug '13

16 Aug '13

Andrey Turkin <andrey.turkin(a)gmail.com> writes: > --- > dlls/ntdll/loader.c | 3 +++ > dlls/ntdll/ntdll.spec | 2 +- > dlls/ntdll/sync.c | 16 ++++++++++++++++ > 3 files changed, 20 insertions(+), 1 deletion(-) > > diff --git a/dlls/ntdll/loader.c b/dlls/ntdll/loader.c > index b3abad0..1eaa123 100644 > --- a/dlls/ntdll/loader.c > +++ b/dlls/ntdll/loader.c > @@ -2747,6 +2747,9 @@ void WINAPI LdrInitializeThunk( void *kernel_start, ULONG_PTR unknown2, > > virtual_release_address_space(); > virtual_clear_thread_stack(); > + > + NtTestAlert(); > + > wine_switch_to_stack( start_process, kernel_start, NtCurrentTeb()->Tib.StackBase ); This would have to run on the thread stack. Also please try to add some tests. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [website] Update git URL
by Michael Stefaniuc 16 Aug '13

16 Aug '13

On 08/16/2013 03:32 PM, Frédéric Delanoy wrote: > --- > templates/en/cvs.template | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/templates/en/cvs.template b/templates/en/cvs.template > index 202cf60..300b9e4 100644 > --- a/templates/en/cvs.template > +++ b/templates/en/cvs.template > @@ -3,9 +3,9 @@ > > <h1 class="title">The Wine CVS Tree</h1> Schouldn't the title change too? And nowadays people know what git is and that's kinda the default SCM that people expect. > > -<p><strong>Note: We have switched development from CVS to <a href="http://git.or.cz">Git</a>. > +<p><strong>Note: We have switched development from CVS to <a href="http://git-scm.com/">Git</a>. > Wine can no longer be downloaded via CVS. > - Instructions for using the <a href="{$root}/site/git">Wine Git tree</a> > + Instructions for using the Wine Git tree > can be found on the <a href="http://wiki.winehq.org/GitWine">GitWine</a> > page on the <a href="http://wiki.winehq.org">Wine Wiki</a>.</strong></p> bye michael

2 1

Re: [PATCH 01/11] widl: New option --rt for enabling rt's specific language extensions.
by Alexandre Julliard 16 Aug '13

16 Aug '13

Kai Tietz <ktietz70(a)googlemail.com> writes: > Due line wrap, I sent patch as attachment. > > Kai > > From 3fcedb770626535b56a789467951a4c9be5f08dc Mon Sep 17 00:00:00 2001 > From: Kai Tietz <ktietz70(a)googlemail.com> > Date: Thu, 8 Aug 2013 12:29:33 +0200 > Subject: [PATCH 01/11] widl: New option --rt for enabling rt's specific > language extensions. > > --- > tools/widl/widl.c | 6 ++++++ > tools/widl/widl.h | 1 + > 2 files changed, 7 insertions(+), 0 deletions(-) Please update the usage message and the manpage accordingly. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [1/3] apphelp: Add definitions
by Dmitry Timoshkov 16 Aug '13

16 Aug '13

Mislav Blazevic <krofnica996(a)gmail.com> wrote: > --- > dlls/apphelp/apphelp.c | 10 +- > dlls/apphelp/apphelp.h | 272 +++++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 273 insertions(+), 9 deletions(-) > create mode 100644 dlls/apphelp/apphelp.h Where all of this comes from? Is there an application that calls apphelp APIs and doesn't work because of failures? How are you testing your implementation? -- Dmitry.

3 6

Re: comsvcs: Add dll comsvcs (resend)
by André Hentschel 16 Aug '13

16 Aug '13

Am 16.08.2013 12:39, schrieb Alistair Leslie-Hughes: > Hi, > > > Changelog: > comsvcs: Add dll comsvcs > > > Best Regards > Alistair Leslie-Hughes you missed the configure.ac part?

1 0

Re: [3/3] ntdll/tests: Add a bunch of overlapped IO tests for files.
by Marvin 16 Aug '13

16 Aug '13

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=26708 Your paranoid android. === WXPX64 (64 bit file) === file.c:1791: Test failed: expected STATUS_SUCCESS, got 0 file.c:1817: Test failed: expected STATUS_SUCCESS, got 0 file.c:1844: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1855: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1874: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1894: Test failed: expected STATUS_SUCCESS, got 0 file.c:1927: Test failed: expected STATUS_SUCCESS, got 0 file.c:1008: Test failed: Invalid ioSb.Information: 3 === WVISTAX64 (64 bit file) === file.c:1791: Test failed: expected STATUS_SUCCESS, got 0 file.c:1817: Test failed: expected STATUS_SUCCESS, got 0 file.c:1844: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1855: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1874: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1884: Test failed: expected STATUS_SUCCESS, got 0 file.c:1894: Test failed: expected STATUS_SUCCESS, got 0 file.c:1903: Test failed: expected STATUS_SUCCESS, got 0 file.c:1927: Test failed: expected STATUS_SUCCESS, got 0 file.c:2030: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2037: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2088: Test failed: expected STATUS_SUCCESS, got 0 file.c:2095: Test failed: expected STATUS_SUCCESS, got 0 file.c:2104: Test failed: expected STATUS_SUCCESS, got 0 file.c:2110: Test failed: expected STATUS_SUCCESS, got 0 file.c:1008: Test failed: Invalid ioSb.Information: 3 === W7PROX64 (64 bit file) === file.c:1791: Test failed: expected STATUS_SUCCESS, got 0 file.c:1817: Test failed: expected STATUS_SUCCESS, got 0 file.c:1844: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1855: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1874: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2030: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2037: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2088: Test failed: expected STATUS_SUCCESS, got 0 file.c:2095: Test failed: expected STATUS_SUCCESS, got 0 file.c:2104: Test failed: expected STATUS_SUCCESS, got 0 file.c:2110: Test failed: expected STATUS_SUCCESS, got 0 file.c:1008: Test failed: Invalid ioSb.Information: 3 === TEST64_W7SP1 (64 bit file) === file.c:1791: Test failed: expected STATUS_SUCCESS, got 0 file.c:1817: Test failed: expected STATUS_SUCCESS, got 0 file.c:1844: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1855: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1874: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:1884: Test failed: expected STATUS_SUCCESS, got 0 file.c:1894: Test failed: expected STATUS_SUCCESS, got 0 file.c:1903: Test failed: expected STATUS_SUCCESS, got 0 file.c:1927: Test failed: expected STATUS_SUCCESS, got 0 file.c:2030: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2037: Test failed: expected STATUS_END_OF_FILE, got 0xc0000011 file.c:2088: Test failed: expected STATUS_SUCCESS, got 0 file.c:2095: Test failed: expected STATUS_SUCCESS, got 0 file.c:2104: Test failed: expected STATUS_SUCCESS, got 0 file.c:2110: Test failed: expected STATUS_SUCCESS, got 0 file.c:1008: Test failed: Invalid ioSb.Information: 3

2 1

Re: [2/3] ntdll/tests: Use official path for temporary files instead of current directory.
by Marvin 16 Aug '13

16 Aug '13

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=26707 Your paranoid android. === WXPX64 (64 bit file) === file.c:1008: Test failed: Invalid ioSb.Information: 3 === WVISTAX64 (64 bit file) === file.c:1008: Test failed: Invalid ioSb.Information: 3 === W7PROX64 (64 bit file) === file.c:1008: Test failed: Invalid ioSb.Information: 3 === TEST64_W7SP1 (64 bit file) === file.c:1008: Test failed: Invalid ioSb.Information: 3

1 0

Re: [3/3] ntdll: Implement compatible typelib section
by Marvin 16 Aug '13

16 Aug '13

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=26698 Your paranoid android. === W2KPROSP4 (32 bit actctx) === Timeout === WXPPROSP2 (32 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === WXPX64 (32 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === W2K3R2SESP2 (32 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === WVISTAX64 (32 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === W7PROX64 (32 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === TEST64_W7SP1 (32 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === WXPX64 (64 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === WVISTAX64 (64 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === W7PROX64 (64 bit actctx) === actctx.c:1375: Test failed: got 409, 1043 === TEST64_W7SP1 (64 bit actctx) === actctx.c:1375: Test failed: got 409, 1043

1 0

Re: [2/3] ntdll: Added support for all typelib section attributes
by Marvin 16 Aug '13

16 Aug '13

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=26697 Your paranoid android. === W2KPROSP4 (32 bit actctx) === Timeout === WXPPROSP2 (32 bit actctx) === actctx.c:1357: Test failed: got 409 === WXPX64 (32 bit actctx) === actctx.c:1357: Test failed: got 409 === W2K3R2SESP2 (32 bit actctx) === actctx.c:1357: Test failed: got 409 === WVISTAX64 (32 bit actctx) === actctx.c:1357: Test failed: got 409 === W2K8SE (32 bit actctx) === actctx.c:1357: Test failed: got 409 === W7PROX64 (32 bit actctx) === actctx.c:1357: Test failed: got 409 === TEST64_W7SP1 (32 bit actctx) === actctx.c:1357: Test failed: got 409 === WXPX64 (64 bit actctx) === actctx.c:1357: Test failed: got 409 === WVISTAX64 (64 bit actctx) === actctx.c:1357: Test failed: got 409 === W7PROX64 (64 bit actctx) === actctx.c:1357: Test failed: got 409 === TEST64_W7SP1 (64 bit actctx) === actctx.c:1357: Test failed: got 409

1 0

Re: [1/3] kernel32: Some tests for typelib sections
by Marvin 16 Aug '13

16 Aug '13

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=26696 Your paranoid android. === W2KPROSP4 (32 bit actctx) === Timeout

1 0

Re: mscoree: Redirect .NET 1.0 to .NET 2.0
by Alexandre Julliard 15 Aug '13

15 Aug '13

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > Hi, > > > Changelog: > mscoree: Redirect .NET 1.0 to .NET 2.0 It doesn't work: ../../../tools/runtest -q -P wine -M mscoree.dll -T ../../.. -p mscoree_test.exe.so mscoree.c && touch mscoree.ok mscoree.c:179: Test failed: version is L"v1.1.4322" , expected L"v2.0.50727" mscoree.c:187: Test failed: GetRequestedRuntimeInfo returned 80131700 mscoree.c:188: Test failed: version is L"" , expected L"v2.0.50727" make: *** [mscoree.ok] Error 3 -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: configure.ac: allow disabling winemac.drv (try 2)
by Alexandre Julliard 15 Aug '13

15 Aug '13

Austin English <austinenglish(a)gmail.com> writes: > try 2: don't unconditionally disable ApplicationServices.h > > Tested on 10.7 > diff --git a/configure.ac b/configure.ac > index 0e3df61..5e1fe10 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -70,6 +70,7 @@ AC_ARG_WITH(pthread, AS_HELP_STRING([--without-pthread],[do not use the pthrea > AC_ARG_WITH(sane, AS_HELP_STRING([--without-sane],[do not use SANE (scanner support)])) > AC_ARG_WITH(tiff, AS_HELP_STRING([--without-tiff],[do not use TIFF])) > AC_ARG_WITH(v4l, AS_HELP_STRING([--without-v4l],[do not use v4l1 (v4l support)])) > +AC_ARG_WITH(winemac, AS_HELP_STRING([--without-winemac],[do not build native Mac (Cocoa) driver])) It should be --disable-winemac.drv, and it should work fine already, same as with any other dll. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: mscoree: Attempt to load the assembly from the application directory
by Vincent Povirk 15 Aug '13

15 Aug '13

This seems suspicious to me. You appear to be taking the namespace of the class and assuming it's an assembly name. Is there really no other information in the registry that could be used to identify the assembly?

3 4

Re: msvcrt: add support for _chsize_s
by Ken Sharp 14 Aug '13

14 Aug '13

On 14/08/13 20:08, morphiend wrote: > -@ stub _chsize_s > +@ cdecl _chsize_s(long int64) msvcrt._chsize > +#@ stub _chsize_s Oops..

1 0

Re: mscoree: Redirect .NET 1.0 to .NET 2.0
by Vincent Povirk 14 Aug '13

14 Aug '13

Looks good, thanks.

1 0

Possibility of adding a new patch status
by Hugh McMaster 14 Aug '13

14 Aug '13

Hi everyone, Wine patches currently have a status described in http://source.winehq.org/patches, yet for patches with the status of 'New', the status becomes confusing. The legend describes 'New' status as "Patch not even looked at yet, there's still hope...". This is ideal for new patches submitted within that 24-hour commit cycle. But I'm finding it difficult to follow patches that remain with a status of 'New' for longer than the 24-hour patch cycle. Obviously, on the weekend, patches are held over till Monday, so a longer lead-time is expected. During weekdays, however, it is unclear what is happening with some patches. This, ultimately, raises the question of timeliness. Has the patch been looked at? If it has, the status often describes what action was taken - committed, rejected, superseded, etc. This is fine, but some patches remain with a status of 'New'. Experience has told me that patches remaining with a status of 'New' are incorrect in some way. But this is not always the case. If the patch is incorrect, but close to being accepted, the patch's status should reflect this, by changing to something like "Revision needed". Of course, the "Rejected" status is also appropriate, depending on the severity of coding error. Also, there are likely to be many times when the maintainer has not had time to evaluate some patches. This means the patch is not new (i.e. recently submitted), but is awaiting review. Once again, I believe the patch status should reflect this situation. The status could be "Not yet reviewed". In summary, the 'New' status should be reserved for patches that are actually new. Just some thoughts.

3 7

Re: gdiplus: Only clip strings if rectangle width and height are positive.
by Dmitry Timoshkov 14 Aug '13

14 Aug '13

Vincent Povirk <madewokherd(a)gmail.com> wrote: > if (!(format_flags & StringFormatFlagsNoClip) && > - scaled_rect.Width != 1 << 23 && scaled_rect.Height != 1 << 23) > + scaled_rect.Width != 1 << 23 && scaled_rect.Height != 1 << 23 && > + rect->Width > 0.0 && rect->Height > 0.0) > { > /* FIXME: If only the width or only the height is 0, we should probably still clip */ > rgn = CreatePolygonRgn(corners, 4, ALTERNATE); Probably FIXME should be removed then. -- Dmitry.

2 1

wine-bugs change?
by Thomas Spear 13 Aug '13

13 Aug '13

Hi all, I've been subscribed to the lists for several years now, archiving all of the conversations in case a situation ever arises where a backup copy is needed. I've had a filter setup in gmail for wine-bugs that has worked perfectly fine for a long time now, but over the last couple of days, I've noticed that all emails sent to or from wine-bugs are now being directed to my inbox in addition to the folder I have the filter setup for. I'm planning on contacting Google as well, but thought I should cover all of the bases by asking here if there have been any changes made to that list specifically that might have caused the filter to not work. I've tried editing the filter so that I can see what messages it searches for, and it appears to be working properly, in that it appears to locate the emails I have the filter setup for, including those that are currently in both my inbox and the folder I specified to have them moved to, so it could be that there is an issue with the "Skip Inbox" setting of the filter, specifically. But again, I thought I would ask (because Google will and I'll need to have an answer for them) what changes, if any, have been made recently on the mailman side of things. Thank you, Thomas

4 6

Wine Gecko repo
by Jacek Caban 13 Aug '13

13 Aug '13

Hi, A few months ago, SourceForge did some deep changes in their Git (and not only) repo hosting. While doing it, the location of repos has changed (which is extremely stupid, if you ask me). The problem is that while doing the update, they also switched to their own Web interface for git. And this shiny new interface doesn't work at all for Wine Gecko repo (wine-gecko.git probably too big and complicated). See [1], it claims to be analyzing the repo, but it has been like that since the switch was done. I contacted them [2] and they are aware of the problem, but it's still not fixed. I already got some questions about outdated sources. Also, our Wiki points to this Git in a few places. I never updated those links because I was waiting for the new repo to work. I think it's time to move the repo. The question is, where should it go? - Github seems to be the choice for most project currently and it's already used by Wine Mono. - Reconsider source.winehq.org. This was not chosen due to account management overhead, but that's the only way to avoid problems similar to recent Sourceforge one (Github nor anyone else is going to give us any guarantees) - Other ideas? I'd appreciate opinions. Thanks, Jacek [1] https://sourceforge.net/p/wine/wine-gecko/ci/master/tree/ [2] https://sourceforge.net/p/forge/site-support/4299/

3 2

Process for reporting security bugs?
by achurch+wine-devel＠achurch.org 13 Aug '13

13 Aug '13

[Please cc: me on any replies since I'm not subscribed to the list.] Hi, Is there a specific process for reporting security-related bugs in Wine? I've looked through winehq.org but haven't found any mention of such; I just wanted to make sure I haven't overlooked anything before posting the bug in a publicly visible manner. Thanks, --Andrew Church achurch+wine-devel(a)achurch.org http://achurch.org/

3 5

Re: [1/2] cmd: Add handler support for Ctrl-C and Ctrl-Break events
by Hugh McMaster 13 Aug '13

13 Aug '13

On Monday, 12 August 2013, Ruslan Kabatsayev wrote: >I've tried applying your both patches, and it appears that Ctrl+C at >cmd prompt still closes cmd, although pressing it while "dir /s /w" is running works as expected. Hi Ruslan, I've just tested the Ctrl-C patches on the most recent version of Wine available from the Git repository. After patching, compiling and installing, I found no problems with either of the patches as you described. Ctrl-C did nothing when cmd.exe was open and no command was running. To be clear, I was running cmd.exe via wineconsole in Linux Mint 14: 1. wineconsole cmd.exe 2. Press Ctrl-C --> cmd.exe remains open. But if you run cmd.exe via wine itself, the terminal will terminate cmd.exe when Ctrl-C is pressed. 1. wine cmd.exe 2. Press Ctrl-C --> returned to terminal prompt. I hope this helps. If the patch still doesn't seem to work correctly, please let me know what operating system you use, and I'll run further tests. Hugh

2 2

Re: Compiler warnings on Debian Sid kfreebsd-i386
by Ken Sharp 12 Aug '13

12 Aug '13

On 12/08/13 20:15, Mislav Blazevic wrote: > It seems that xmlBufPtr was renamed to xmlBufferPtr in new libxml. It does seem that way. :-) Or maybe :-(

1 0

Re: Compiler warnings on Debian Sid kfreebsd-i386
by Mislav Blazevic 12 Aug '13

12 Aug '13

It seems that xmlBufPtr was renamed to xmlBufferPtr in new libxml. On Mon, Aug 12, 2013 at 9:15 PM, Mislav Blazevic <krofnica996(a)gmail.com>wrote: > It seems that xmlBufPtr was renamed to xmlBufferPtr in new libxml. > > > > On Mon, Aug 12, 2013 at 9:11 PM, Ken Sharp <kennybobs(a)o2.co.uk> wrote: > >> Following my previous e-mail (http://www.winehq.org/** >> pipermail/wine-devel/2013-**August/100754.html<http://www.winehq.org/pipermail/wine-devel/2013-August/100754.html>) >> I have since moved from Wheezy to Sid to work around a Debian bug. >> >> libxml2 has been updated (2.8.0+dfsg1-7+nmu1 --> 2.9.1+dfsg1-3) as a >> result and introduced some new compiler warnings: >> >> /home/ken/wine-git/dlls/**msxml3/node.c: In function >> ‘htmldtd_dumpcontent’: >> /home/ken/wine-git/dlls/**msxml3/node.c:908:9: warning: passing argument >> 1 of ‘xmlBufferWriteQuotedString’ from incompatible pointer type [enabled >> by default] >> xmlBufferWriteQuotedString(**buf->buffer, cur->ExternalID); >> ^ >> In file included from /usr/include/libxml2/libxml/**parser.h:16:0, >> from /home/ken/wine-git/dlls/**msxml3/node.c:28: >> /usr/include/libxml2/libxml/**tree.h:1123:3: note: expected >> ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ >> xmlBufferWriteQuotedString(**xmlBufferPtr buf, >> ^ >> /home/ken/wine-git/dlls/**msxml3/node.c:912:13: warning: passing >> argument 1 of ‘xmlBufferWriteQuotedString’ from incompatible pointer type >> [enabled by default] >> xmlBufferWriteQuotedString(**buf->buffer, cur->SystemID); >> ^ >> In file included from /usr/include/libxml2/libxml/**parser.h:16:0, >> from /home/ken/wine-git/dlls/**msxml3/node.c:28: >> /usr/include/libxml2/libxml/**tree.h:1123:3: note: expected >> ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ >> xmlBufferWriteQuotedString(**xmlBufferPtr buf, >> ^ >> /home/ken/wine-git/dlls/**msxml3/node.c:918:9: warning: passing argument >> 1 of ‘xmlBufferWriteQuotedString’ from incompatible pointer type [enabled >> by default] >> xmlBufferWriteQuotedString(**buf->buffer, cur->SystemID); >> ^ >> In file included from /usr/include/libxml2/libxml/**parser.h:16:0, >> from /home/ken/wine-git/dlls/**msxml3/node.c:28: >> /usr/include/libxml2/libxml/**tree.h:1123:3: note: expected >> ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ >> xmlBufferWriteQuotedString(**xmlBufferPtr buf, >> ^ >> /home/ken/wine-git/dlls/**msxml3/node.c: In function >> ‘node_transform_node’: >> /home/ken/wine-git/dlls/**msxml3/node.c:974:21: warning: passing >> argument 1 of ‘xmlBufferContent’ from incompatible pointer type [enabled by >> default] >> content = xmlBufferContent(output->**buffer); >> ^ >> In file included from /usr/include/libxml2/libxml/**parser.h:16:0, >> from /home/ken/wine-git/dlls/**msxml3/node.c:28: >> /usr/include/libxml2/libxml/**tree.h:734:3: note: expected >> ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ >> xmlBufferContent (const xmlBufferPtr buf); >> ^ >> >> Hopefully these mean something to someone. >> >> >> >

1 0

Compiler warnings on Debian Sid kfreebsd-i386
by Ken Sharp 12 Aug '13

12 Aug '13

Following my previous e-mail (http://www.winehq.org/pipermail/wine-devel/2013-August/100754.html) I have since moved from Wheezy to Sid to work around a Debian bug. libxml2 has been updated (2.8.0+dfsg1-7+nmu1 --> 2.9.1+dfsg1-3) as a result and introduced some new compiler warnings: /home/ken/wine-git/dlls/msxml3/node.c: In function ‘htmldtd_dumpcontent’: /home/ken/wine-git/dlls/msxml3/node.c:908:9: warning: passing argument 1 of ‘xmlBufferWriteQuotedString’ from incompatible pointer type [enabled by default] xmlBufferWriteQuotedString(buf->buffer, cur->ExternalID); ^ In file included from /usr/include/libxml2/libxml/parser.h:16:0, from /home/ken/wine-git/dlls/msxml3/node.c:28: /usr/include/libxml2/libxml/tree.h:1123:3: note: expected ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ xmlBufferWriteQuotedString(xmlBufferPtr buf, ^ /home/ken/wine-git/dlls/msxml3/node.c:912:13: warning: passing argument 1 of ‘xmlBufferWriteQuotedString’ from incompatible pointer type [enabled by default] xmlBufferWriteQuotedString(buf->buffer, cur->SystemID); ^ In file included from /usr/include/libxml2/libxml/parser.h:16:0, from /home/ken/wine-git/dlls/msxml3/node.c:28: /usr/include/libxml2/libxml/tree.h:1123:3: note: expected ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ xmlBufferWriteQuotedString(xmlBufferPtr buf, ^ /home/ken/wine-git/dlls/msxml3/node.c:918:9: warning: passing argument 1 of ‘xmlBufferWriteQuotedString’ from incompatible pointer type [enabled by default] xmlBufferWriteQuotedString(buf->buffer, cur->SystemID); ^ In file included from /usr/include/libxml2/libxml/parser.h:16:0, from /home/ken/wine-git/dlls/msxml3/node.c:28: /usr/include/libxml2/libxml/tree.h:1123:3: note: expected ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ xmlBufferWriteQuotedString(xmlBufferPtr buf, ^ /home/ken/wine-git/dlls/msxml3/node.c: In function ‘node_transform_node’: /home/ken/wine-git/dlls/msxml3/node.c:974:21: warning: passing argument 1 of ‘xmlBufferContent’ from incompatible pointer type [enabled by default] content = xmlBufferContent(output->buffer); ^ In file included from /usr/include/libxml2/libxml/parser.h:16:0, from /home/ken/wine-git/dlls/msxml3/node.c:28: /usr/include/libxml2/libxml/tree.h:734:3: note: expected ‘xmlBufferPtr’ but argument is of type ‘xmlBufPtr’ xmlBufferContent (const xmlBufferPtr buf); ^ Hopefully these mean something to someone.

1 0

Question about implementing application compatibility
by Mislav Blazevic 12 Aug '13

12 Aug '13

For past two weeks I've been implementing apphelp.dll. I got that one working and moved onto kernel32 part of magic. According to various papers and blogs it seems that windows logs all exes and dlls ever executed in registry[1] and uses it as cache to lookup if binary needs to be shimmed or patched (if file isn't found, hard lookup of shim database is done) It is loaded on boot and saved back to registry on shutdown. This is awfully overcomplicated (plus I do not know how to make such a "global" variable in wine) so I was wondering is it OK to implement this differently than windows does it. It would be simpler to control app compat with environmental variable. It would mean that Base*Appcompat* kernel32 APIs would remain unimplemented but as far as I know, no application uses them anyway, other than windows internally. What do you think? [1] https://github.com/mandiant/ShimCacheParser

2 1

Re: ntdll/tests: test FileDispositionInformation file class
by Marvin 12 Aug '13

12 Aug '13

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=26661 Your paranoid android. === WXPX64 (32 bit file) === file.c:1318: Test failed: Unexpected NtQueryInformationFile result (expected STATUS_INVALID_INFO_CLASS, got c0000002) === WVISTAX64 (32 bit file) === file.c:1318: Test failed: Unexpected NtQueryInformationFile result (expected STATUS_INVALID_INFO_CLASS, got c0000002) === W7PROX64 (32 bit file) === file.c:1318: Test failed: Unexpected NtQueryInformationFile result (expected STATUS_INVALID_INFO_CLASS, got c0000002) === TEST64_W7SP1 (32 bit file) === file.c:1318: Test failed: Unexpected NtQueryInformationFile result (expected STATUS_INVALID_INFO_CLASS, got c0000002)

1 0

Re: [1/2] cmd: Add handler support for Ctrl-C and Ctrl-Break events
by Ruslan Kabatsayev 11 Aug '13

11 Aug '13

Hi, On Sun, Aug 11, 2013 at 12:33 PM, Hugh McMaster <hugh.mcmaster(a)masterindexing.com> wrote: > Jason Edmeades and I have developed a patch that handles Ctrl-C and Ctrl-Break events in wineconsole's cmd.exe. > > The patch supports interactive mode, batch contexts and both the cmd /c and cmd /k modes, meaning that pressing Ctrl-C no longer closes cmd.exe. I've tried applying your both patches, and it appears that Ctrl+C at cmd prompt still closes cmd, although pressing it while "dir /s /w" is running works as expected. > > For loops, commands such as 'dir /s /w' and other such commands can be stopped. Batch scripts prompt the user whether the current batch execution should end. > > --- > programs/cmd/builtins.c | 95 ++++++++++++++++++++++++++++++++++++++++++----- > programs/cmd/cmd.rc | 2 + > programs/cmd/wcmd.h | 7 ++++ > programs/cmd/wcmdmain.c | 65 +++++++++++++++++++++++++++++++- > 4 files changed, 158 insertions(+), 11 deletions(-) > > > Regards, Ruslan

1 0

Compiler warnings on Debian kfreebsd-i386
by Ken Sharp 11 Aug '13

11 Aug '13

Some interesting, some not: /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/jscript/parser.y: conflicts: 1 shift/reduce, 18 reduce/reduce /home/ken/wine-git/dlls/msi/sql.y: conflicts: 1 reduce/reduce /home/ken/wine-git/dlls/ntdll/directory.c: In function ‘wine_getdirentries’: /home/ken/wine-git/dlls/ntdll/directory.c:1710:5: warning: passing argument 4 of ‘getdirentries’ from incompatible pointer type [enabled by default] In file included from /home/ken/wine-git/dlls/ntdll/directory.c:29:0: /usr/include/dirent.h:313:18: note: expected ‘__off_t * __restrict__’ but argument is of type ‘long int *’ /home/ken/wine-git/dlls/vbscript/parser.y: conflicts: 6 shift/reduce /home/ken/wine-git/programs/winedbg/dbg.y: conflicts: 1 shift/reduce, 1 reduce/reduce /home/ken/wine-git/programs/winedbg/dbg.y: conflicts: 1 shift/reduce, 1 reduce/reduce

2 5

Is it a little bug in wbemprox?
by Jing Li 10 Aug '13

10 Aug '13

Hi, When I ran a simple win32 program just via WMI with query of "SELECT * from Win32_Directory", wine1.6 crashed. And when I checked the related code I found that the function of "static UINT seed_dirs( struct dirstack *dirstack, const struct expr *cond, WCHAR root, UINT *count )", in dlls/wbemprox/builtin.c:1086, accessing cond's member without checking if it's NULL! Unfortunately a NULL is passed to it. That's why my program crashed. It's a bug, right?

2 1

Re: [PATCH 01/11] widl: New option --rt for enabling rt's specific language extensions.
by André Hentschel 09 Aug '13

09 Aug '13

Am 08.08.2013 17:51, schrieb Kai Tietz: > Hi, > > this patch adds new --rt option to widl. By it you can control, if > RT's IDL-language-extension(?s) getting active. > This initial patch just adds the option-scanning and the global > flag-variable to widl. > > Ok for apply? I'd say no, you are adding dead code, so you might want to merge it with 2/11 to avoid that.

3 2

Re: [2/2] wineserver: perform access check when creating event objects (try 2)
by Andrew Cook 09 Aug '13

09 Aug '13

Actually ignore this patch, the problem was introduced by 92db6d2c back in 2007, server: Don't do access checks on the security descriptors of newly created objects which fixes sync.c:363 "CreateEventW with blank sd failed" i'll need to take another look at this, calling CreateEventEx with generic flags fails to correctly set permissions, the patch as it stands though will likely cause regressions. On 02/08/13 21:58, Andrew Cook wrote: > --- > dlls/ntdll/tests/om.c | 4 ++-- > server/event.c | 5 +---- > 2 files changed, 3 insertions(+), 6 deletions(-) > >

1 0

Re: configure.ac: allow disabling winemac.drv
by Ken Thomases 08 Aug '13

08 Aug '13

On Aug 8, 2013, at 5:56 PM, Austin English wrote: > +AC_ARG_WITH(winemac, AS_HELP_STRING([--without-winemac],[do not build native Mac (Cocoa) driver]), > + [if test "x$withval" = "xno"; then ac_cv_header_ApplicationServices_ApplicationServices_h=no; fi]) A switch to disable the Mac driver is fine with me, but you can't pretend that the ApplicationServices framework header isn't present. There are other places within Wine which check that. http://source.winehq.org/source/dlls/windowscodecs/icnsformat.c#L24 http://source.winehq.org/source/dlls/winspool.drv/info.c#L48 -Ken

1 0

Re: [PATCH 04/11] widl: Handle default value for public vs hidden attribute.
by André Hentschel 08 Aug '13

08 Aug '13

Am 08.08.2013 17:57, schrieb Kai Tietz: > - if (! is_attr(attrs, ATTR_PUBLIC)) > + if (! is_attr(attrs, ATTR_PUBLIC) && ! is_attr (attrs, ATTR_HIDDEN)) style issue: space after !

2 1

Re: [PATCH 09/11] widl: Handle LLP64 target same as LP64 one.
by André Hentschel 08 Aug '13

08 Aug '13

Am 08.08.2013 18:07, schrieb Kai Tietz: > Hi, > > this thing here puzzles me most. I reject this patch, but would like > to get here some discussion to find the actual intended meaning of > this xstrtoul function. Then you should remove it from the patchset and send it as RFC to wine-devel

1 0

Re: Synchronization-patches of widl
by André Hentschel 08 Aug '13

08 Aug '13

Am 08.08.2013 10:56, schrieb Kai Tietz: > All these changes were necessary to support the VLC-people to begin > with their WinRT port. By this we began on mingw-w64 to improve - > thanks to Jacek - our IDL-generated header-sources. So you are on a good way with WinRT, nice. What about Windows RT, was there already some ARM work? > A general note I would like to make about the use of sizeof(void*) in > widl to make out what type-library-variant to be choosen. That seems > to be a bit wacky variant, as it makes widl non-useable as cross-tool > on host with different pointer-size. isn't that controled by command line arguments?

1 0

winetestbot question
by Stefan Leichter 08 Aug '13

08 Aug '13

Hello, i like to know why the WXPX64 VM of winetestbot does not have the 32 bit vc runtime 2008. The other 64 bit VMs have the runtime installed. Because of the missing runtime each patch for the unit tests of the dlls msvcr90 and msvcp90 is marked as "Failed" at http://source.winehq.org/patches -- Stefan

2 1

re: msvcrt: strncpy doesn't compliant C standard (try 3)
by Dan Kegel 08 Aug '13

08 Aug '13

Minor problem: +static void test_strncpy(void) +{ + size_t len = 10; + char *ret; + char dst[len + 1]; Hmm. That last line is a VLA, and might not compile in all C compilers because it's not allowed in C89. http://stackoverflow.com/questions/448844/variable-sized-arrays-in-c Wine seems to want to build in c89 compilers: http://www.winehq.org/pipermail/wine-cvs/2011-April/076721.html Try using a #define with a more unique name for the length instead, or better yet, replace len=10 with len=sizeof(dst)-1 or something. Larger problem: ret = strncpy(dst, not_null_less, len); This is ill-defined. You're copying 10 bytes from a 5 byte buffer. Don't do that. Also, passing a not-null-terminated string in as a special test case doesn't seem interesting... just pass in the longer null-terminated string, and specify a shorter length. Fewer test strings needed, easier to read code. Please run 'make check', and test using http://testbot.winehq.org, before submitting patches.

2 1

Re: [PATCH 1/1] msvcrt: Set ?: environment variable in _(w)chdir functions.
by Marvin 08 Aug '13

08 Aug '13

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=26639 Your paranoid android. === WINEBUILD (build) === Make failed

1 0

Re: msvcrt: strncpy doesn't compliant C standard (try 3)
by Marvin 07 Aug '13

07 Aug '13

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=26638 Your paranoid android. === W2KPROSP4 (32 bit string) === string.c:2585: Test failed: dst != 012345 === WXPPROSP2 (32 bit string) === string.c:2585: Test failed: dst != 012345 === WXPX64 (32 bit string) === string.c:2585: Test failed: dst != 012345 === W2K3R2SESP2 (32 bit string) === string.c:2585: Test failed: dst != 012345 === WVISTAX64 (32 bit string) === string.c:2585: Test failed: dst != 012345 === W2K8SE (32 bit string) === string.c:2585: Test failed: dst != 012345 === W7PROX64 (32 bit string) === string.c:2585: Test failed: dst != 012345 === TEST64_W7SP1 (32 bit string) === string.c:2585: Test failed: dst != 012345

1 0

Re: msvcp90/tests: Add dynamically loaded functions for ARM
by Marvin 07 Aug '13

07 Aug '13

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=26637 Your paranoid android. === WXPX64 (32 bit misc) === No test summary line found === WXPX64 (32 bit ios) === No test summary line found

1 0

Re: Request to add wow64 keyword to bugzilla
by Ken Sharp 07 Aug '13

07 Aug '13

+1 from me. On 07/08/13 15:03, Rosanne DiMesio wrote: > On Wed, 07 Aug 2013 14:23:53 +0100 > Ken Sharp <kennybobs(a)o2.co.uk> wrote: > > >> Would I be right in assuming you would like to see bugs in 32-bit >> applications that are only present in a wow64 WINEPREFIX? Are there many? > > Yes. As to how many there are, I've been able to find 7 by doing searches of summaries/comments for things like "wow64" and "64" and then reading the bugs to see if they fit, but since people have such wildly varying ways of phrasing things, I don't know if that's all of them. I have seen it often enough on the forum that I now routinely tell 64 bit users to try a 32 bit wineprefix and have added instructions on how to create one to the FAQ. That's why I'd like an easier way to track these bugs. I see. A lot of people say this over and over again. +1 from me for a wow64 keyword. They may even be fairly easy to solve (picking the right paths / registry entries). > > As to keyword usage, some of those bugs were tagged with the win64 keyword, some weren't. I know there is also a win32 keyword, and perhaps that is meant to be used for the kinds of bugs I am looking for, but if so, that is not at all clear from the keyword definition, and it is not how people are using the keywords. > There's no win32 keyword?

3 2

Re: msvcp90/tests: Skip tests on missing functions
by Piotr Caban 07 Aug '13

07 Aug '13

On 08/06/13 23:00, André Hentschel wrote: > @@ -749,6 +749,13 @@ static BOOL init(void) > > SET(p_basic_istream_char_read_uint64, > "??5?$basic_istream@DU?$char_traits@D@std@@@std@@QAEAAV01@AA_K@Z"); > + > + if (!p_basic_istream_char_read_uint64) > + { > + skip("_basic_istream_char_read_uint64 not found, skipping tests\n"); > + return FALSE; > + } Why are you skipping the tests instead of properly initializing functions pointers on ARM?

2 1

Request to add wow64 keyword to bugzilla
by Rosanne DiMesio 07 Aug '13

07 Aug '13

Can wow64 be added as a keyword to bugzilla? I know we already have win64 as a keyword, but that's being used for both 64 bit apps and 32 bit apps in a 64 bit wineprefix. I'm interested in being able to track the latter, as it's hitting increasing numbers of users. -- Rosanne DiMesio <dimesio(a)earthlink.net>

2 1

Re: [PATCH] typo fixes (https://github.com/vlajos/misspell_fixer)
by Frédéric Delanoy 07 Aug '13

07 Aug '13

On Mon, Aug 5, 2013 at 9:41 PM, Veres Lajos <vlajos(a)gmail.com> wrote: > > Signed-off-by: Veres Lajos <vlajos(a)gmail.com> > --- > dlls/comctl32/tests/trackbar.c | 2 +- > documentation/ChangeLog.ALPHA | 80 ++++++++++++++++++++-------------------- > documentation/ChangeLog.BETA | 44 +++++++++++----------- > documentation/ChangeLog.OLD | 24 ++++++------ > tools/winebuild/utils.c | 2 +- > 5 files changed, 76 insertions(+), 76 deletions(-) The "documentation/ChangeLog.*" files have been removed so the patch won't apply. Frédéric Delanoy

1 0

Re: msvcrt: strncpy doesn't compliant C standard (try 2)
by Frédéric Delanoy 07 Aug '13

07 Aug '13

On Tue, Aug 6, 2013 at 10:44 PM, Álvaro Nieto <alvaro.nieto(a)gmail.com> wrote: > Remove strlen and implement some tests You should probably add tests for when the source string is not null-terminated. Frédéric Delanoy

1 0

Re: msvcp90/tests: Skip tests on missing functions
by Marvin 06 Aug '13

06 Aug '13

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=26628 Your paranoid android. === WXPX64 (32 bit misc) === No test summary line found === WXPX64 (32 bit ios) === No test summary line found

1 0

Re: msvcr90/tests: Skip when _type_info_name_internal_method is not found
by Marvin 06 Aug '13

06 Aug '13

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=26627 Your paranoid android. === WXPX64 (32 bit msvcr90) === No test summary line found

1 0

Re: ws2_32/tests: Fix a comment
by Juan Lang 06 Aug '13

06 Aug '13

Hi Andre, - /* this is a io heavy test, do it at the end so the kernel doesn't start dropping packets */ + /* this is a heavy io test, do it at the end so the kernel doesn't start dropping packets */ To my eyes, this isn't an improvement. A slight improvement might be "this is a io-heavy test", or "this is an io-heavy test", but I'm not sure it's enough of an improvement to bother. --Juan

1 0

Re: msvcrt: strncpy doesn't compliant C standard
by Nikolay Sivov 06 Aug '13

06 Aug '13

On 8/6/2013 11:09, Álvaro Nieto wrote: > This patch solves [Bug 34211]. The implementation of strncpy function > doesn't compliant with C standard [1]. Also Microsoft Visual Studio > C/C++ compiler is ok with the standard [2]. > > Extract from msdn; > > "The strncpy function copies the initial count characters of strSource > to strDest and returns strDest. If count is less than or equal to the > length of strSource, a null character is not appended automatically to > the copied string. If count is greater than the length of strSource, > the destination string is padded with null characters up to length > count. The behavior of strncpy is undefined if the source and > destination strings overlap." > > [1] http://pubs.opengroup.org/onlinepubs/009695399/functions/strncpy.html > [2] http://msdn.microsoft.com/en-us/library/vstudio/xdsywd25.aspx This needs tests.

1 0

Fwd: [AppDB] Submitted test data deleted
by Ken Sharp 05 Aug '13

05 Aug '13

Deleting test results because the Wine version is old? What the Hell is the plan there? -------- Original Message -------- Subject: [AppDB] Submitted test data deleted Date: Mon, 05 Aug 2013 15:48:43 -0500 From: AppDB <appdb-noreply(a)winehq.org> Reply-To: AppDB <appdb-noreply(a)winehq.org> To: appdb-noreply(a)winehq.org Submitted test data deleted ------------------------------------------------------- The test report you submitted for 'Microsoft Access 2007' has been deleted. The action was performed by Joerg Schiermeier Reasons given Old wine - deprecated. Best regards. The AppDB team http://appdb.winehq.org/ If you don't want to receive any other e-mail, please change your preferences: http://appdb.winehq.org/preferences.php

1 0

wine mannual is wrong?
by 中川祥 05 Aug '13

05 Aug '13

I'm interpretering wine.man. I found it says"this requires X11 to run". But the newest ver. does not require X11,does it?

3 2

Draft: improve IMC/IMCC related functions
by Qian Hong 05 Aug '13

05 Aug '13

Hello, This is a follow up of http://www.winehq.org/pipermail/wine-devel/2013-July/100459.html ( [PATCH] imm32: Fixed crashing in ImmGetIMCCSize. ) I believe the attached patchset is the correct fix for the Office crashing bug, how ever, I'm looking for a more graceful way to improve ImmCreateContext and other IMC functions, I appreciate to any feedback on the patches especially Patch 6/6. My tests show that Windows return a HGLOBAL hmem for ImmCreateIMCC() as the returned HIMCC, so it is easy to re-implement ImmCreateIMCC in a graceful way. However, Windows return a magic number for ImmCreateContext, (for instance, have a look at 000B01D9 in the test show in https://testbot.winehq.org/JobDetails.pl?Key=26619&log_207=1#k207) I have no idea how that magic number comes, while I understand there is no need to re-implement ImmCreateContext in the same way as Windows does, I'm still interesting in a more beautiful implementation like my ImmCreateIMCC reached, get rid of casting InputContextData to HIMC if possible. Thanks in advance for any suggestions! If there is no better way, I'll send this series to wine-patches. -- Regards, Qian Hong - http://www.winehq.org

1 0

Re: inputscope.idl: Imported from mingw-w64.
by Ken Sharp 05 Aug '13

05 Aug '13

On 05/08/13 12:14, Jacek Caban wrote: > + * No warranty is given; refer to the file DISCLAIMER.PD within this package. It's a minor point but this may be a bit confusing given the file doesn't exist.

1 0

Re: msvcrt: invoke invalid_handler in _wcscpy_s
by Piotr Caban 05 Aug '13

05 Aug '13

On 08/02/13 20:08, Daniel Lehman wrote: > + if(!MSVCRT_CHECK_PMT(wcDest)) > + { > + *MSVCRT__errno() = MSVCRT_EINVAL; > return MSVCRT_EINVAL; > + } The MSVCRT_CHECK_PMT macro sets the errno value. There's also MSVCRT_CHECK_PMT_ERR macro if you need to set different error.

1 0

po: Add English (Philippines) resource
by Ken Sharp 05 Aug '13

05 Aug '13

Hi everyone, Would anyone mind looking at the attached patch? It adds the English (Philippines) resource by linking (or copying) the English (US) resource. See http://bugs.winehq.org/show_bug.cgi?id=23124 Actually, the attached version links to fr.po (French) to make testing much easier. The final won't do that. It's explained in the bug. I have tested this successfully in Linux only. I wanted to test others but: Cygwin 1.7.22: creates its own "symlink" but no translations are built at all. Compilation is basically broken. PC-BSD 9.1: complains about gettext although the config.log seems to complain about gm4. I gave up. OpenIndiana: can't seem to find its own packages. Gave up. Debian kFreeBSD: all sorts of problems in VBox. Might try again later. So clearly I've had no luck on other platforms. If anyone else could quickly test it, that would be great! Thanks to Austin and François for the help thus far. Of course, all feedback welcome. TIA, Ken

2 3

Wiki RFC: Redirects, swarm tactics, etc.
by Kyle Auble 05 Aug '13

05 Aug '13

So I've finished with pretty much all of the edits I had in mind for the wiki, but before I ride off into the sunset for a while, I wanted to toss out a few ideas. 1. Do we want some kind of guideline on redirects for the wiki? Some stable "interface" pages to the main site might be good, but beyond that, I think minimizing redirects makes sense in this use case. 2. There's still a lot of old/missing content on the wiki, and much of it requres a good sense of where the code is at. Also, it might be too overwhelming for one person to work in depth on more than a few pages at this point. I feel like a semi-coordinated swarm of editing might be the best bet for further improvements. I was picturing a table of all relevant pages, then people could adopt one or two to work on, then strike/delete a record once that page is finished. Any thoughts? 3. There are actually a few more fixes to the theme code at the head of my bitbucket repo (and also branches for 2 different Moinmoin upgrade paths). I'm cool with keeping the theme code for the near future; if and when we move it onto WineHQ's git server though, just let me know so I can note that I'm not upstream anymore. 4. Finally, spammers... they keep coming... and they're getting smarter. We can mostly stalemate them with the regex filter, but it blocks legit edits too sometimes (in very annoying fashion). In fact, I think there are a few of them that have learned to turn the filter against us by completely wiping pages with false positives so we can't revert the page. Are there any relatively easy things we could do to cut the spam? I don't know, but it's possible the newer version of Moinmoin has more potent anti-spam tools. -Kyle

5 5

help [Abount Sql2005 and WMI]
by Jing Li 04 Aug '13

04 Aug '13

Hi, all: I have met a problem when installing sql2005, and it seems related to some WMI problems. As WMI has just been implemented in wine-1.6 ( right? ), I want to know in what degree it has been implemented.By the way is there anyone succeed in install of it? Thanks. Appreciate your kind help.

2 1

Vacation
by Alexandre Julliard 02 Aug '13

02 Aug '13

Folks, I'll be on vacation for the next 10 days, so you'll have to live without commits for a while... -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [docs] wineusr: Assorted formatting & tagging fixes (try 2)
by Alexandre Julliard 02 Aug '13

02 Aug '13

Frédéric Delanoy <frederic.delanoy(a)gmail.com> writes: > @@ -710,17 +714,18 @@ Path="c:\windows;c:\windows\system;e:\;e:\test;f:\" > choose between several edition modes: > <itemizedlist> > <listitem> > - <para> > - Emacs: the same keybindings as under > - emacs are available. For example, Ctrl-A > - will bring the cursor to the beginning > - of the edition line. See your emacs > - manual for the details of the commands. > - </para> > + <para> > + <literal>Emacs</literal>: the same keybindings as under > + <command>emacs</command> are available. For example, > + <keycombo><keycap>Ctrl</keycap><keycap>A</keycap></keycombo> > + will bring the cursor to the beginning > + of the edition line. See your <command>emacs</command> > + manual for the details of the commands. > + </para> Please avoid messing up the indentation. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: gdi32: Avoid a crash by loading AppKit before using Core Text if might use the Mac driver.
by Alexandre Julliard 02 Aug '13

02 Aug '13

Ken Thomases <ken(a)codeweavers.com> writes: > + /* If the Mac driver might be used, then load AppKit now before using the Core Text API. > + Otherwise, AppKit crashes on Mac OS X 10.7+ because CTFontDescriptor isn't toll-free > + bridged to NSCTFontDescriptor. That bridging is only set up if AppKit has been loaded > + when CTFontDescriptor is first used. */ > + /* @@ Wine registry key: HKCU\Software\Wine\Drivers */ > + if (!RegOpenKeyA(HKEY_CURRENT_USER, "Software\\Wine\\Drivers", &hkey)) > + { > + char buffer[MAX_PATH]; > + DWORD type, count = sizeof(buffer); > + > + if (!RegQueryValueExA(hkey, "Graphics", 0, &type, (LPBYTE) buffer, &count)) > + { > + char *name = buffer; > + > + usingMacDriver = FALSE; > + while (name) > + { > + char *next = strchr(name, ','); > + if (next) *next++ = 0; > + > + if (!strcmp(name, "mac")) > + { > + usingMacDriver = TRUE; > + break; > + } > + > + name = next; > + } > + } > + > + RegCloseKey(hkey); > + } That's ugly. freetype.c has no business knowing about the details of the graphics driver. -- Alexandre Julliard julliard(a)winehq.org

2 5

Re: riched20: Make ITextDocument stub reusable for ITextServices (try 4)
by Alexandre Julliard 02 Aug '13

02 Aug '13

Caibin Chen <tigersoldi(a)gmail.com> writes: > diff --git a/dlls/riched20/tomimpl.h b/dlls/riched20/tomimpl.h > new file mode 100644 > index 0000000..886c3a8 > --- /dev/null > +++ b/dlls/riched20/tomimpl.h > @@ -0,0 +1,59 @@ > +/* > + * RichEdit - TOM interfaces implementations > + * > + * Copyright 2013 by Caibin Chen > + * > + * This library is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. > + * > + * This library is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this library; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA > + */ > + > +#ifndef __RICHOLE_H > +#define __RICHOLE_H > + > +#include <tom.h> > + > +#include "editstr.h" > + > +typedef struct ITextDocumentImpl ITextDocumentImpl; > +typedef struct ITextSelectionImpl ITextSelectionImpl; > + > +/* ITextDocument */ > + > +struct ITextDocumentImpl { > + ITextDocument ITextDocument_iface; > + IUnknown *outer_unk; > + ME_TextEditor *editor; > + ITextSelectionImpl *txtSel; > +}; You shouldn't need a new header either. Particularly since implementation details shouldn't be global. > +/** > + * Create an ITextDocumentImpl object, which acts as an inner object of COM aggression. I'm sure writing COM code can make you feel aggressive, but that's not the right term ;-) > + * > + * The ITextDocumentImpl object will delegate all of its IUnknown calls to the specified > + * {@code outer_unk}. > + * > + * @param outer_unk The outer objects that creates and delegates ITextDocument > + * method calls to the created ITextDocumentImpl. > + * @param editor The editor implementation. > + * > + * @return An ITextDocumentImpl object that should be destroyed with > + * {@code ITextDocumentImpl_destroy} > + */ Please don't use that sort of documentation markup. Besides, things like parameters and return should be obvious from the code, you shouldn't have to add documentation for this, especially not for internal functions. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: xmllite: Improve attribute value parsing
by Alexandre Julliard 02 Aug '13

02 Aug '13

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > +static void reader_normalize_space(xmlreader *reader, WCHAR *ptr) > +{ > + encoded_buffer *buffer = &reader->input->buffer->utf16; > + > + if (!is_wchar_space(*ptr)) return; > + > + if (*ptr == '\r' && *ptr == '\n') That's not going to happen very often... -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] oleaut32: ERR on 32bit typelibs on 64bit
by Alexandre Julliard 02 Aug '13

02 Aug '13

Marcus Meissner <meissner(a)suse.de> writes: > Triggered by the Novell Groupwise installer currently, > from the ISBE64W.exe Installshield Helper program. > > The typelib is contained directly in ISBE64W.exe itself :/ > > Basically if we encounter this case the program will crash > anyway (but with random backtrace), so output an ERR(). That looks like it's just hiding the problem. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: d3dx9 [patch 1/2]: Implement D3DXCreatePolygon
by Matteo Bruni 02 Aug '13

02 Aug '13

2013/8/1 Nozomi Kodama <nozomi.kodama(a)yahoo.com>: > + vertices = HeapAlloc(GetProcessHeap(), 0, 2 * (sides + 1) * sizeof(D3DXVECTOR3)); + if (!vertices) + { + TRACE("Not memory enough for vertex buffer\n"); + polygon->lpVtbl->Release(polygon); + return E_OUTOFMEMORY; + } + + hr = polygon->lpVtbl->LockVertexBuffer(polygon, D3DLOCK_DISCARD, (VOID **)&vertices); Why are you allocating memory and immediately dropping track of it? Recheck how ID3DXMesh::LockVertexBuffer works. Also I'm having a deja-vu here, this patch looks suspiciously similar (up to having this same issue) to one I've already reviewed in the past.

2 1

appwiz: Correct Wine Mono installer title in several languages
by Ken Sharp 02 Aug '13

02 Aug '13

Hi Alexandre, http://source.winehq.org/patches/data/97460 should be correct but would I be right in assuming you would rather have the changes made along with the next string for each language?

2 4

Re: [PATCH 4/5] d3dx9/tests: Add ID3DXConstantTable struct test.
by Matteo Bruni 02 Aug '13

02 Aug '13

2013/7/24 Rico Schüller <kgbricola(a)web.de>: > --- > dlls/d3dx9_36/tests/shader.c | 308 > +++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 308 insertions(+) > This is okay, but as a followup can you add some tests with mixed-type structs? Something like: struct { float f; int i; bool b; }; If you have already written tests of this kind, I'd like to know what does the compiler do in this case :)

2 6

Re: [1/2] ntdll: Implement NtQueryEvent
by Alexandre Julliard 02 Aug '13

02 Aug '13

Andrew Cook <ariscop(a)gmail.com> writes: > diff --git a/dlls/ntdll/tests/Makefile.in b/dlls/ntdll/tests/Makefile.in > index 10d6674..a44b880 100644 > --- a/dlls/ntdll/tests/Makefile.in > +++ b/dlls/ntdll/tests/Makefile.in > @@ -7,6 +7,7 @@ C_SRCS = \ > directory.c \ > env.c \ > error.c \ > + event.c \ > exception.c \ > file.c \ > generated.c \ Please don't add a new file for just a couple of tests, merge this into an existing file. -- Alexandre Julliard julliard(a)winehq.org

1 0

jscript FTBFS in Cygwin 1.7.22
by Ken Sharp 02 Aug '13

02 Aug '13

Probably better to post this here rather than the forums: http://forum.winehq.org/viewtopic.php?f=2&t=19501 "Not sure if this is a Wine bug (as usual) so I'd rather put this here than to open a new bug. On Cygwin 1.7.22 the compilation stops at jscript, apparently a conflict in the declaration of dtoa. $ make ccache cc -c -I/home/ken/wine-git/dlls/jscript -I. -I/home/ken/wine-git/include -I../../include -D__WINESRC__ -D_REENTRANT -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wwrite-strings -Wpointer-arith -Wlogical-op -gdwarf-2 -gstrict-dwarf -fno-omit-frame-pointer -g -O0 -D_WIN32 -o jscript_main.o /home/ken/wine-git/dlls/jscript/jscript_main.c ccache cc -c -I/home/ken/wine-git/dlls/jscript -I. -I/home/ken/wine-git/include -I../../include -D__WINESRC__ -D_REENTRANT -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wwrite-strings -Wpointer-arith -Wlogical-op -gdwarf-2 -gstrict-dwarf -fno-omit-frame-pointer -g -O0 -D_WIN32 -o lex.o /home/ken/wine-git/dlls/jscript/lex.c ccache cc -c -I/home/ken/wine-git/dlls/jscript -I. -I/home/ken/wine-git/include -I../../include -D__WINESRC__ -D_REENTRANT -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wwrite-strings -Wpointer-arith -Wlogical-op -gdwarf-2 -gstrict-dwarf -fno-omit-frame-pointer -g -O0 -D_WIN32 -o number.o /home/ken/wine-git/dlls/jscript/number.c /home/ken/wine-git/dlls/jscript/number.c:57:20: error: conflicting types for ‘dtoa’ In file included from /home/ken/wine-git/include/wine/port.h:47:0, from /home/ken/wine-git/dlls/jscript/number.c:20: /usr/include/stdlib.h:161:35: note: previous declaration of ‘dtoa’ was here Makefile:199: recipe for target `number.o' failed make: *** [number.o] Error 1 dlls/jscript/number.c: static inline void dtoa(double d, WCHAR *buf, int size, int *dec_point) /usr/include/stdlib.h: char * _EXFUN(dtoa,(double, int, int, int *, int*, char**));"

1 0

Re: ntdll: Implement compatible FindActCtxSectionString() for window class section (try3)
by Alexandre Julliard 02 Aug '13

02 Aug '13

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > @@ -1031,6 +1105,17 @@ static BOOL parse_window_class_elem(xmlbuf_t* xmlbuf, struct dll_redirect* dll) > > if (!(entity->u.class.name = xmlstrdupW(&content))) return FALSE; > > + /* each class entry needs index, data and string data */ > + acl->actctx->wndclass_section_len += sizeof(struct wndclass_index); > + acl->actctx->wndclass_section_len += sizeof(struct wndclass_redirect_data); > + /* original name is stored separately */ > + acl->actctx->wndclass_section_len += aligned_string_len((content.len+1)*sizeof(WCHAR)); > + /* versioned name and module name are stored one after another */ > + len = get_assembly_version(assembly, NULL) + content.len + 2 /* null terminator and '!' separator */; > + len += strlenW(dll->name) + 1; > + acl->actctx->wndclass_section_len += aligned_string_len(len*sizeof(WCHAR)); > + acl->actctx->wndclass_count++; > + It would be better to do this at the time you build the structure. The parser shouldn't need to worry about this. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: po: Add English (Canada) resource [Take 2]
by Ken Sharp 02 Aug '13

02 Aug '13

Please disregard this patch. I missed a couple of words and with those corrected en_CA.po becomes identical to en.po (except that "serialisation" is fuzzy) and Wine defaults to that anyway for Canadian English. Will re-submit if there are any changes (just noticed -able/-eable). Will keep an eye on the English translations. On 30/07/13 22:37, Ken Sharp wrote: > - Re-based to latest git > - Removed all the fuzzies thanks to http://www.etymonline.com/index.php > > Turns out all the -ise/-ize words used in Wine are of Latin origin anyway. > > > -------- Original Message -------- > Subject: po: Add English (Canada) resource > Date: Mon, 22 Jul 2013 19:28:51 +0100 > From: Ken Sharp <kennybobs(a)o2.co.uk> > To: wine-patches(a)winehq.org > > Canadian English follows some British/French rules but not others. > > I have left some of the -ise/-ize words fuzzy because: > > "the etymological convention that verbs derived from Greek roots are > spelled with -ize and those from Latin with -ise is preserved in that > practice." > http://en.wikipedia.org/wiki/Canadian_english#Spelling_and_dictionaries > > I tried looking for the origins of the words and I couldn't get any. The > non-fuzzies are correct. > > With British, US and Canadian I believe all the English variations (used > in Wine) are covered, except Philippines (follows US rules). > > >

1 1

Re: reg.exe: Add query function
by Nikolay Sivov 02 Aug '13

02 Aug '13

On 8/1/2013 17:44, Hugh McMaster wrote: > This patch builds significantly on the query function stub in programs/reg.exe. The patch includes recursion for subkeys. > > Some examples of usage: > > wine reg.exe query "HKCU" > wine reg.exe query "HKCU\Console" > wine reg.exe query "HKCU\Console" /v "ScreenSize" > wine reg.exe query "HKCU\Software" /s > > --- > programs/reg/reg.c | 232 +++++++++++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 221 insertions(+), 11 deletions(-) > > +static LPVOID WINAPI mem_alloc(size_t size) > +{ > + return HeapAlloc(GetProcessHeap(), 0, size); > +} > + > +static void WINAPI mem_free(LPVOID mem) > +{ > + HeapFree(GetProcessHeap(), 0, mem); > +} Why WINAPI? It's better to follow naming from existing module, there's plenty of them with similar helpers. > + case REG_SZ: > + case REG_MULTI_SZ: > + case REG_EXPAND_SZ: No tabs please. > +static void reg_output_subkey(LPWSTR key_name, LPWSTR subkeyName) > +{ > + reg_printfW(formatSW, key_name); > + > + if (subkeyName > 0) > + { > + reg_printfW(formatSW, slashW); > + reg_printfW(formatSW, subkeyName); > + reg_printfW(formatSW, newlineW); > + } > +} Comparison looks a bit odd. Also why do you need to break it down to three printf calls instead of one? > + p = strchrW(key_name,'\\'); > + if (!p) > + { > + p = 0; > + } > + else p++; I'm not sure what this is supposed to do. > + case REG_BINARY: > + case REG_NONE: > + pValue = value; > + for (i=0; i<valueSize; i++, pValue++) > + reg_printfW(formatXW, *pValue); > + break; There's no need for separate variable here nor for incrementing pointer.

2 3

Clang warns on "wrong" header guard in "config.h"
by Charles Davis 02 Aug '13

02 Aug '13

Hi, I've been building Wine with Clang, and it (apparently, unlike GCC) produces warnings like this: ../../include/config.h:4:9: warning: 'WINE_CROSSTEST' is used as a header guard here, followed by #define of a different macro [-Wheader-guard] #ifndef WINE_CROSSTEST ^~~~~~~~~~~~~~ ../../include/config.h:5:9: note: '__WINE_CONFIG_H' is defined here; did you mean 'WINE_CROSSTEST'? #define __WINE_CONFIG_H ^~~~~~~~~~~~~~~ WINE_CROSSTEST I tracked this down to these three lines in configure.ac: AH_TOP([#ifndef WINE_CROSSTEST #define __WINE_CONFIG_H]) AH_BOTTOM([#endif /* WINE_CROSSTEST */]) These lines generate what Clang thinks is the header guard for the "config.h" header, but we know that it isn't--that header AFAICT has never had an include guard, and __WINE_CONFIG_H is just there to indicate that it's been included. Should we change the __WINE_CONFIG_H macro to be a real header guard instead, or (as I'll admit it has been in the past) is Clang broken here? I'd imagine you guys don't want to rig configure to add -Wno-header-guard to CFLAGS if you can help it. I will point out that lots of these warnings are generated when building with Clang, because (for obvious reasons) this particular header is included all over the place. It makes it harder to see real warnings (like the ones that caused AJ not to commit my last two patches). And please don't tell me to use GCC instead. I suspect that GCC might pick up a similar warning--there's been lots of cross pollination going on between GCC and Clang, so we might have to deal with this sooner or later--and I'd prefer sooner. (I know the commit that added the WINE_CROSSTEST checks is from over 3 years ago, but this warning was only added recently to Clang, which is why I'm only bringing this up now.) Chip

2 1

Re: [PATCH] d3dx9_36: Implement ID3DXEffect_FindNextValidTechnique + add tests.
by Rico Schüller 02 Aug '13

02 Aug '13

On 02.08.2013 00:03, Christian Costa wrote: > + technique = (D3DXHANDLE)0xdeadbeef; > + hr = effect->lpVtbl->FindNextValidTechnique(effect, technique1, &technique); > + ok(hr == D3D_OK, "FindNextValidTechnique failed, got %#x, expected %#x\n", hr, D3D_OK); > + ok(technique == technique2, "Technique returned %p, expected %p\n", technique, technique2); > + > + technique = (D3DXHANDLE)0xdeadbeef; > + hr = effect->lpVtbl->FindNextValidTechnique(effect, technique2, &technique); > + ok(hr == S_FALSE, "FindNextValidTechnique, got %#x, expected %#x\n", hr, S_FALSE); Looks fine to me, but I found one minor issue. I'd write "FindNextValidTechnique failed, ..." as in the line above. Cheers Rico

1 0

Re: [11/11] apphelp: Add tests.
by Marvin 01 Aug '13

01 Aug '13

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=26579 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

gdi32: Don't let the gdi32 driver allocate memory in the dplayx region.
by Alexei Svitkine 01 Aug '13

01 Aug '13

Don't let the gdi32 driver allocate memory in the dplayx region. Fixes bug: http://bugs.winehq.org/show_bug.cgi?id=34095 The issue is that the dplayx code in wine needs to allocate some memory at a static address (0x50000000) and loading the Mac native gdi driver ends up using that part of the address space. This patch fixes that by reserving that part of memory while the display driver is initialized and releasing it after. --- dlls/gdi32/driver.c | 10 ++++++++++ 1 file changed, 10 insertions(+)

2 5

Re: [PATCH 1/4] d3dx9: Track ID3DXEffect::Begin/End().
by Bruno Jesus 01 Aug '13

01 Aug '13

On Thu, Aug 1, 2013 at 9:14 AM, Rico Schüller <kgbricola(a)web.de> wrote: > --- > dlls/d3dx9_36/effect.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > + if (!This->started) > + return D3D_OK; > + > + This->started = FALSE; > + > + return D3D_OK; Sorry to bother but why not simply force false and using a single return? Best wishes, Bruno

2 1

Re: [PATCH 1/11] apphelp: Add definitions.
by Alexandre Julliard 01 Aug '13

01 Aug '13

Mislav Blazevic <krofnica996(a)gmail.com> writes: > @@ -25,6 +26,209 @@ extern "C" { > > BOOL WINAPI ApphelpCheckShellObject(REFCLSID, BOOL, ULONGLONG *); > > +typedef enum _PATH_TYPE { > + DOS_PATH, > + NT_PATH > +} PATH_TYPE; > + > +typedef struct _DB { > + HANDLE file; > + DWORD size; > + PBYTE data; > +} DB, *PDB; > + > +typedef DWORD TAGID; > +typedef WORD TAG; > + > +#define TAGID_NULL 0x0 > +#define TAGID_ROOT 0x0 > + > +/* The above definition of TAGID_ROOT is used in winapi but doesn't make > + * sense, especially internally, because TAGID represents offset into > + * database data and there is a header at offset 0, NOT a tag. > + * Therfore, this definition should only be used internally. */ > +#define _TAGID_ROOT 0xC If it's internal it shouldn't be in the public header. In fact I don't see any of these definitions in the SDK header. -- Alexandre Julliard julliard(a)winehq.org

1 0

reg.exe: Add query function
by Hugh McMaster 01 Aug '13

01 Aug '13

This patch significantly builds on the query function stub in programs/reg.exe. The patch includes recursion for subkeys. Some examples of usage: wine reg.exe query "HKCU" wine reg.exe query "HKCU\Console" wine reg.exe query "HKCU\Console" /v "ScreenSize" wine reg.exe query "HKCU\Software" /s --- programs/reg/reg.c | 232 +++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 221 insertions(+), 11 deletions(-)

2 2

Re: [PATCH 2/2] ddraw: Fix double to float conversion warnings
by Henri Verbeet 01 Aug '13

01 Aug '13

On 1 August 2013 12:25, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > diff --git a/dlls/ddraw/executebuffer.c b/dlls/ddraw/executebuffer.c > index 2925cfa..5c12272 100644 > --- a/dlls/ddraw/executebuffer.c > +++ b/dlls/ddraw/executebuffer.c > @@ -382,10 +382,10 @@ HRESULT d3d_execute_buffer_execute(struct d3d_execute_buffer *buffer, > dst->u8.tv = src->u8.tv; > > /* Now, the matrix multiplication */ > - dst->u1.sx = (src->u1.x * mat._11) + (src->u2.y * mat._21) + (src->u3.z * mat._31) + (1.0 * mat._41); > - dst->u2.sy = (src->u1.x * mat._12) + (src->u2.y * mat._22) + (src->u3.z * mat._32) + (1.0 * mat._42); > - dst->u3.sz = (src->u1.x * mat._13) + (src->u2.y * mat._23) + (src->u3.z * mat._33) + (1.0 * mat._43); > - dst->u4.rhw = (src->u1.x * mat._14) + (src->u2.y * mat._24) + (src->u3.z * mat._34) + (1.0 * mat._44); > + dst->u1.sx = (src->u1.x * mat._11) + (src->u2.y * mat._21) + (src->u3.z * mat._31) + (1.0f * mat._41); > + dst->u2.sy = (src->u1.x * mat._12) + (src->u2.y * mat._22) + (src->u3.z * mat._32) + (1.0f * mat._42); > + dst->u3.sz = (src->u1.x * mat._13) + (src->u2.y * mat._23) + (src->u3.z * mat._33) + (1.0f * mat._43); > + dst->u4.rhw = (src->u1.x * mat._14) + (src->u2.y * mat._24) + (src->u3.z * mat._34) + (1.0f * mat._44); > I get where the 1.0f comes from, but I think you might as well just remove the multiplication completely. And please get rid of the tabs if you're resending anyway.

1 0

Re: windowscodecs: Workaround libtiff bug when it defines toff_t as 32-bit for 32-bit builds. Take 3.
by Alexandre Julliard 01 Aug '13

01 Aug '13

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > @@ -208,9 +233,22 @@ static TIFF* tiff_open_stream(IStream *stream, const char *mode) > zero.QuadPart = 0; > IStream_Seek(stream, zero, STREAM_SEEK_SET, NULL); > > + /* Workaround for broken libtiff 4.x headers on some 64-bit hosts which > + * define TIFF_UINT64_T/toff_t as 32-bit for 32-bit builds, while they > + * are supposed to be always 64-bit. > + * TIFF_UINT64_T doesn't exist in libtiff 3.x, it was introduced in 4.x. > + */ > +#ifdef TIFF_UINT64_T > + /* libtiff 4.x */ > + return pTIFFClientOpen("<IStream object>", mode, stream, tiff_stream_read, > + tiff_stream_write, (void *)tiff_stream_seek, tiff_stream_close, > + (void *)tiff_stream_size, (void *)tiff_stream_map, (void *)tiff_stream_unmap); > +#else > + /* libtiff 3.x */ > return pTIFFClientOpen("<IStream object>", mode, stream, tiff_stream_read, > - tiff_stream_write, tiff_stream_seek, tiff_stream_close, > - tiff_stream_size, tiff_stream_map, tiff_stream_unmap); > + tiff_stream_write, tiff_stream_seek_v3, tiff_stream_close, > + tiff_stream_size_v3, tiff_stream_map_v3, tiff_stream_unmap_v3); > +#endif Now that you are using ifdefs there's no reason to have two sets of functions. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: oledb32: Add IDataSourceLocator support
by Alistair Leslie-Hughes 01 Aug '13

01 Aug '13

Ignore this patch On 1/08/2013 6:36 PM, Alistair Leslie-Hughes wrote: > Hi, > > > Changelog: > oledb32: Add IDataSourceLocator support > > > Best Regards > Alistair Leslie-Hughes > > >

1 0

Re: Re: apphelp: Implement SdbTagToString.
by Marvin 01 Aug '13

01 Aug '13

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=26567 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH 11/11] appelp: Add tests.
by Marvin 31 Jul '13

31 Jul '13

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=26546 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH 1/4] riched20: Basic ITextDocument implementation for ITextServices (try 3)
by Alexandre Julliard 31 Jul '13

31 Jul '13

Caibin Chen <tigersoldi(a)gmail.com> writes: > Thanks Jacek for reviewing. This time I send all the changes I have in > 4 patches to make it clear that ReTxtDoc is reused between txtsrv.c > and richole.c > - Suppressed Patch 97331 > - Change log from try 2: > - follow COM naming convention > - move ReTxtDoc definition to header, removed > ReTxtDoc_get_ITextDocument() function > - remove COM aggregation test for ITextDocument, only test > ITextService supports querying for the interface. > > --- > dlls/riched20/Makefile.in | 1 + > dlls/riched20/tests/txtsrv.c | 10 ++ > dlls/riched20/txtdoc.c | 270 +++++++++++++++++++++++++++++++++++++++++++ > dlls/riched20/txtdoc.h | 51 ++++++++ > dlls/riched20/txtsrv.c | 6 + Please avoid adding too many source files. In particular, if you have to add new headers and define objects there, it's a sign that things should be grouped in the same file. That's even more true for ITextDocument that already exists. -- Alexandre Julliard julliard(a)winehq.org

2 4

Re: [PATCH] d3dx9_36: Filter out D3DCompile warning messages that are not present with D3DCompileShader.
by Stefan Dösinger 31 Jul '13

31 Jul '13

Hi, Am 31.07.2013 um 09:10 schrieb Christian Costa <titan.costa(a)gmail.com>: > + if (!(strstr(line_out, "warning X3206:") && strstr(line_out, "implicit truncation of vector type"))) > + line_out += line_size; msvc localizes compiler errors and warnings. Are you sure d3dcompiler.dll doesn't do that? Wrt the parsing code, wouldn't it be nicer to use strtok_r to split the buffer into separate lines? I am not sure that parsing the errors and warnings like that is a good thing, but I don't know d3dx9 and d3dcompiler well enough to give a final judgement on that.

2 2

Re: ws2_32/tests: Test the precedence of parameters while creating a socket in WSASocket()
by Austin English 31 Jul '13

31 Jul '13

On Tue, Jul 30, 2013 at 5:13 PM, Bruno Jesus <00cpxxx(a)gmail.com> wrote: > > You forgot the patch. -- -Austin

3 2

d3dx9: Replace M_PI by its D3DX equivalent
by Nozomi Kodama 31 Jul '13

31 Jul '13

What is the problem with patch: http://source.winehq.org/patches/data/97500 M_PI is never used in D3DX (moreover it is deprecated in C99) Nozomi

2 1

Re: kernel32: Fix parameters checking for GetVolumePathName()
by Alexandre Julliard 31 Jul '13

31 Jul '13

Bruno Jesus <00cpxxx(a)gmail.com> writes: > @@ -1813,6 +1821,8 @@ BOOL WINAPI GetVolumePathNameW(LPCWSTR filename, LPWSTR volumepathname, DWORD bu > volumepathname[3] = 0; > return TRUE; > } > + > + SetLastError(ERROR_FILENAME_EXCED_RANGE); > return FALSE; This doesn't really make sense as a general error. -- Alexandre Julliard julliard(a)winehq.org

1 0

d3dx9: Replace M_PI by its D3DX equivalent
by Nozomi Kodama 31 Jul '13

31 Jul '13

Hello What is the problem with patch http://source.winehq.org/patches/data/97500 M_PI is never used in D3DX. Nozomi

1 0

Re: msxml3: Unescape '&' back to '&' in attribute value
by Marvin 31 Jul '13

31 Jul '13

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=26523 Your paranoid android. === W2KPROSP4 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === WXPPROSP2 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === WXPX64 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === W2K3R2SESP2 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === WVISTAX64 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === W2K8SE (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === W7PROX64 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === TEST64_W7SP1 (32 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === WXPX64 (64 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === WVISTAX64 (64 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === W7PROX64 (64 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) === TEST64_W7SP1 (64 bit saxreader) === saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & ) saxreader.c:2334: Test failed: len 30 (L" attr_value A & & "), expected 29 ( attr_value A & & )

1 0

Re: kernel32/tests: Improve GetVolumePathNameA tests
by Bruno Jesus 30 Jul '13

30 Jul '13

Please ignore this patch, I understand that as is it will not be commited. On Tue, Jul 30, 2013 at 3:30 AM, Bruno Jesus <00cpxxx(a)gmail.com> wrote: > Erich Hoover contacted me about a patch he sent months ago related to > GetVolumePathNameA. He developed a better test infrastructure using a > for loop instead of multiple repetitions of test lines. This patch > will make it easier for him to rebase and send his tests and function > implementation later. > > I used the same approach from [1] by decomposing the "todo_wine" in > winetest_start_todo("wine") and winetest_end_todo("wine"). > > http://source.winehq.org/source/programs/cmd/tests/batch.c#L292 -- universe* god::bigbang (void); //and then it all began...

1 0

Re: [PATCH] imm32: Fixed crashing in ImmGetIMCCSize.
by Nikolay Sivov 30 Jul '13

30 Jul '13

On 7/9/2013 16:50, Qian Hong wrote: > Fixed a crash in MS Office 2010 Chinese version. > Thanks Jiang Yike for the helps on testing :) > > --- > dlls/imm32/imm.c | 6 ++++++ > dlls/imm32/tests/imm32.c | 13 +++++++++++++ > 2 files changed, 19 insertions(+) > > > > > + if (IsBadReadPtr(internal, sizeof(HIMCC))) > + { > + SetLastError(ERROR_INVALID_HANDLE); > + return 0; > + } > + Do you really need this for Office 2010 or null check is enough? If it's passing 0 handle here that's what you should check imho. If HIMCC is really a handle and not a struct pointer like in wine, that's not how a handle validity check will look like.

2 4

Call for Windows tests
by Bruno Jesus 30 Jul '13

30 Jul '13

Hi, I'm looking for help to test WSAEnumProtocols, I need to collect more information about the values returned from different versions of windows, especially 7 and 8. If you can help please download the file below, unzip, run the .bat and send me back the result.txt file (sources included). http://influenza.blog.br/wsa.zip There is no need to answer to the list. Virustotal.com scan: http://bit.ly/1catW4a Thanks in advance, Bruno

1 0

Re: windowscodecs: Workaround libtiff bug when it defines toff_t as 32-bit for 32-bit builds. Take 2.
by Vincent Povirk 30 Jul '13

30 Jul '13

There's a TIFF_VERSION define that seems to have been renamed to TIFF_VERSION_CLASSIC in 4.0. It doesn't make sense to do the check at runtime, given we're only going to link to one of the versions.

3 7

Re: [1/2] msxml3: Implement IMXAttributes_removeAttributes()
by Jacek Caban 30 Jul '13

30 Jul '13

Hi Nikolay, On 07/30/13 12:42, Nikolay Sivov wrote: > + dst = &This->attr[index]; > + src = &This->attr[index+1]; > + > + memmove(dst, src, This->length-index-1); *sizeof(*dst) Cheers, Jacek

1 0

Re: [2/2] wininet: Handle NULL input string in str_to_buffer.
by Marvin 30 Jul '13

30 Jul '13

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=26482 Your paranoid android. === WXPX64 (64 bit http) === http.c:4661: Test failed: 4832: expected status 100 got 10 http.c:4661: Test failed: 4876: expected status 100 got 40 http.c:4634: Test failed: ar->dwResult = 0, expected 1 http.c:4661: Test failed: 4885: expected status 70 got 100 http: unhandled exception c0000005 at 0000000000495D94

1 0

Re: [1/2] wininet: Ignore INTERNET_FLAG_NO_CACHE_WRITE only for GET requests.
by Marvin 30 Jul '13

30 Jul '13

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=26481 Your paranoid android. === WXPX64 (32 bit http) === http.c:4588: Test failed: 4759: expected status 100 got 10 http.c:4588: Test failed: 4803: expected status 100 got 40 http.c:4561: Test failed: ar->dwResult = 0, expected 1 http.c:4588: Test failed: 4812: expected status 70 got 100 http: unhandled exception c0000005 at 7722EDEA

1 0

Re: [2/2] iphlpapi: Add partial support for the module classes in GetExtendedTcpTable and GetExtendedUdpTable.
by Marvin 30 Jul '13

30 Jul '13

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=26480 Your paranoid android. === WVISTAX64 (32 bit iphlpapi) === iphlpapi: unhandled exception c0000005 at 74A19E2F === W2K8SE (32 bit iphlpapi) === iphlpapi: unhandled exception c0000005 at 7596A09A === W7PROX64 (32 bit iphlpapi) === iphlpapi: unhandled exception c0000005 at 77E03915 === WVISTAX64 (64 bit iphlpapi) === Timeout === W7PROX64 (64 bit iphlpapi) === No test summary line found

1 0

Re: [1/2] iphlpapi: Add support for the listener and connection classes in GetExtendedTcpTable.
by Marvin 30 Jul '13

30 Jul '13

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=26479 Your paranoid android. === WVISTAX64 (32 bit iphlpapi) === iphlpapi: unhandled exception c0000005 at 74A19E2F === W2K8SE (32 bit iphlpapi) === iphlpapi: unhandled exception c0000005 at 7596A09A === WVISTAX64 (64 bit iphlpapi) === Timeout

1 0

Re: riched20: ITextDocument stubs for ITextServices (retry)
by Caibin Chen 30 Jul '13

30 Jul '13

Anybody reviewing this patch? It's almost all stubs, so although it's long, it shouldn't be hard to review 2013/7/15 Caibin Chen <tigersoldi(a)gmail.com>: > (Retry with real name in mail. The name in the patch is already my real name) > > This patch implements a standalone stub of ITextDocument. The stub is > currently used in txtsrv.c and will replace the stub in richole.c > > --- > dlls/riched20/Makefile.in | 1 + > dlls/riched20/tests/txtsrv.c | 16 ++- > dlls/riched20/txtdoc.c | 282 +++++++++++++++++++++++++++++++++++++++++++ > dlls/riched20/txtdoc.h | 46 +++++++ > dlls/riched20/txtsrv.c | 6 + > 5 files changed, 350 insertions(+), 1 deletion(-) > create mode 100644 dlls/riched20/txtdoc.c > create mode 100644 dlls/riched20/txtdoc.h

2 4

[PATCH 1/2] ntdll: Implement NtQuerySystemInformation class 0x37
by Chris Moeller 30 Jul '13

30 Jul '13

2 1

GetNumaHighestNodeNumber tests
by Chris Moeller 29 Jul '13

29 Jul '13

Here is a basic stand-alone test program, which I should probably learn how to implement into the standard testing array. On my test Windows 7 system with a single processor (4 cores), it outputs the following: Testing NtQuerySystemInformationSystemNumaHighestNodeNumber = 0x37, ...): ... with invalid size returned c0000004 ... with valid size returned 0, HighestNodeNumber = 0, ResultLength = 4 Testing GetNumaHighestNodeNumber: ... returned 1, HighestNodeNumber = 0

1 0

[PATCH 2/2] kernel32: Implement GetNumaHighestNodeNumber
by Chris Moeller 29 Jul '13

29 Jul '13

1 0

[PATCH 0/2] Implement basic GetNumaHighestNodeNumber
by Chris Moeller 29 Jul '13

29 Jul '13

This patch against HEAD does the following in two parts: Part 1) Implements NtQuerySystemInformation class 0x37, which I have given the name SystemNumaHighestNodeNumber, since that appears to be all it does. It currently is limited to simulating a single node system, and returns the correct error status if queried with too small a return buffer. Part 2) Implements GetNumaHighestNodeNumber based on the above interface, and behaves just like a Windows 7 32-bit kernel32.dll does in case of errors in that function.

1 0

kernel32: Correct log on / logon (noun / verb)
by Ken Sharp 29 Jul '13

29 Jul '13

Evening Winos, Attached is a patch to correct some winerr messages. Specifically change a noun to a verb, and update the .po files with it. Firstly, could you take a look and point out any obvious errors I may have made? I have marked some of the translations fuzzy because it's not clear if the original translator realised that this should have been the verb and not the noun. A quick look through the translations on Wikipedia suggests that most languages do use different terms for the noun and the verb. https://cs.wikipedia.org/wiki/Login https://de.wikipedia.org/wiki/Login_(Informationstechnik) https://es.wikipedia.org/wiki/Login and so on. Some of these listings seem to use completely different words to the current translation in Wine so I wouldn't dare make a guess. Secondly, what do people think about the use of "Can't" as opposed to "Cannot"? I have not made the change in the patch but I'm wondering if people think that this should be done. Can't is informal and as my old English teacher would say, "Can't is not a word!" It is, but being pedantic "cannot" is probably the better choice. I assume Windows uses "can't" seen as it listed in winerror.mc. Thanks all! Ken

2 1

Re: [1/1] kernel32: Implemented basic NUMA functions to replace the stubs
by Matteo Bruni 29 Jul '13

29 Jul '13

2013/7/30 Chris Moeller <kode54(a)gmail.com>: > > On Jul 29, 2013, at 11:22 AM, Matteo Bruni <matteo.mystral(a)gmail.com> wrote: > >> 2013/7/29 Chris Moeller <kode54(a)gmail.com>: >>> without looking very deeply into the CRT source code. >> >> You're talking about MS source code, right? That makes you tainted for >> contributing to Wine at >> the very least for C runtime-related code :( >> If you haven't looked into any other source code, this patch might be >> okay (barring other issues, I haven't read the patch at all). At least >> http://wiki.winehq.org/DeveloperFaq#head-fed5011434f62ae1a88baebfb8193a37ea… >> seems to suggest so. > > I did not look at any CRT source code to determine how to implement the functionality of this patch. I only looked at CRT source code to determine why the CRT was crashing in the first place. The HighestNode function popping a warning in my wine logs should have been hint enough to discover that function was stubbed. > > I actually determined how these functions worked by testing their behavior against a working system, and also with the aid of another developer, who pointed out a handy Win32 function for retrieving a suitable value for the free memory check. That's probably fine, although as a general rule you should stay as far as possible from Microsoft's source code. Cheers, Matteo.

1 0

[1/1] kernel32: Implemented basic NUMA functions to replace the stubs
by Chris Moeller 29 Jul '13

29 Jul '13

Main changes from HEAD: 1) Implements GetNumaHighestNodeNumber which simulates a typical single node system. 2) Implements GetNumaNodeProcessorMask which returns the global processor affinity mask for node 0, otherwise returns an error. 3) Implements GetNumaAvailableMemoryNode which returns available physical memory for node 0, otherwise returns an error. This much of the behavior was observed based on a single processor (4 cores) Windows 7 system. These functions are utilized by the synchronization and threading primitives incorporated in the MSVC 2012 and newer C and C++ runtime libraries. concrt.h/Concurrency::critical_section, which is used by both std::mutex and std::thread, will throw an exception the moment the object is locked if GetNumaHighestNodeNumber fails, and there are at least a few examples of software which don't bother to catch this exception and thus crash, such as the alpha version of Cube World, and certain versions of the foo_wave_seekbar component for the foobar2000 audio player. The associated scheduling code appears to make use of GetNumaNodeProcessorMask as well to control how many threads are allowed to spin up at once, or so I guess without looking very deeply into the CRT source code.

3 2

Re: [PATCH] Ignore MotionNotify when XI2 is handling mouse motion
by Alexandre Julliard 29 Jul '13

29 Jul '13

Josef Schimke <sgtcapslock(a)lavabit.com> writes: > Hello, > > When XInput2 takes over the mouse, regular MotionNotify events continue to be > handled alongside it. This causes the cursor to erratically skip around when > the slower method (X11 input) tries to reposition the cursor while XInput2 is > already handling future events. We want to process MotionNotify to resync the cursor position, XInput events are not enough. -- Alexandre Julliard julliard(a)winehq.org

1 0

How to get involved in Windows Store API implement?
by 中川祥 29 Jul '13

29 Jul '13

I do not know much about the store apps development. What is Windows.winmd? Do I need disassemble it and create wine version ?If I need C++,I wouldn't do. Is there any planning to support them?

2 1

NtCreateFile fails
by Mislav Blazevic 29 Jul '13

29 Jul '13

First of all, pardon my ignorance, I am new to wine and winapi. I am trying to implement apphelp.dll and I ran into issues with NtCreateFile: It always returns 0xc0000103, STATUS_NOT_A_DIRECTORY, and I am not sure what that means. Here is code: /* This function is not in API, but written for internal use */ HANDLE WINAPI SdbOpenFile( LPCWSTR path, PATH_TYPE type ) { /* SNIP */ if (type == DOS_PATH) if (!RtlDosPathNameToNtPathName_U(path, &str, NULL, NULL)) return NULL; InitializeObjectAttributes(&attr, &str, OBJ_CASE_INSENSITIVE, NULL, NULL); status = NtCreateFile(&file, GENERIC_READ | SYNCHRONIZE | FILE_READ_ATTRIBUTES, &attr, &io, NULL, FILE_ATTRIBUTE_NORMAL, FILE_SHARE_READ, FILE_OPEN, FILE_DIRECTORY_FILE, NULL, 0); /* SNIP */ return file; } Function is called like this: WCHAR szFileName[1024] = {0}; ExpandEnvironmentStringsW(L"%SystemRoot%\\sample.sdb", szFileName, 1024); // for testing purposes SdbOpenFile(szFileName, DOS_PATH); Compiled with winegcc. I am sure that C:\windows\sample.sdb exists. What am I doing wrong?

3 2

Re: [PATCH 3/5] d3dx9: Add support for structs to ID3DXConstantTable.
by Matteo Bruni 28 Jul '13

28 Jul '13

2013/7/24 Rico Schüller <kgbricola(a)web.de>: > --- > dlls/d3dx9_36/shader.c | 79 > ++++++++++++++++++++++++++++++++++++++++++++++++-- > 1 file changed, 77 insertions(+), 2 deletions(-) > So there was actually a logic (an insane one, but yeah...). Have you found any game actually using this? Annoying question: have you checked if it's the same with newer d3dx9 dlls (e.g. d3dx9_43)? It still looks to me like buggy behavior and it might be that MS fixed that in later d3dx9 versions. This is not blocking the patch of course.

2 1

Re: windowscodecs: Workaround libtiff bug when it defines toff_t as 32-bit for 32-bit builds.
by Vincent Povirk 27 Jul '13

27 Jul '13

I have no opinion on this sort of work-around (it should work afaict, and I haven't found any other places where we currently use toff_t), but did you file a bug with whatever distro has such broken headers?

3 3

Windows 7 64-bit?
by Ken Sharp 27 Jul '13

27 Jul '13

I have to ask: Do we really think that this user is running Wine 1.0.1 on Windows 7 64-bit? http://appdb.winehq.org/objectManager.php?sClass=version&iId=28587&iTesting…

3 4

Need help with debugging a directx9 game crashing
by Qian Hong 25 Jul '13

25 Jul '13

Hello, I was debugging on a popular game [1] [2] (9 GB), it crashes on start. +relay,+seh,+tid log show that there are some calls to wined3d.wined3d_mutex_lock / wined3d.wined3d_buffer_map / wined3d.wined3d_mutex_unlock before crashing, so I turn on +d3d9 trace to get a new log in details. To my surprise, the game doesn't crash anymore with +d3d9 trace! After a serials of bisect, I found d3d9_AddRef is the key function call related to the crashing. If I remove the TRACE statement in line 71, the game will crash with +d3d9 as well; if I upgrade the TRACE in 71 to FIXME, the game will not crash even without +d3d9. 66 static ULONG WINAPI d3d9_AddRef(IDirect3D9Ex *iface) 67 { 68 struct d3d9 *d3d9 = impl_from_IDirect3D9Ex(iface); 69 ULONG refcount = InterlockedIncrement(&d3d9->refcount); 70 71 TRACE("%p increasing refcount to %u.\n", iface, refcount); 72 73 return refcount; 74 } Further tests show that the simplest hack to avoid crashing is replacing line 71 to: FIXME("anything %x\n", 0xdeadbeef); I have no idea what the real fix is, any suggestion what is the next step to debug? Thanks in advance! Environment: Wine 1.6-rc5 Ubuntu 12.04.2 LTS gcc 4.6.3 (Ubuntu/Linaro 4.6.3-1ubuntu5) The installer ship with native d3dx9_40.dll Backtrace on crash: Wine-dbg>c Unhandled exception: page fault on read access to 0x00000001 in 32-bit code (0x0649e6e9). Register dump: CS:0073 SS:007b DS:007b ES:007b FS:0033 GS:003b EIP:0649e6e9 ESP:0033cc60 EBP:00000000 EFLAGS:00210206( R- -- I - -P- ) EAX:00000001 EBX:19b871e0 ECX:00000000 EDX:00000000 ESI:23f420b0 EDI:23f4f20c Stack dump: 0x0033cc60: 80004005 19c0e0c0 00000000 19b871e0 0x0033cc70: 00000000 23f420b0 23f4f1f0 19b871e0 0x0033cc80: 00000018 00000040 0000001c 00000000 0x0033cc90: 0a9101df 00000020 00000034 7bc34159 0x0033cca0: 00110064 ffffffff 0033ccd8 7bc34d2d 0x0033ccb0: 00000000 0000000c 00000010 00000001 Backtrace: =>0 0x0649e6e9 in kg3dengine (+0x16e6e9) (0x00000000) 0x0649e6e9: movl 0x0(%eax),%ecx [1] http://jx3.xoyo.com/download [2] http://jx3.client.cdn.kingsoft.com/JXOnline3-v3.0.4.4980/JXOnline3-v3.0.4.4… -- Regards, Qian Hong - http://www.winehq.org

4 11

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string. (try 3)
by Ken Thomases 25 Jul '13

25 Jul '13

On Jul 25, 2013, at 10:49 AM, Charles Davis wrote: > Try 3: Don't advertise WGL_EXT_swap_control if vsync is disabled. Looks good. Thanks, Ken

1 0

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string. (try 2)
by Ken Thomases 25 Jul '13

25 Jul '13

On Jul 24, 2013, at 9:34 PM, Charles Davis wrote: > Try 2: Don't duplicate a constant string; we might need to add more legacy > extensions. (Hopefully not. ;) Thanks for making that change. Looks good to me. Cheers, Ken

1 2

Re: [PATCH 1/6] scrrun: Store reference count inside IFileSystem3 object
by Nikolay Sivov 25 Jul '13

25 Jul '13

On 7/25/2013 16:35, Piotr Caban wrote: > --- > dlls/scrrun/filesystem.c | 163 > +++++++++++++++++++++++++++++++++++------------ > 1 file changed, 123 insertions(+), 40 deletions(-) Why do you need this?

2 1

What to do when an application misuses an API ...
by Michael Casadevall 25 Jul '13

25 Jul '13

Hey all, I've been working on trying to get Sid Meier's Civilization V (Civ5) up to a point where it works relatively flawlessly on WINE. The largest issue is that installing the latest expansion pack has completely hosed the game under WINE unless some rather ugly workarounds have been taken. Specifically, the main issue is that Civ5 loads its data files in the order returned by Find{First,Next}File(), and breaks spectacularly if that order is disrupted. I documented my findings here: http://bugs.winehq.org/show_bug.cgi?id=34122 As I sat to look at a possible patch, I'm not sure what I can actually do. Its clearly stated on the MSDN page that the order returned is dependent on the underlying filesystem, and Googling suggests that while NTFS seems to always return in a roughly alphabetical order, FAT32 will return by creation date, and with network drives all bets are off. I was even able to reproduce similar bugs in Civ5 by launching it from a network drive (which I had already tested in advance to make sure it returned an odd file order). FindFirstFile essentially calls down to NtQueryDirectoryFile, which in turn either calls a system specific syscall or readdir to get a list of files in a directory. My first thought is to extend the underlying read_directory_* functions in ntdll/directory.c so that they return an alphabetized list (essentially taking the results from getdents/getdirentries/readdir, sort them, then returning in the sort order whichever entry is next in line). This would produce a behavior similar (but not identical) to what is seen when operating under an NTFS filesystem. This approach seems wrong to me, and is further compounded by the fact this is a bug in an application, and not in WINE specifically; Under the correct circumstances, Windows will return a "non-sorted" list of files and in those cases Civilization V breaks. The flipside is that in the most common case, it works correctly, and furthermore, given how easy it is to assume that Find*File does return a sorted list, I won't be surprised if this issue has cropped up elsewhere (figuring out the underlying cause was exceedingly annoying). Any insight in to what the most correct course of action would be most appreciated I'm perfectly willing to put the legwork in fixing this bug, but I want to make sure I'm fixing it in a way that will be accepted :-). Regards, Michael

3 3

Re: [PATCH] winemac.drv: Also return the number of pixel formats if the caller didn't supply a PIXELFORMATDESCRIPTOR.
by Ken Thomases 24 Jul '13

24 Jul '13

On Jul 23, 2013, at 10:26 PM, Charles Davis wrote: > This is akin to what glxdrv already does, and apparently what Windows > does. It allows the GLView sample from the OpenGL SuperBible to > correctly enumerate pixel formats with the Mac driver. > --- > dlls/winemac.drv/opengl.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/winemac.drv/opengl.c b/dlls/winemac.drv/opengl.c > index 0922371..fb294d1 100644 > --- a/dlls/winemac.drv/opengl.c > +++ b/dlls/winemac.drv/opengl.c > @@ -3281,7 +3281,7 @@ int macdrv_wglDescribePixelFormat(HDC hdc, int fmt, UINT size, PIXELFORMATDESCRI > > TRACE("hdc %p fmt %d size %u descr %p\n", hdc, fmt, size, descr); > > - if (fmt <= 0 || fmt > ret) return ret; > + if (fmt <= 0 || fmt > ret || !descr) return ret; > if (size < sizeof(*descr)) return 0; > > pf = &pixel_formats[fmt - 1]; > -- > 1.7.5.4 Looks good to me. Thanks! -Ken

1 0

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string.
by Ken Thomases 24 Jul '13

24 Jul '13

Hi, On Jul 23, 2013, at 10:26 PM, Charles Davis wrote: > @@ -1226,8 +1227,10 @@ static BOOL init_gl_info(void) > gl_info.glVersion = HeapAlloc(GetProcessHeap(), 0, strlen(str) + 1); > strcpy(gl_info.glVersion, str); > str = (const char*)opengl_funcs.gl.p_glGetString(GL_EXTENSIONS); > - gl_info.glExtensions = HeapAlloc(GetProcessHeap(), 0, strlen(str) + 1); > + gl_info.glExtensions = HeapAlloc(GetProcessHeap(), 0, strlen(str) + > + sizeof(" WGL_EXT_extensions_string WGL_EXT_swap_control")); > strcpy(gl_info.glExtensions, str); > + strcat(gl_info.glExtensions, " WGL_EXT_extensions_string WGL_EXT_swap_control"); As a minor nitpick, I would prefer to avoid that duplicated string literal. Otherwise, looks good to me. Cheers, Ken

1 0

Re: [PATCH 4/5] scrrun: Add IFileSystem3::GetTempName implementation
by Alexandre Julliard 24 Jul '13

24 Jul '13

Piotr Caban <piotr(a)codeweavers.com> writes: > @@ -174,6 +174,7 @@ BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, LPVOID reserved ) > case DLL_PROCESS_ATTACH: > DisableThreadLibraryCalls( hinst ); > scrrun_instance = hinst; > + srand(GetTickCount()); You don't want to do that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: po: Update English (US) resource
by Francois Gouget 24 Jul '13

24 Jul '13

On Mon, 22 Jul 2013, Ken Sharp wrote: > Logon/Log on as with the British/neutral patch. > > msgid "Can't logon with inter-domain trust account.\n" > -msgstr "Can't logon with inter-domain trust account.\n" > +msgstr "Can't log on with inter-domain trust account.\n" [...] I think these fixes should really be applied to the source winerror.mc file instead. If I remember correctly en_US.po is just a dummy translation where msgid and msgstr strings should be identical (there might (with maybe one or two exceptions for things like um -> µm). -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Be careful of reading health books, you might die of a misprint. -- Mark Twain

2 1

Re: [PATCH 1/3] msvcrt: Fix _mktemp implementation
by Marvin 24 Jul '13

24 Jul '13

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=26413 Your paranoid android. === W2KPROSP4 (32 bit file) === file: unhandled exception c0000005 at 7801B5A7 === WXPPROSP2 (32 bit file) === file: unhandled exception c0000005 at 77C2F1C3 === W2K3R2SESP2 (32 bit file) === file: unhandled exception c0000005 at 77BC00A3 === WXPX64 (64 bit file) === Timeout

1 0

Re: [PATCH 1/3] d3dx9: Fix initialization of D3DXPC_OBJECTs.
by Rico Schüller 23 Jul '13

23 Jul '13

On 23.07.2013 11:46, Rico Schüller wrote: > --- > dlls/d3dx9_36/effect.c | 22 +++++++++++++++++++--- > 1 file changed, 19 insertions(+), 3 deletions(-) > > > > > These may not apply correctly. Sorry for the noise. I'll resend a new version. Cheers Rico

1 0

Re: [docs] "sgml" is not a valid output format
by Francois Gouget 23 Jul '13

23 Jul '13

On Tue, 23 Jul 2013, Frédéric Delanoy wrote: [...] > -ALLFORMATS = html pdf sgml ps txt > +ALLFORMATS = html pdf ps txt 'make sgml' can be used to generate the translated SGML source files for our many documentation translations. So I don't think this patch should be applied. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ The last time religion ruled, it was called the dark ages.

1 0

Re: [docs] "sgml" is not a valid output format
by Frédéric Delanoy 23 Jul '13

23 Jul '13

On Tue, Jul 23, 2013 at 5:26 PM, Frédéric Delanoy <frederic.delanoy(a)gmail.com> wrote: > --- > Make.rules.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Make.rules.in b/Make.rules.in > index fda8e81..d326247 100644 > --- a/Make.rules.in > +++ b/Make.rules.in > @@ -100,7 +100,7 @@ dummy: > > # Manual Dependencies > > -ALLFORMATS = html pdf sgml ps txt > +ALLFORMATS = html pdf ps txt Please discard this patch ; it seems to be used in the translated guides to generate sgml from english version+po file

1 0

Re: [1/5] oledb32: Stub for IRowPosition
by Marvin 23 Jul '13

23 Jul '13

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=26406 Your paranoid android. === WINEBUILD (build) === Make failed

2 1

Re: [PATCH 1/2] wined3d: Implement special fog behavior for orthogonal projection matrices.
by Henri Verbeet 23 Jul '13

23 Jul '13

On 22 July 2013 20:24, Matteo Bruni <mbruni(a)codeweavers.com> wrote: > @@ -3614,10 +3614,19 @@ void wined3d_ffp_get_vs_settings(const struct wined3d_state *state, const struct > & WINED3D_FFP_LIGHT_TYPE_MASK) << WINED3D_FFP_LIGHT_TYPE_SHIFT(i); > } > > + settings->ortho_fog = 0; > if (!state->render_states[WINED3D_RS_FOGENABLE]) > settings->fog_mode = WINED3D_FFP_VS_FOG_OFF; > else if (state->render_states[WINED3D_RS_FOGTABLEMODE] != WINED3D_FOG_NONE) > + { > settings->fog_mode = WINED3D_FFP_VS_FOG_DEPTH; > + > + if (state->transforms[WINED3D_TS_PROJECTION].u.m[0][3] == 0.0f > + && state->transforms[WINED3D_TS_PROJECTION].u.m[1][3] == 0.0f > + && state->transforms[WINED3D_TS_PROJECTION].u.m[2][3] == 0.0f > + && state->transforms[WINED3D_TS_PROJECTION].u.m[3][3] == 1.0f) > + settings->ortho_fog = 1; > + } I'm going to guess "No, because the projection matrix is ignored.", but does this "feature" apply to transformed vertices as well?

1 0

Re: [PATCH 2/2] include/ddk: add usbioctl.h
by Nikolay Sivov 22 Jul '13

22 Jul '13

On 7/22/2013 22:38, Damjan Jovanovic wrote: > Changelog: > * add usbioctl.h > > Damjan Jovanovic Hi, Damjan. You forgot patches.

3 2

Re: appwiz: Correct Wine Mono installer messages
by Alexandre Julliard 22 Jul '13

22 Jul '13

Ken Sharp <kennybobs(a)o2.co.uk> writes: > Where the correction is obvious I have done so, but where a full > translation is needed I have simply removed the incorrect one. This > will mark that line as untranslated and hopefully someone will see > that. It will also default to English with the correct message until > translated. That's what fuzzy is for, there's no reason to remove them. -- Alexandre Julliard julliard(a)winehq.org

3 3

Wine 1.6 list of deferred patches
by Bruno Jesus 22 Jul '13

22 Jul '13

This is the list of (hopefully) all deferred patches for wine 1.6, all authors are in bcc. I hope it's useful for developers. 97187 Deferred Dmitry Timoshkov [3/3] kernel32: GetShortPathName for a non-existent short file name should fail. 97131 Deferred Rosen Diankov [ole32] Bugfix for implicit MTA apartment initialization 97130 Deferred Rosen Diankov [oleaut32] Bugfix patches for marshaling VT_UNKNOWN in SAFEARRAYS 97124 Deferred Maarten Lankhorst [RFC PATCH] loader: allow overriding pthread_create correctly on linux 97119 Deferred Fabian Ebner [PATCH] dinput: Check the message queue in GetDeviceState * 97085 Deferred Dmitry Timoshkov ntdll: NtAllocateVirtualMemory should fail to commit if an address range is already committed for a memory mapped file. 96974 Deferred Stefan Dösinger [PATCH 2/2] wined3d: Avoid calculating 1 / (fog_end - fog_start) in the shader 96961 Deferred Mathis Beer ntdll: make msync() call asynchronous in NtFlushVirtualMemory (try 3.5, alternate version) 96925 Deferred Hans Leidekker server: The token user SID must be present in the default DACL. 96899 Deferred Stefan Dösinger [PATCH 1/5] wined3d: Consolidate d3d_info BOOLs into a flags field 96855 Deferred Alistair Leslie-Hughes oledb32: Add IErrorInfo Support OK 96854 Deferred Alistair Leslie-Hughes msdaps: Add support for ISourcesRowset (try 2) 96831 Deferred Christian Costa [PATCH] d3dxof: Remove LP stuff and rename variables for IDirectXFile methods. 96819 Deferred Nikolay Sivov comctl32: Properly handle repainting for HDF_OWNERDRAW items 96797 Deferred André Hentschel ntdll: Add stub for LdrResolveDelayLoadedAPI and reference it in kernel32 (try 2) 96793 Deferred Christian Costa [PATCH] d3dx9_36: Implement D3DXGetShaderInputSemantics + tests. 96791 Deferred morphiend msvcrt: add support for _chsize_s 96765 Deferred Christian Costa [PATCH 2/4] d3dxof: Handle float in binary mode and remove code that becomes useless. * this patch was already commited.

1 0

Re: [PATCH 3/4] d3dx9: Improve ID3DXConstantTable::Set*().
by Alexandre Julliard 22 Jul '13

22 Jul '13

Rico Schüller <kgbricola(a)web.de> writes: > +static UINT set(struct ID3DXConstantTableImpl *table, IDirect3DDevice9 *device, struct ctab_constant *constant, > + const void **indata, D3DXPARAMETER_TYPE intype, UINT *size, UINT incol, D3DXPARAMETER_CLASS inclass, UINT index, > + BOOL is_pointer) > +{ > + DWORD (*get_index)(const void **indata, UINT index) = NULL; > + D3DXCONSTANT_DESC *desc = &constant->desc; > + UINT l, i, regcount = 1, regsize = 1, cin = 1, rin = 1, ret, last = 0; > + DWORD tmp; > + > + if (is_pointer) get_index = &get_index_pointer; > + else get_index = &get_index_array; It would be cleaner and easier to follow to have a simple helper with a is_pointer flag, rather than using function pointers with such trivial functions. -- Alexandre Julliard julliard(a)winehq.org

1 0

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-devel