- wine-devel - winehq.org

Re: [PATCH 5/5] d3dcompiler: Implement basic expressions parsing.
by Stefan Dösinger 17 Jul '12

17 Jul '12

Am Montag, 16. Juli 2012, 20:39:23 schrieb Matteo Bruni: > + HLSL_IR_BINOP_MUL, > ... > + HLSL_IR_BINOP_DOT, HLSL has 3 somewhat related multiplication operations: "*", "mul" and "dot". Dot is fairly straightforward, but "*" and "mul" have subtle differences. I recommend to write tests to find out the exact details, but as far as I remember "float4 * float4" gives you a float4 with a component-wise multiplication, whereas mul(float, float) returns a scalar like the dot product. "float4x4 * float4" is an error, but mul(float4x4, float4) performs a matrix-vector multiplication and returns a float4. > +static struct hlsl_type *expr_common_type(struct hlsl_type *t1, struct > hlsl_type *t2, > + struct source_location *loc) This type inference system might be a bit too limited, you have to take the operation into account. e.g. a dot(float3, float4) returns a scalar and crs(float4, float4) returns a float3.

2 1

Re: riched20: Avoid uninitialized field in STGMEDIUM structure (LLVM/Clang).
by Alexandre Julliard 17 Jul '12

17 Jul '12

Nicolas Le Cam <niko.lecam(a)gmail.com> writes: > @@ -1098,6 +1098,8 @@ static BOOL ME_RTFInsertOleObject(RTF_Info *info, HENHMETAFILE hemf, HBITMAP hbm > BOOL ret = FALSE; > DWORD conn; > > + memset(&stgm, 0, sizeof(stgm)); > + Please fix the bugs (if any) instead of silencing the warnings by memsetting everything to 0. -- Alexandre Julliard julliard(a)winehq.org

1 0

re: msvcp90: handle npos as a len in basic_string<>::replace
by Dan Kegel 16 Jul '12

16 Jul '12

Daniel Lehman wrote: @@ -1949,7 +1949,7 @@ basic_string_char* __thiscall basic_string_char_replace_cstr_len(basic_string_ch - if(off+len > this->size) + if(off+len < off || off+len > this->size) len = this->size-off; Wouldn't this be more elegant: if(num > this->size-pos) num = this->size-pos; ? Also, you may have missed the similar case in basic_string_char_compare_substr_cstr_len (which uses 'pos' and 'num' rather than 'off' and 'len'). See http://bugs.winehq.org/show_bug.cgi?id=31085#c5

2 1

Build failure of "[PATCH 5/5] kernel32/tests: Add a IOCTL_DVD_READ_STRUCTURE (DvdManufacturerDescriptor) test (try 4)" ?
by GOUJON Alexandre 16 Jul '12

16 Jul '12

Hi, My patch (http://source.winehq.org/patches/data/88072) is marked as "Build failure". I tried with the latest (today) HEAD and I successfully applied it. Compilation also works for me at least on my x86 and x86_64 computers. Moreover, the Wine Test Bot tested it without any issue, see https://testbot.winehq.org/JobDetails.pl?Key=19936. I know your machine is a bit "mystic". Can you show me your output to help me diagnosing your build failure ? Thanks

2 2

Re: msi/tests: Mark some test results as broken.
by Marvin 16 Jul '12

16 Jul '12

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=20071 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:6053: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon)

1 0

Re: kernel32: Also report empty drives as available in GetLogicalDrives
by Alexandre Julliard 16 Jul '12

16 Jul '12

Alexandre Goujon <ale.goujon(a)gmail.com> writes: > --- > dlls/kernel32/volume.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/dlls/kernel32/volume.c b/dlls/kernel32/volume.c > index 8377e41..67e2eb1 100644 > --- a/dlls/kernel32/volume.c > +++ b/dlls/kernel32/volume.c > @@ -1441,19 +1441,26 @@ DWORD WINAPI GetLogicalDrives(void) > DWORD ret = 0; > int i; > > - if (!(buffer = HeapAlloc( GetProcessHeap(), 0, strlen(config_dir) + sizeof("/dosdevices/a:") ))) > + if (!(buffer = HeapAlloc( GetProcessHeap(), 0, strlen(config_dir) + sizeof("/dosdevices/a::") ))) > { > SetLastError( ERROR_NOT_ENOUGH_MEMORY ); > return 0; > } > strcpy( buffer, config_dir ); > - strcat( buffer, "/dosdevices/a:" ); > - dev = buffer + strlen(buffer) - 2; > + strcat( buffer, "/dosdevices/a::" ); > + dev = buffer + strlen(buffer) - 3; > > for (i = 0; i < 26; i++) > { > *dev = 'a' + i; > + dev[2] = 0; > if (!stat( buffer, &st )) ret |= (1 << i); > + /* Windows also report empty drives as available */ > + else > + { > + dev[2] = ':'; > + if (!stat( buffer, &st )) ret |= (1 << i); > + } You can't do that, the device symlinks don't reflect the availability of the drive. It would have to go through the mountmgr. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/9] hhctrl.ocx: Centralize merging HH_WINTYPE data (resend).
by Alexandre Julliard 16 Jul '12

16 Jul '12

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > @@ -28,6 +28,13 @@ > > WINE_DEFAULT_DEBUG_CHANNEL(htmlhelp); > > +#define HHWIN_PARAM_MASK (HHWIN_PARAM_PROPERTIES|HHWIN_PARAM_STYLES|HHWIN_PARAM_EXSTYLES|HHWIN_PARAM_RECT\ > + |HHWIN_PARAM_NAV_WIDTH|HHWIN_PARAM_SHOWSTATE|HHWIN_PARAM_INFOTYPES\ > + |HHWIN_PARAM_TB_FLAGS|HHWIN_PARAM_EXPANSION|HHWIN_PARAM_TABPOS\ > + |HHWIN_PARAM_TABORDER|HHWIN_PARAM_HISTORY_COUNT|HHWIN_PARAM_CUR_TAB) > +#define MERGE_PARAM(s,d,o,p) ((s->fsValidMembers & p) && (!(d->fsValidMembers & p) || o)) > +#define MERGE_PROP(s,d,o,p) (s->p != 0 && (d->p == 0 || o)) These macros are fairly ugly, you should be able to do without that. > @@ -1752,16 +1752,16 @@ void ReleaseHelpViewer(HHInfo *info) > list_remove(&info->entry); > > /* Free allocated strings */ > - heap_free(info->pszType); > - heap_free(info->pszCaption); > - heap_free(info->pszToc); > - heap_free(info->pszIndex); > - heap_free(info->pszFile); > - heap_free(info->pszHome); > - heap_free(info->pszJump1); > - heap_free(info->pszJump2); > - heap_free(info->pszUrlJump1); > - heap_free(info->pszUrlJump2); > + heap_free((LPVOID)info->WinType.pszType); > + heap_free((LPVOID)info->WinType.pszCaption); > + heap_free((LPVOID)info->WinType.pszToc); > + heap_free((LPVOID)info->WinType.pszIndex); > + heap_free((LPVOID)info->WinType.pszFile); > + heap_free((LPVOID)info->WinType.pszHome); > + heap_free((LPVOID)info->WinType.pszJump1); > + heap_free((LPVOID)info->WinType.pszJump2); > + heap_free((LPVOID)info->WinType.pszUrlJump1); > + heap_free((LPVOID)info->WinType.pszUrlJump2); Don't do that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 6/6] jscript: Don't set constructor property to each object instance, it belongs to their prototypes
by Marvin 16 Jul '12

16 Jul '12

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=20070 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 5/6] jscript: Expose RegExpError constructor in global object
by Marvin 16 Jul '12

16 Jul '12

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=20069 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 4/6] jscript: Properly set Error object's constructor property
by Marvin 16 Jul '12

16 Jul '12

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=20068 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 3/6] jscript: Properly set Function constructor's constructor property
by Marvin 16 Jul '12

16 Jul '12

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=20067 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 2/6] jscript: Properly handle constructor property for most builtin constructors
by Marvin 16 Jul '12

16 Jul '12

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=20066 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 1/6] jscript: Properly set Number.prototyp.constructor
by Marvin 16 Jul '12

16 Jul '12

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=20065 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

mshtml: Always set the URL policy in the error cases?
by Francois Gouget 16 Jul '12

16 Jul '12

gcc 4.7 is complaining that policy may be used uninitialized in InternetHostSecurityManager_QueryCustomPolicy() and indeed there are many error cases in confirm_safety_load() and confirm_safety() where the policy is not set. The patch belowe seems to plug all these holes but I don't know if it really makes sense. diff --git a/dlls/mshtml/secmgr.c b/dlls/mshtml/secmgr.c index ff349ac..3f64a42 100644 --- a/dlls/mshtml/secmgr.c +++ b/dlls/mshtml/secmgr.c @@ -104,10 +104,9 @@ static HRESULT confirm_safety_load(HTMLDocumentNode *This, struct CONFIRMSAFETY CATID init_catid = CATID_SafeForInitializing; hres = ICatInformation_IsClassOfCategories(This->catmgr, &cs->clsid, 1, &init_catid, 0, NULL); + *ret = hres == S_OK ? URLPOLICY_ALLOW : URLPOLICY_DISALLOW; if(FAILED(hres)) return hres; - - *ret = hres == S_OK ? URLPOLICY_ALLOW : URLPOLICY_DISALLOW; } return S_OK; @@ -157,13 +156,17 @@ static HRESULT confirm_safety(HTMLDocumentNode *This, const WCHAR *url, struct C if(!This->catmgr) { hres = CoCreateInstance(&CLSID_StdComponentCategoriesMgr, NULL, CLSCTX_INPROC_SERVER, &IID_ICatInformation, (void**)&This->catmgr); - if(FAILED(hres)) + if(FAILED(hres)) { + *ret = URLPOLICY_DISALLOW; return hres; + } } hres = ICatInformation_IsClassOfCategories(This->catmgr, &cs->clsid, 1, &scripting_catid, 0, NULL); - if(FAILED(hres)) + if(FAILED(hres)) { + *ret = URLPOLICY_DISALLOW; return hres; + } if(hres != S_OK) { *ret = URLPOLICY_DISALLOW; -- 1.7.10

2 3

Re: [1/9] gdiplus: Make sure that GdipBitmapLockBits stores valid lockmode.
by Vincent Povirk 16 Jul '12

16 Jul '12

I think this is a correct change, even though it looks wrong. If you use some invalid flag and/or add a comment, that may help explain what you're doing.

1 0

Re: [1/2] gdiplus: Make sure that 1bpp bitmap images get loaded in native format.
by Vincent Povirk 16 Jul '12

16 Jul '12

Uh, this is going to cause regressions if you don't give the image a palette.

2 2

Re: [4/4] gdiplus: GdipMeasureString operates internally in pixels but in/out rectangles are in device units. Resend.
by Vincent Povirk 16 Jul '12

16 Jul '12

Values returned from Measure[String|CharacterRanges] have to match what DrawString will draw. If they don't currently match, you need to explain why that is. If they do, you need to bring DrawString into line as well.

1 0

re: Query MediaType of SD-Cards using IOCTL_GET_DISK_DRIVE_GEOMETRY
by Dan Kegel 15 Jul '12

15 Jul '12

> Is it worth for working on patches for these issues, or is this > completely impossible with the current architecture? Do you know yet why no symlink is made in dosdevices for /media/XXXX-YYYY? I haven't heard any rumblings about the current architecture being hopelessly broken.

2 1

Re: dsound: Add detection of output format and allow a maximum of 8 channels
by Andrew Eikum 14 Jul '12

14 Jul '12

Thanks, Donny. Nice work overall. Some comments below. On Fri, Jul 13, 2012 at 05:57:48PM +1000, Donny Yang wrote: > This patch makes dsound automatically get the output format when a > output device is initialised and also allows up to 8 output channels > to be used for ALSA and PulseAudio. Using ALSA with over 2 channels > outputs the channels in the wrong speakers with my testing but I don't > know why so I can't fix that. You probably already discovered that ALSA's multi-channel support is primitive at best. You have to test each channel and hand-craft an asoundrc to match your hardware. See some more information here: <http://drona.csa.iisc.ernet.in/~uday/alsamch.shtml> > @@ -1492,6 +1493,24 @@ HRESULT DirectSoundDevice_Initialize(DirectSoundDevice ** ppDevice, LPCGUID lpcG > } else > WARN("DSOUND_PrimaryCreate failed: %08x\n", hr); > > + hr = IAudioClient_GetMixFormat(device->client, &pwfx); > + if(FAILED(hr)){ > + WARN("IAudioClient_GetMixFormat failed: %08x; Falling back to default output format\n", hr); > + }else{ > + DWORD oldPriolevel = device->priolevel; > + device->priolevel = DSSCL_WRITEPRIMARY; > + hr = primarybuffer_SetFormat(device, pwfx); > + device->priolevel = oldPriolevel; This looks like a hack. I think it fits better in DirectSoundDevice_Create, where the default values are copied in. Is there a reason it couldn't be done there? > -void put_mono2stereo(const IDirectSoundBufferImpl *dsb, DWORD pos, DWORD channel, float value) > +void put_mono(const IDirectSoundBufferImpl *dsb, DWORD pos, DWORD channel, float value) > { > - dsb->put_aux(dsb, pos, 0, value); > - dsb->put_aux(dsb, pos, 1, value); > + /* XXX: Is this how Windows does this? */ > + DWORD c; > + for (c = 0; c < dsb->device->pwfx->nChannels; ++c) > + dsb->put_aux(dsb, pos, c, value); > } > I think this is fine. I wonder if we should skip the LOWFREQ channel, though we'd have to be careful not to affect performance too much, as this runs in a really tight loop. > void mixieee32(float *src, float *dst, unsigned samples) > diff --git a/dlls/dsound/dsound_main.c b/dlls/dsound/dsound_main.c > index 072b25a..538534d 100644 > --- a/dlls/dsound/dsound_main.c > +++ b/dlls/dsound/dsound_main.c > @@ -642,7 +642,7 @@ DirectSoundCaptureEnumerateW( > */ > > typedef HRESULT (*FnCreateInstance)(REFIID riid, LPVOID *ppobj); > - > + > typedef struct { > IClassFactory IClassFactory_iface; > REFCLSID rclsid; > @@ -702,7 +702,7 @@ static HRESULT WINAPI DSCF_CreateInstance( > *ppobj = NULL; > return This->pfnCreateInstance(riid, ppobj); > } > - > + > static HRESULT WINAPI DSCF_LockServer(LPCLASSFACTORY iface, BOOL dolock) > { > IClassFactoryImpl *This = impl_from_IClassFactory(iface); The only changes to this file are whitespace changes. I'd exclude these changes when you resend. > diff --git a/dlls/dsound/dsound_private.h b/dlls/dsound/dsound_private.h > index 1cf6daa..68f6242 100644 > --- a/dlls/dsound/dsound_private.h > +++ b/dlls/dsound/dsound_private.h > @@ -202,7 +202,7 @@ struct IDirectSoundBufferImpl > }; > > float get_mono(const IDirectSoundBufferImpl *dsb, DWORD pos, DWORD channel) DECLSPEC_HIDDEN; > -void put_mono2stereo(const IDirectSoundBufferImpl *dsb, DWORD pos, DWORD channel, float value) DECLSPEC_HIDDEN; > +void put_mono(const IDirectSoundBufferImpl *dsb, DWORD pos, DWORD channel, float value) DECLSPEC_HIDDEN; > > HRESULT IDirectSoundBufferImpl_Create( > DirectSoundDevice *device, > @@ -288,7 +288,7 @@ HRESULT primarybuffer_SetFormat(DirectSoundDevice *device, LPCWAVEFORMATEX wfex) > LONG capped_refcount_dec(LONG *ref) DECLSPEC_HIDDEN; > > /* duplex.c */ > - > + > HRESULT DSOUND_FullDuplexCreate(REFIID riid, LPDIRECTSOUNDFULLDUPLEX* ppDSFD) DECLSPEC_HIDDEN; > > /* mixer.c */ > @@ -305,7 +305,7 @@ DWORD CALLBACK DSOUND_mixthread(void *ptr) DECLSPEC_HIDDEN; > void DSOUND_Calc3DBuffer(IDirectSoundBufferImpl *dsb) DECLSPEC_HIDDEN; > > /* capture.c */ > - > + > HRESULT DSOUND_CaptureCreate(REFIID riid, LPDIRECTSOUNDCAPTURE *ppDSC) DECLSPEC_HIDDEN; > HRESULT DSOUND_CaptureCreate8(REFIID riid, LPDIRECTSOUNDCAPTURE8 *ppDSC8) DECLSPEC_HIDDEN; > Likewise, more unneeded whitespace changes. > diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c > index 93e2b8d..fa6b05a 100644 > --- a/dlls/winealsa.drv/mmdevdrv.c > +++ b/dlls/winealsa.drv/mmdevdrv.c > @@ -1544,7 +1544,7 @@ static HRESULT WINAPI AudioClient_IsFormatSupported(IAudioClient *iface, > goto exit; > } > if(max > 8) > - max = 2; > + max = 8; > if(fmt->nChannels > max){ > hr = S_FALSE; > closest->nChannels = max; > @@ -1649,8 +1649,8 @@ static HRESULT WINAPI AudioClient_GetMixFormat(IAudioClient *iface, > goto exit; > } > > - if(max_channels > 2) > - fmt->Format.nChannels = 2; > + if(max_channels > 8) > + fmt->Format.nChannels = 8; > else > fmt->Format.nChannels = max_channels; > This needs to be a separate patch. Also, winealsa's multi-channel handling needs a closer examination. My current thinking is GetMixFormat should just return some sane, supported default (2ch, 48kHz, 16bps, or less as supported). It seems impossible to detect what the ALSA device actually supports, so we probably shouldn't even try like we do now. > diff --git a/dlls/winepulse.drv/mmdevdrv.c b/dlls/winepulse.drv/mmdevdrv.c > index b374b53..e86ed08 100644 > --- a/dlls/winepulse.drv/mmdevdrv.c > +++ b/dlls/winepulse.drv/mmdevdrv.c > @@ -1050,13 +1050,13 @@ static HRESULT pulse_spec_from_waveformat(ACImpl *This, const WAVEFORMATEX *fmt) > This->ss.format = PA_SAMPLE_INVALID; > switch(fmt->wFormatTag) { > case WAVE_FORMAT_IEEE_FLOAT: > - if (!fmt->nChannels || fmt->nChannels > 2 || fmt->wBitsPerSample != 32) > + if (!fmt->nChannels || fmt->nChannels > 8 || fmt->wBitsPerSample != 32) > break; > This->ss.format = PA_SAMPLE_FLOAT32LE; > pa_channel_map_init_auto(&This->map, fmt->nChannels, PA_CHANNEL_MAP_ALSA); > break; > case WAVE_FORMAT_PCM: > - if (!fmt->nChannels || fmt->nChannels > 2) > + if (!fmt->nChannels || fmt->nChannels > 8) > break; > if (fmt->wBitsPerSample == 8) > This->ss.format = PA_SAMPLE_U8; Wine doesn't have a PulseAudio driver yet, so this chunk won't apply. Andrew

2 2

GetDriveType and empty CD/DVD-ROM drives
by GOUJON Alexandre 14 Jul '12

14 Jul '12

Hi, I'm currently working on bug #17037. The root cause is that GetDriveType("D:\") (if D is mapped to /dev/cdrom) returns DRIVE_NO_ROOT_DIR while on Windows it returns DRIVE_CDROM when there is no disc in the D drive. GetDriveType tries to open D:\ which is converted into \\??\\D:\\ then ~/.wine/dosdevices/d:/ The fact is that there is a dosdevices/d:: file (symlinked to /dev/cdrom) but no dosdevices/d: except if a CD or DVD is in and mounted somewhere. The conversion \\??\\D:\\ -> ~/.wine/dosdevices/d:/ is done in wine_nt_to_unix_file_name (ntdll/directory.c, line 2897). I'm asking here because I wonder if modifying wine_nt_to_unix_file_name is the right thing to do and sane: it's called at several locations and I don't want to cause trouble and regressions, well, if possible. Thanks for any advice

1 0

Suppressing My Computer in File Open dialog box?
by Dan Kegel 14 Jul '12

14 Jul '12

I'm creating a page of tips at http://wiki.winehq.org/DisplayingUnixFilenames for vendors shipping Windows apps on Linux with Wine who want their apps to use Windows paths throughout internally, but display Linux paths in the GUI This is something I've been meaning to do ever since I helped port Picasa. Picasa was an easy case, since it didn't use any standard widgets. So now I'm wondering how to coax standard widgets into not leaking Windows paths or naming conventions visually. Does anyone know how to keep the Save File dialog box from listing My Computer in the LookIn pulldown? (I know how to keep it from showing /, but I'd like to do the opposite, and *only* show / and maybe the user's home directory, if possible.)

2 2

Re: dsound: Add support for adjusting volume of more than 2 channels
by Andrew Eikum 13 Jul '12

13 Jul '12

On Fri, Jul 13, 2012 at 05:59:03PM +1000, Donny Yang wrote: > - if (channels != 1 && channels != 2) > - { > - FIXME("There is no support for %u channels\n", channels); > + if (dsb->device->pwfx->wFormatTag == WAVE_FORMAT_EXTENSIBLE) > + channelMask = ((WAVEFORMATEXTENSIBLE*)dsb->device->pwfx)->dwChannelMask; > + else if (channels == 1) > + channelMask = SPEAKER_FRONT_CENTER; > + else if (channels == 2) > + channelMask = SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT; > + else { > + FIXME("There is no support for %u channels without an extensible wave format\n", channels); > return; > } Before you resend, please consider testing how Windows behaves in this condition. Does it actually refuse to apply any volume at all? Andrew

1 0

Re: [PATCH 9/9] mshtml: Added test of calling IDispatchEx properties
by Marvin 13 Jul '12

13 Jul '12

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=19995 Your paranoid android. === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: Comdlg32: Select initial text in edit control (try 2)
by Dmitry Timoshkov 13 Jul '12

13 Jul '12

Suresh Murty <murtys.wine(a)gmail.com> wrote: > Sends a EM_SETSEL message after the initial text has been set. This is wrong place for both a test and a fix. The text should be selected on focus by a dialog control itself. -- Dmitry.

1 0

Re: Comdlg32: Select initial text in edit control (try 2)
by Marvin 13 Jul '12

13 Jul '12

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=19988 Your paranoid android. === W7PROX64 (32 bit filedlg) === filedlg: unhandled exception c0000005 at 75B94C19

1 0

Re: joy.cpl: Correct joystick testing thread behavior
by Alexandre Julliard 12 Jul '12

12 Jul '12

Lucas Zawacki <lfzawacki(a)gmail.com> writes: > @@ -419,14 +421,19 @@ static INT_PTR CALLBACK test_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM > SendDlgItemMessageW(hwnd, IDC_TESTSELECTCOMBO, CB_SETCURSEL, 0, 0); > test_handle_joychange(hwnd, data); > > - thread = CreateThread(NULL, 0, input_thread, (void*) data, 0, &tid); > + ResumeThread(thread); > } > } > break; > > + case PSN_KILLACTIVE: > + SuspendThread(thread); > + break; You never want to use SuspendThread, it can't possibly work reliably. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/2] scrrun: Added IDispatch support for IFileSystem3
by Alistair Leslie-Hughes 12 Jul '12

12 Jul '12

On 12/07/2012 5:02 PM, Nikolay Sivov wrote: > For http://bugs.winehq.org/show_bug.cgi?id=9638 and similar cases > I have already submitted a patch for this http://source.winehq.org/patches/data/87925 Best Regards Alistair Leslie-Hughes

2 2

Re: [PATCH 4/5] winex11: Release memory on xrandr cleanup.
by Alexandre Julliard 12 Jul '12

12 Jul '12

Henri Verbeet <hverbeet(a)codeweavers.com> writes: > diff --git a/dlls/winex11.drv/xrandr.c b/dlls/winex11.drv/xrandr.c > index 83137a4..a8dccd4 100644 > --- a/dlls/winex11.drv/xrandr.c > +++ b/dlls/winex11.drv/xrandr.c > @@ -289,6 +289,12 @@ done: > wine_tsx11_unlock(); > } > > +void X11DRV_XRandR_Cleanup(void) > +{ > + HeapFree( GetProcessHeap(), 0, real_xrandr_rates_count ); > + HeapFree( GetProcessHeap(), 0, real_xrandr_rates ); > +} It's not worth adding complexity to free memory, in practice the graphics driver is never going to be unloaded except at process exit. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: msi: fix incorrect decoding of msi stream-names
by Hans Leidekker 12 Jul '12

12 Jul '12

On Wed, 2012-07-11 at 15:03 +0000, Oleg Jakobsen wrote: > @@ -203,6 +203,7 @@ BOOL decode_streamname(LPCWSTR in, LPWSTR out) > *out++ = mime2utf(ch&0x3f); > count++; > ch = mime2utf((ch>>6)&0x3f); > + continue; > } > } > *out++ = ch; Why is it incorrect? It seems to me that the upper bits should be written as the next character but now you are discarding them.

2 1

Re: [2/2] windowscodecs: Add a test for palette associated with black&white TIFF image.
by Marvin 12 Jul '12

12 Jul '12

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=19971 Your paranoid android. === WXPPROSP3 (32 bit tiffformat) === tiffformat.c:115: Test failed: CreateDecoderFromStream error 0x88982f8e tiffformat: unhandled exception c0000005 at 00413EB2 === WVISTAADM (32 bit tiffformat) === tiffformat.c:115: Test failed: CreateDecoderFromStream error 0x88982f8e tiffformat: unhandled exception c0000005 at 00413EB2

1 0

Re: msi: fix incorrect decoding of msi stream-names
by Nikolay Sivov 12 Jul '12

12 Jul '12

On 7/11/2012 18:03, Oleg Jakobsen wrote: > Looked at how you decode msi-stream names, and spotted a minor error > whith the decoding. > > regards > Oleg > > Why do you think it's an error? Looks like in this case it decodes one input WCHAR to two output WCHARs in a single pass.

1 0

Re: [3/5] windowscodecs: Create a predefined black&white palette if 1 bpp TIFF image doesn't have one.
by Vincent Povirk 11 Jul '12

11 Jul '12

Are you sure about this? My understanding is that the result of CopyPalette is unspecified when your pixel format is not an indexed color format (in this case, a grayscale format). So, while it's not really a problem to do that (unless a program relies on the exact behavior of this decoder, which seems unlikely), for correctness you shouldn't ever be relying on this change.

2 5

Re: msvcrt: Add bsearch_s implementation by reusing code and tests from ntdll (resend)
by Alexandre Julliard 11 Jul '12

11 Jul '12

André Hentschel <nerv(a)dawncrow.de> writes: > @@ -39,10 +44,19 @@ static void init(void) > pmemcpy_s = (void*)GetProcAddress(hmod, "memcpy_s"); > pI10_OUTPUT = (void*)GetProcAddress(hmod, "$I10_OUTPUT"); > pstrerror_s = (void *)GetProcAddress(hmod, "strerror_s"); > + pbsearch_s = (void *)GetProcAddress(hmod, "bsearch_s"); > p_get_doserrno = (void *)GetProcAddress(hmod, "_get_doserrno"); > p_get_errno = (void *)GetProcAddress(hmod, "_get_errno"); > p_set_doserrno = (void *)GetProcAddress(hmod, "_set_doserrno"); > p_set_errno = (void *)GetProcAddress(hmod, "_set_errno"); > + p_set_invalid_parameter_handler = (void *)GetProcAddress(hmod, "_set_invalid_parameter_handler"); set_invalid_parameter_handler is not exported in native so this won't test anything. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [5/5] gdiplus: Add some tests for GdipBitmapLockBits/GdipBitmapUnlockBits.
by Vincent Povirk 11 Jul '12

11 Jul '12

Shouldn't your second LockBits in this loop always use PixelFormat24bppRGB and ImageLockModeRead? As it is, if the LockBits/UnlockBits is writing but not reading, you won't be able to tell. Also, you might want to try flags == 0 in some pixel format other than 24bppRGB. As I understand it, all modes without ImageLockModeUserInputBuf are treated as read+write regardless of flags when the bitmap is locked in its native format, as an optimization. You may simply be running into that optimization. On Wed, Jul 11, 2012 at 2:49 AM, Dmitry Timoshkov <dmitry(a)baikal.ru> wrote: > With additional tests for flags set to ImageLockModeUserInputBuf only. > --- > dlls/gdiplus/tests/image.c | 184 +++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 184 insertions(+) > > diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c > index 29182aa..6ce100d 100644 > --- a/dlls/gdiplus/tests/image.c > +++ b/dlls/gdiplus/tests/image.c > @@ -3342,6 +3342,189 @@ todo_wine > GdipDisposeImage(image); > } > > +static void test_bitmapbits(void) > +{ > + > + static const BYTE pixels_24[24] = {0xff,0xff,0xff, 0,0,0, 0xff,0xff,0xff, 0,0,0, > + 0xff,0xff,0xff, 0,0,0, 0xff,0xff,0xff, 0,0,0}; > + static const BYTE pixels_24_00[24] = {0,0,0, 0,0,0, 0,0,0, 0,0,0, > + 0,0,0, 0,0,0, 0,0,0, 0,0,0}; > + static const BYTE pixels_24_77[32] = {0xff,0xff,0xff, 0,0,0, 0xff,0xff,0xff, 0,0,0,0x77,0x77,0x77,0x77, > + 0xff,0xff,0xff, 0,0,0, 0xff,0xff,0xff, 0,0,0,0x77,0x77,0x77,0x77}; > + static const BYTE pixels_24_88[32] = {0xff,0xff,0xff, 0,0,0, 0xff,0xff,0xff, 0,0,0,0x88,0x88,0x88,0x88, > + 0xff,0xff,0xff, 0,0,0, 0xff,0xff,0xff, 0,0,0,0x88,0x88,0x88,0x88}; > +#if 0 /* FIXME: these tests crash gdiplus in Wine */ > + static const BYTE pixels_8[8] = {0x01,0,0x01,0,0x01,0,0x01,0}; > + static const BYTE pixels_8_00[8] = {0,0,0,0,0,0,0,0}; > + static const BYTE pixels_8_77[32] = {0x01,0,0x01,0,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0x01,0,0x01,0,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77}; > + static const BYTE pixels_8_88[32] = {0x01,0,0x01,0,0x88,0x88,0x88,0x88, > + 0x88,0x88,0x88,0x88,0x88,0x88,0x88,0x88, > + 0x01,0,0x01,0,0x88,0x88,0x88,0x88, > + 0x88,0x88,0x88,0x88,0x88,0x88,0x88,0x88}; > + static const BYTE pixels_a7[32] = {0xa7,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0xa7,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77}; > + static const BYTE pixels_1[8] = {0xa0,0,0,0,0xa0,0,0,0}; > +#endif > + static const BYTE pixels_77[32] = {0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77, > + 0x77,0x77,0x77,0x77,0x77,0x77,0x77,0x77}; > + static const BYTE pixels_88[32] = {0x88,0x88,0x88,0x88,0x88,0x88,0x88,0x88, > + 0x88,0x88,0x88,0x88,0x88,0x88,0x88,0x88, > + 0x88,0x88,0x88,0x88,0x88,0x88,0x88,0x88, > + 0x88,0x88,0x88,0x88,0x88,0x88,0x88,0x88}; > + static const struct test_data > + { > + PixelFormat format; > + ImageLockMode mode; > + UINT width, height, stride, size; > + const BYTE *pixels; > + const BYTE *pixels_unlocked; > + } td[] = > + { > + { PixelFormat24bppRGB, 0xfff0, 4, 2, 12, 24, pixels_24, pixels_24_00 }, > + { PixelFormat24bppRGB, 0, 4, 2, 12, 24, pixels_24, pixels_24_00 }, > + > + { PixelFormat24bppRGB, ImageLockModeRead, 4, 2, 12, 24, pixels_24, pixels_24_00 }, > + { PixelFormat24bppRGB, ImageLockModeWrite, 4, 2, 12, 24, pixels_24, pixels_24_00 }, > + { PixelFormat24bppRGB, ImageLockModeRead|ImageLockModeWrite, 4, 2, 12, 24, pixels_24, pixels_24_00 }, > + { PixelFormat24bppRGB, ImageLockModeRead|ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_24_77, pixels_24_88 }, > + { PixelFormat24bppRGB, ImageLockModeWrite|ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_77, pixels_88 }, > + { PixelFormat24bppRGB, ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_77, pixels_88 }, > +#if 0 /* FIXME: these tests crash gdiplus in Wine */ > + { PixelFormat8bppIndexed, ImageLockModeRead, 4, 2, 4, 8, pixels_8, pixels_8 }, > + { PixelFormat8bppIndexed, ImageLockModeWrite, 4, 2, 4, 8, pixels_8, pixels_8 }, > + { PixelFormat8bppIndexed, ImageLockModeRead|ImageLockModeWrite, 4, 2, 4, 8, pixels_8, pixels_8_00 }, > + { PixelFormat8bppIndexed, ImageLockModeRead|ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_8_77, pixels_8_88 }, > + { PixelFormat8bppIndexed, ImageLockModeWrite|ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_77, pixels_88 }, > + { PixelFormat8bppIndexed, ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_77, pixels_88 }, > + > + { PixelFormat1bppIndexed, ImageLockModeRead, 4, 2, 4, 8, pixels_1, pixels_1 }, > + { PixelFormat1bppIndexed, ImageLockModeWrite, 4, 2, 4, 8, pixels_1, pixels_1 }, > + { PixelFormat1bppIndexed, ImageLockModeRead|ImageLockModeWrite, 4, 2, 4, 8, pixels_1, pixels_1 }, > + { PixelFormat1bppIndexed, ImageLockModeRead|ImageLockModeUserInputBuf, 4, 2, 16, 32, pixels_a7, pixels_a7 }, > +#endif > + }; > + BYTE buf[32]; > + GpStatus status; > + GpBitmap *bitmap; > + UINT i; > + BitmapData data; > + struct > + { > + ColorPalette pal; > + ARGB entries[1]; > + } palette; > + > + for (i = 0; i < sizeof(td)/sizeof(td[0]); i++) > + { > + BYTE pixels[sizeof(pixels_24)]; > + memcpy(pixels, pixels_24, sizeof(pixels_24)); > + status = GdipCreateBitmapFromScan0(4, 2, 12, PixelFormat24bppRGB, pixels, &bitmap); > + expect(Ok, status); > + > + /* associate known palette with pixel data */ > + palette.pal.Flags = PaletteFlagsGrayScale; > + palette.pal.Count = 2; > + palette.pal.Entries[0] = 0; > + palette.pal.Entries[1] = 0xffffffff; > + status = GdipSetImagePalette((GpImage *)bitmap, &palette.pal); > + expect(Ok, status); > + > + memset(&data, 0xfe, sizeof(data)); > + if (td[i].mode & ImageLockModeUserInputBuf) > + { > + memset(buf, 0x77, sizeof(buf)); > + data.Scan0 = buf; > + data.Stride = 16; > + } > + status = GdipBitmapLockBits(bitmap, NULL, td[i].mode, td[i].format, &data); > + ok(status == Ok || broken(status == InvalidParameter) /* XP */, "%u: GdipBitmapLockBits error %d\n", i, status); > + if (status != Ok) > + { > + GdipDisposeImage((GpImage *)bitmap); > + continue; > + } > + ok(td[i].width == data.Width, "%u: expected %d, got %d\n", i, td[i].width, data.Width); > + ok(td[i].height == data.Height, "%u: expected %d, got %d\n", i, td[i].height, data.Height); > + ok(td[i].stride == data.Stride, "%u: expected %d, got %d\n", i, td[i].stride, data.Stride); > + ok(td[i].format == data.PixelFormat, "%u: expected %d, got %d\n", i, td[i].format, data.PixelFormat); > + ok(td[i].size == data.Height * data.Stride, "%u: expected %d, got %d\n", i, td[i].size, data.Height * data.Stride); > + if (td[i].mode & ImageLockModeUserInputBuf) > + ok(data.Scan0 == buf, "%u: got wrong buffer\n", i); > + if (td[i].size == data.Height * data.Stride) > + { > + int match = memcmp(data.Scan0, td[i].pixels, td[i].size) == 0; > + if ((td[i].mode & (ImageLockModeRead|ImageLockModeWrite|ImageLockModeUserInputBuf)) == ImageLockModeWrite && td[i].format != PixelFormat24bppRGB) > + ok(!match, "%u: data shouldn't match\n", i); > + else > + { > + /* copying to user buffer is very buggy even in win7 */ > + ok(match || broken(!match && (td[i].mode & ImageLockModeUserInputBuf)), > + "%u: data should match\n", i); > + if (!match) > + { > + UINT j; > + BYTE *bits = data.Scan0; > + printf("%u: data mismatch for format %#x:", i, td[i].format); > + for (j = 0; j < td[i].size; j++) > + printf(" %02x", bits[j]); > + printf("\n"); > + } > + } > + > + memset(data.Scan0, 0, td[i].size); > + } > + > + status = GdipBitmapUnlockBits(bitmap, &data); > + ok(status == Ok, "%u: GdipBitmapUnlockBits error %d\n", i, status); > + > + memset(&data, 0xfe, sizeof(data)); > + if (td[i].mode & ImageLockModeUserInputBuf) > + { > + memset(buf, 0x88, sizeof(buf)); > + data.Scan0 = buf; > + data.Stride = 16; > + } > + status = GdipBitmapLockBits(bitmap, NULL, td[i].mode, td[i].format, &data); > + ok(status == Ok, "%u: GdipBitmapLockBits error %d\n", i, status); > + ok(td[i].width == data.Width, "%u: expected %d, got %d\n", i, td[i].width, data.Width); > + ok(td[i].height == data.Height, "%u: expected %d, got %d\n", i, td[i].height, data.Height); > + ok(td[i].stride == data.Stride, "%u: expected %d, got %d\n", i, td[i].stride, data.Stride); > + ok(td[i].format == data.PixelFormat, "%u: expected %d, got %d\n", i, td[i].format, data.PixelFormat); > + ok(td[i].size == data.Height * data.Stride, "%u: expected %d, got %d\n", i, td[i].size, data.Height * data.Stride); > + if (td[i].mode & ImageLockModeUserInputBuf) > + ok(data.Scan0 == buf, "%u: got wrong buffer\n", i); > + /* copying indexed data without associated palette from BitmapData leads > + to funny effects, that's a design flaw with lock/unlock pixel data */ > + if (td[i].size == data.Height * data.Stride && !(td[i].format & PixelFormatIndexed)) > + { > + int match = memcmp(data.Scan0, td[i].pixels_unlocked, td[i].size) == 0; > + ok(match, "%u: data should match\n", i); > + if (!match) > + { > + UINT j; > + BYTE *bits = data.Scan0; > + printf("%u: data mismatch for format %#x:", i, td[i].format); > + for (j = 0; j < td[i].size; j++) > + printf(" %02x", bits[j]); > + printf("\n"); > + } > + } > + > + status = GdipBitmapUnlockBits(bitmap, &data); > + ok(status == Ok, "%u: GdipBitmapUnlockBits error %d\n", i, status); > + > + status = GdipDisposeImage((GpImage *)bitmap); > + expect(Ok, status); > + } > +} > + > START_TEST(image) > { > struct GdiplusStartupInput gdiplusStartupInput; > @@ -3354,6 +3537,7 @@ START_TEST(image) > > GdiplusStartup(&gdiplusToken, &gdiplusStartupInput, NULL); > > + test_bitmapbits(); > test_tiff_palette(); > test_GdipGetAllPropertyItems(); > test_tiff_properties(); > -- > 1.7.11.1 > > >

2 1

Re: [PATCH 4/5] winex11: Simplify X11DRV_XRandR_Init() error handling a bit.
by Alexandre Julliard 11 Jul '12

11 Jul '12

Henri Verbeet <hverbeet(a)codeweavers.com> writes: > @@ -276,18 +274,15 @@ void X11DRV_XRandR_Init(void) > > /* see if Xrandr is available */ > wine_tsx11_lock(); > - ok = pXRRQueryExtension(gdi_display, &xrandr_event, &xrandr_error); > - if (ok) > - { > - X11DRV_expect_error(gdi_display, XRandRErrorHandler, NULL); > - ok = pXRRQueryVersion(gdi_display, &xrandr_major, &xrandr_minor); > - if (X11DRV_check_error()) ok = FALSE; > - } > - if (ok) > - { > - TRACE("Found XRandR - major: %d, minor: %d\n", xrandr_major, xrandr_minor); > - xrandr_init_modes(); > - } > + if (!pXRRQueryExtension( gdi_display, &xrandr_event, &xrandr_error )) goto done; > + X11DRV_expect_error( gdi_display, XRandRErrorHandler, NULL ); > + if (!pXRRQueryVersion( gdi_display, &xrandr_major, &xrandr_minor )) goto done; > + if (X11DRV_check_error()) goto done; You have to call X11DRV_check_error after X11DRV_expect_error in all cases. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/2] vbscript: Added UCase implementation
by Marvin 11 Jul '12

11 Jul '12

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=19930 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 2/2 try2] advapi32: Pass service name as argv[0] to ServiceMain
by Marvin 11 Jul '12

11 Jul '12

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

1 0

Re: kernel32: implement IsValidLocaleName (as a wrapper around IsValidLocale) (2/2) (try 2)
by Alexandre Julliard 10 Jul '12

10 Jul '12

Austin English <austinenglish(a)gmail.com> writes: > @@ -2202,6 +2202,19 @@ BOOL WINAPI IsValidLocale( LCID lcid, DWORD flags ) > (LPCWSTR)LOCALE_ILANGUAGE, LANGIDFROMLCID(lcid)) != 0; > } > > +/****************************************************************************** > + * IsValidLocaleName (KERNEL32.@) > + * > + * See IsValidLocale(). > + */ > +BOOL WINAPI IsValidLocaleName( LPCWSTR locale ) > +{ > + LCID lcid; > + > + TRACE( "(%s)\n", debugstr_w(locale) ); > + lcid = LocaleNameToLCID( locale, 0 ); > + return IsValidLocale( lcid, 0 ); > +} That's probably not going to do the right thing for invalid names. Some tests would be a good idea. -- Alexandre Julliard julliard(a)winehq.org

1 0

re: Re-enable editing on the forum?
by Dan Kegel 10 Jul '12

10 Jul '12

No objection here. Seems like an obvious thing to do. I'll do it later today if nobody objects.

3 2

Re: [1/2] gdiplus: GdipBitmapLockBits should treat flags == 0 as ImageLockModeRead.
by Vincent Povirk 10 Jul '12

10 Jul '12

I think you might want (!(flags & (ImageLockModeRead|ImageLockModeWrite))), in case someone passes ImageLockModeUserInputBuf On Tue, Jul 10, 2012 at 7:45 AM, Dmitry Timoshkov <dmitry(a)baikal.ru> wrote: > --- > dlls/gdiplus/image.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c > index db79eca..45520fd 100644 > --- a/dlls/gdiplus/image.c > +++ b/dlls/gdiplus/image.c > @@ -942,6 +942,8 @@ GpStatus WINGDIPAPI GdipBitmapLockBits(GpBitmap* bitmap, GDIPCONST GpRect* rect, > return WrongState; > } > > + if (!flags) flags = ImageLockModeRead; > + > if (bitmap->bits && bitmap->format == format && !(flags & ImageLockModeUserInputBuf)) > { > /* no conversion is necessary; just use the bits directly */ > -- > 1.7.11.1 > > >

2 1

Re: kernel32: implement IsValidLocaleName (as a wrapper around IsValidLocale)
by Dmitry Timoshkov 10 Jul '12

10 Jul '12

Austin English <austinenglish(a)gmail.com> wrote: > +BOOL WINAPI IsValidLocaleName( LPCWSTR locale ) > +{ > + LCID lcid; > + BOOL ret; > + > + TRACE( "locale: %s\n", debugstr_w(locale) ); > + lcid = LocaleNameToLCID( locale, 0 ); > + ret = IsValidLocale( lcid, 0 ); > + return ret; > +} Avoiding (useless) intermediate variables along with the word 'locale' in the trace would be better IMO. -- Dmitry.

2 1

Re: some patches to read files faster (especially for baldur's gate and infinity engine games)
by Austin English 10 Jul '12

10 Jul '12

On Mon, Jul 9, 2012 at 2:06 PM, Emmanuel Anne <emmanuel.anne(a)gmail.com> wrote: > Hello, I installed baldur's gate lately and noticed it was still slow in > wine, especially if I install a few mods. > See the description of the bug here : > http://bugs.winehq.org/show_bug.cgi?id=17956 > > So after reading the page about case insensitive filesystems there > http://wiki.winehq.org/CaseInsensitiveFilenames > I decided that one possible solution was to have everything in lower case. > So I made a 1st patch (see attachement). > After this, installing a weidu mod in baldur's gate becomes faster, but > loading a savegame is still very slow compared to windows. > > After some more investigation using strace, it's because the program uses > NtQueryDirectoryFile to check if a file is present in the override directory > instead of trying to open it directly, which produces a getdents call in > linux, which is extremely unefficient. So I just added a short cut for this > function : if the filename argument has no wildcard, then just use stat to > return wether the file exists or not. This is in the 2nd patch. > After this, loading savegames was extremely fast, comparable to windows > speed, finally, and there are no more slowdowns in game, it runs smoothly > all the time ! > > Well I am sure you'll find I didn't make them the right way, but I hope the > ideas will be useful to you and that you'll merge them in one form or > another into wine. > Oh yes, after applying them, all the new files created by wine will be in > lower case, but you'll eventually have to convert all the files to lower > case in the wine directories (it's required at least in the baldur's gate 2 > directory). > Also having a wineprefix with mixed lower and upper case characters would > create problems. > But except that, it works excessively well ! :) > So maybe you'll want this enabled only if the user explecitely chooses to > enable it. > For me I know I'll keep it ! > I attach the simple perl script I used to convert everything to lower case. It would be great to document this patch at http://wiki.winehq.org/InterestingPatches. -- -Austin

1 0

Re-enable editing on the forum? - Excuse me!
by newsletter [at] Schiermeier-Software 10 Jul '12

10 Jul '12

Hello list, sorry for my multiple posting in the subject "Re-enable editing on the forum?" I had a mail account problem. Thanks for your appreciation! -- Gruss von Joerg Schiermeier

1 0

Re: "winetricks dotnet40" fails verifier because System.EnterpriseServices.Wrapper.dll missing?
by Dan Kegel 09 Jul '12

09 Jul '12

On Mon, Jul 9, 2012 at 1:42 PM, Hans Leidekker <hans(a)codeweavers.com> wrote: >> That file does seem to be on disc, but somewhere else: >> c:/windows/Microsoft.NET/Framework/v4.0.30319/System.EnterpriseServices.Wra… >> >> Not sure how serious it is that that's misplaced. Have other folks seen this? > > This should be fixed by 45473a65a0a1fe2e61809d80f6d09bb96f923126. Indeed it is. Thanks! - Dan

1 0

Re-enable editing on the forum?
by Schiermeier Software 09 Jul '12

09 Jul '12

Guten Abend Rosanne, > but someone (Jeremy?) needs to correct the other page. maybe you want to send a patch to: WineHQ - patch <wine-patches(a)winehq.org> subject: [website] <your subject> This works easier. -- Gruss von Joerg Schiermeier

2 1

Re: [PATCH 3/4] d3dx9/tests: Add effect parameter value GetMatrixPointerArray() test.
by Matteo Bruni 09 Jul '12

09 Jul '12

2012/7/8 Rico Schüller <kgbricola(a)web.de>: > --- > dlls/d3dx9_36/tests/effect.c | 90 > ++++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 90 insertions(+), 0 deletions(-) > Hi Rico, maybe it's just me misunderstanding this, but: + FLOAT fvalue[EFFECT_PARAMETER_VALUE_ARRAY_SIZE]; + D3DXMATRIX *matrix_pointer_array[sizeof(fvalue)/sizeof(D3DXMATRIX)]; + UINT l, k, m, element; + + for (element = 0; element <= res_desc->Elements + 1; ++element) + { + memset(fvalue, 0xab, sizeof(fvalue)); + for (l = 0; l < element; ++l) + { + matrix_pointer_array[l] = (D3DXMATRIX *)&fvalue[l * sizeof(**matrix_pointer_array) / sizeof(*matrix_pointer_array)]; + } I don't quite understand what you're trying to achieve with "sizeof(**matrix_pointer_array) / sizeof(*matrix_pointer_array)". As far as I can see, that depends on the size of the pointer (32/64 bits) and I'm not sure that the offset should change in that case. Maybe you meant to use sizeof(float) as divisor?

2 1

Re-enable editing on the forum?
by Joerg Schiermeier 09 Jul '12

09 Jul '12

Guten Abend Rosanne, > but someone (Jeremy?) needs to correct the other page. maybe you want to send a patch to: WineHQ - patch <wine-patches(a)winehq.org> subject: [website] <your subject> This works easier. -- Gruss von Joerg Schiermeier

1 0

Re-enable editing on the forum?
by newsletter [at] Schiermeier-Software 09 Jul '12

09 Jul '12

Guten Abend Rosanne, > but someone (Jeremy?) needs to correct the other page. maybe you want to send a patch to: WineHQ - patch <wine-patches(a)winehq.org> subject: [website] <your subject> This works easier. -- Gruss von Joerg Schiermeier

1 0

Re: [PATCH 3/4] ieframe: Added init_test helper
by Marvin 09 Jul '12

09 Jul '12

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=19864 Your paranoid android. === WNT4WSSP6 (32 bit webbrowser) === webbrowser.c:3325: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3325: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3325: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3476: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3325: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3441: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3419: Test failed: Could not get WebBrowserV1 instance: 80040154 webbrowser: unhandled exception c0000005 at 0040AF8E === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2627: Test failed: expected GetOverridesKeyPath webbrowser.c:2632: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2633: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3052: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 00402FCA

1 0

Re: [PATCH 4/4] vbscript: Added support for exit for stateemnt in 'for in' loops
by Marvin 09 Jul '12

09 Jul '12

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

1 0

Re: [PATCH 2/4] ieframe: Added more DoVerb tests
by Marvin 09 Jul '12

09 Jul '12

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=19863 Your paranoid android. === WNT4WSSP6 (32 bit webbrowser) === webbrowser.c:3313: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3313: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3313: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3472: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3313: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3435: Test failed: Creating WebBrowser object failed: 80040154 webbrowser.c:3410: Test failed: Could not get WebBrowserV1 instance: 80040154 webbrowser: unhandled exception c0000005 at 0040AF8E === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2627: Test failed: expected GetOverridesKeyPath webbrowser.c:2632: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2633: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3052: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 00402FCA

1 0

Re: [PATCH 1/8] hhctrl.ocx: Implement HH_CLOSE_ALL.
by Jacek Caban 09 Jul '12

09 Jul '12

Hi Erich, On 07/04/12 20:32, Erich E. Hoover wrote: > This series of patches is intended to fix the problems with HTML Help > associated with Bug #11303 (HtmlHelp causes problems in Elster). This > patch implements some of the framework required by the other patches > through implementing the HH_CLOSE_ALL functionality needed by the > application to safely shutdown the HTML Help windows on exit. The > framework provided in this patch is the ability to register specific > HTML Help windows (needed later to set and get window properties on > the fly). While it's not about this particular patch, I can see that you're changing hhctrl.ocx to the point where it has an useful API (as opposed to simple hh.exe-like help viewer). It is really nice to see, thanks. It would be great to have some tests for this. Please consider adding them (I'm sure you already have some code for testing, so I imagine it would be mostly integrating it with Wine test suite). Thanks, Jacek

2 2

App bundle patches for OS X
by Per Johansson 08 Jul '12

08 Jul '12

Hi, I think I've reached a committable stage for the application bundle code I took over from Steven Edwards last summer. But there are some controversial parts so I figured I'd ask the list first. The patch extracts all XDG specific parts of winemenubuilder into a separate file, and there is also a new file for app bundles. Which one is used is controlled by a dispatch table and is choosable in the registry. The new code reacts to the same events but instead of a .desktop file an app bundle is created, start menu is replicated at ~/Applications/Wine/. Associations are handled by modifying start.exe to make it able to receive the file to open using AppleEvent, and an app bundle execing start.exe is put in ~/Library/Wine/Associations/. The actual file name contains the associated extension but the display name is modified to be the application that will be launched. It should work out of the box, after you've installed this patch and run winemenubuilder -a you should be able to right click a text file and choose "Open with notepad" (there's a cache involved as well so you might also need to remove the wine fileassociations key in the registry). Commits can be found at https://github.com/morth/wine/commits/appbundle-proper (original commits are on the master branch). The major controversity is in the start.exe code (commit < https://github.com/morth/wine/commit/ec68ac2b2abc0a1d4559eb5490b1f12b21000e…>). It uses a private API to access the AppleEvent containing the file to open. I choose this way because: 1. The only official way to access this information is by using Objective-C, building a proper Mac application. 2. Another option was to use the deprecated Carbon, but would also take a lot of code to finish. IMO neither of the two are good options, which leaves either doing it this way or to ask Apple to open up the API. Unfortunately I don't think Apple will respond quickly to such a request. Now that 10.8 is close (maybe even GMed) it's very unlikely to happen before 10.9, and perhaps not even then without backing from a major developer (I suppose the wine project might count as a such). I've made sure to check with configure if the functions exist before calling, but it is of course not safe in case they change the signature. OTOH, I don't think that is overly likely to happen. Another thing to note is that the app bundle script will copy some environment variables when created (commit https://github.com/morth/wine/commit/58773927eb7da431ba47923d66768cd129e4da… function). The ones copied are: PATH, DYLD_FALLBACK_LIBRARY_PATH and WINEPREFIX. DYLD_FALLBACK_LIBRARY_PATH is needed or wine won't work much at all. PATH is copied because the default one is only /bin and /usr/bin, and on OS X wine is rarely in one of those locations but rather installed with MacPorts of fink or similar, which use non-default paths. Anyway, if I don't hear anything from the list I'll likely send in the commits. I suppose that will be a second chance to review them. Once again I'd like to mention that Steven Edwards did a major part of this code, thanks goes to him and also to the wine project as a whole, I really enjoy using it. Finally, I only work on this code once every few months or so, so if it's deemed that there's too many problems, anyone should feel free to pick up this code and make it done (it'd be nice if you notified me though). Regards, -- Per Johansson

1 0

Re-enable editing on the forum?
by Rosanne DiMesio 08 Jul '12

08 Jul '12

The main reason for disabling editing on the forum was the confusion it caused for mailing list users. Since the two are no longer connected, that is no longer an issue. Are there any objections to re-enabling editing on the forum? -- Rosanne DiMesio <dimesio(a)earthlink.net>

1 0

Re: [2/2] msxml3: Support xmlns[:*] attribute nodes intelligently.
by Nikolay Sivov 08 Jul '12

08 Jul '12

On 6/12/2012 11:55, Ulrik Dickow wrote: > Patch 2 of 2 from http://bugs.winehq.org/show_bug.cgi?id=26226 . > > > > + /* Emit appropriate fixme or warning in case of unsupported or invalid namespace. > + * The W3C spec would like us to exit earlier for attempts to redefine the namespace for > + * the reserved "xmlns" attribute or prefix (http://www.w3.org/TR/xml-names/#ns-decl), > + * likehttp://gdome2.cs.unibo.it/gtk-doc/gdome2-gdomeelement.html#GDOME-EL-SETATTRIBUTENS, > + * but since native msxml3 accepts anything here, it's safest to just continue with warning. > + */ The question is more like what libxml2 thinks about that. If it follows w3c here which is likely the question will be - won't it break somewhere internally in libxml2 code if we hack it that way? > + if(namespaceURI && namespaceURI[0] && node_type != NODE_ELEMENT) > + { > + if(node_type == NODE_ATTRIBUTE) > + switch((!strcmpW(name, xmlnsW) || > + !strncmpW(name, xmlnscW, sizeof(xmlnscW)/sizeof(WCHAR))) + > + !strcmpW(namespaceURI, w3xmlns)) Could you please simplify that, it's a bit hard to read. > + * Note: We would be more in line with the native msxml if we only detected a > + * redundancy/conflict for prefixes existing on the element itself (in element->ns > + * or in the linked list element->nsDef). But as long as our *_get_xml() doesn't > + * eliminate redundant namespace definitions in children, this behaviour may be > + * an advantage in some cases. As disadvantage is that we incorrectly regard the > + * redefinition in element 'b' in '<a xmlns="x"></a>' as a conflict. > + * Libxml2 currently doesn't have a function or option to only search for namespaces > + * on the current node; but it would be easy to make a reduced version of xmlSearchNs > + * (currently defined inhttp://git.gnome.org/browse/libxml2/tree/tree.c#n5871). > + * However, most apps probably set the ns to y on node b creation, so no conflict here. > + * I don't think it's better to potentially break one thing while fixing the other. And I guess I like a thought on making xmlSearchNs version that will do what we want from it. Just to clarify - is that right that libxml2 problem here is a lack of some checks that lead to duplication and conflicts later? (that are visible in doc dumps too) > +/* Convenient helper function for creating an attribute node in a given namespace. > + * Note: CHECK_HR() should not be here, but in caller, to give optimal line number info. > + */ You mean EXPECT_HR here I guess. > +static HRESULT create_attribute_ns( > + IXMLDOMDocument *doc, > + const char *attr_name, > + const char *nsURI, > + IXMLDOMAttribute **attr_ptr) > +{ > + HRESULT hr; > + VARIANT var; > + IXMLDOMNode *node; > + > + V_VT(&var) = VT_I1; > + V_I1(&var) = NODE_ATTRIBUTE; > + > + hr = IXMLDOMDocument_createNode(doc, var,_bstr_(attr_name),_bstr_(nsURI), &node); > + if (hr == S_OK) > + { > + IXMLDOMNode_QueryInterface(node, &IID_IXMLDOMAttribute, (void**) attr_ptr); > + IXMLDOMNode_Release(node); > + } > + return hr; > +} Please null out pointer in case of failure, or better always before createNode().

2 4

winebuild port to ARM?
by Roger Cruz 07 Jul '12

07 Jul '12

Has anyone worked or is currently working on porting the x86 specific code in Winebuild to ARM? Specifically the relay.c file? Thanks Roger R. Cruz

2 9

Re: [PATCH] ntdll: Add stubs for WinSqmStartSession / WinSqmEndSession
by André Hentschel 07 Jul '12

07 Jul '12

Am 07.07.2012 01:08, schrieb Detlef Riekenberg: > It was to simple for a stub. > I added more parameter, until the 32-Bit app stopped crashing. > > -- > By by ... Detlef > --- > dlls/ntdll/rtl.c | 18 ++++++++++++++++++ > 1 files changed, 18 insertions(+), 0 deletions(-) > > diff --git a/dlls/ntdll/rtl.c b/dlls/ntdll/rtl.c > index 34869e7..ec35a23 100644 > --- a/dlls/ntdll/rtl.c > +++ b/dlls/ntdll/rtl.c > @@ -1310,3 +1310,21 @@ BOOL WINAPI RtlSetCurrentTransaction(HANDLE new_transaction) > FIXME("(%p) :stub\n", new_transaction); > return FALSE; > } > + > +/****************************************************************************** > + * WinSqmEndSession [NTDLL.@] > + */ > +NTSTATUS WINAPI WinSqmEndSession(void *unknown1) > +{ > + FIXME("(%p) :stub\n", unknown1); > + return STATUS_NOT_IMPLEMENTED; > +} > + > +/****************************************************************************** > + * WinSqmStartSession [NTDLL.@] > + */ > +NTSTATUS WINAPI WinSqmStartSession(void *unknown1, void *unknown2, void *unknown3) > +{ > + FIXME("(%p, %p, %p) :stub\n", unknown1, unknown2, unknown3); > + return STATUS_NOT_IMPLEMENTED; > +} > You forgot the spec update. -- Best Regards, André Hentschel

1 0

Debugging wine/windows applications
by John Yani 07 Jul '12

07 Jul '12

I tried to run wine under gdb and failed. Using multiprocess gdb I endup with weird trace: 0xf7ffd430 0x7bc846f9 0x7bc8480f 0x7bc84855 0x7bc42a94 0x7bc433b1 0x7b8772f7 0x7ebab89b 0x7bc80014 0x7bc8005d Where 0x7****** addresses are not connected to any module. And 0xf****** addresses are from /lib/ld-linux.so.2, debug symbols for which I can't find. Can somebody explain what's hapenning? Running wine with winedbg under gdb fails with message "err:module:LdrInitializeThunk Main exe initialization for L"C:\\windows\\system32\\notepad.exe" failed, status c0000022" Is there any tutorial on how to run wine with applications under gdb?

2 10

Policy for the Wiki?
by Kyle Auble 07 Jul '12

07 Jul '12

Hello all, I've been working on the wiki recently and noticed that a lot of out-of-date or redundant information is retained. For example, completed tasks on to-do lists are checked off and never removed. Isthere an unofficial policy of not deleting info, even if it may exist elsewhere (in the git repo history, wikipage revision history, etc)? Other instances might be stubs dedicated to features in development years ago, or pages on bugs that have already been fixed or triaged in Bugzilla.I know even page deletions can still be reverted, but I wanted to check first before cleaning things out more aggressively. If there are a few unwritten rules everyone likes the wiki to follow, I could add them somewhere as editing guidelines. While the topic's up, I was also wondering how moinmoin was chosen as the wiki engine. I couldn't really find a discussion on the mailing lists so I figured it was the best option when the wiki was first started. I can appreciate that it doesn't require a database and it's written in python instead of php. Allowing only inline CSS in tables causes some headaches, but mainly it can be really slow sometimes, especially when editing, and unless using wikilinks or going directly to a page, navigation is tricky. Has the possibility of migrating the wiki to a different engine ever come up? Or does the Wine wiki code just need some tweaking and maybe some tools? To be honest, I don't know if there is a better alternative (MediaWiki is designed for a very different use case and has really messy code). It just seems like something is discouraging even basic upkeep on the wiki, which kind of defeats its whole purpose. -Kyle

2 2

Building Wine on amd64 FreeBSD?
by Hilko Bengen 06 Jul '12

06 Jul '12

Following Marcus Meissner's advice how to build a WoW64 setup (thanks!), I have succeeded in building a set of co-installable i386/amd64 Debian packages. Those won't make it into the "wheezy" release as I was just a bit late for the freeze, but I have uploaded packages to experimental. What didn't work was building the 64-bits of wine on Debian/kfreebsd-amd64 (Debian userland including GNU libc on a FreeBSD kernel): <https://buildd.debian.org/status/fetch.php%3Fpkg%3Dwine%26arch%3Dkfreebsd-a…> Apparently the reason is that the uc_mcontext structure does not store the DS, ES, FS, and GS registers. After I made the changes below, everything compiled, but how useful is the result? Are those registers contained in uc_mcontext on "plain" FreeBSD? Cheers, -Hilko

2 1

Re: kernel32: add a stub for WerSetFlags
by Dmitry Timoshkov 05 Jul '12

05 Jul '12

Austin English <austinenglish(a)gmail.com> wrote: > +HRESULT WINAPI WerSetFlags(DWORD flags) > +{ > + FIXME("(%d) stub!\n", flags); > + return E_NOTIMPL; > +} 1. 'flags' is unsigned 2. if it's supposed to be bit flags then printing it in decimal is not helpful. -- Dmitry.

1 0

"winetricks dotnet40" fails verifier because System.EnterpriseServices.Wrapper.dll missing?
by Dan Kegel 05 Jul '12

05 Jul '12

With current git, when I try rm -rf .wine sh winetricks -q dotnet40 sh winetricks -q dotnet_verifier wine netfx_setupverifier.exe the verifier reports failure: ... [07/04/12,21:45:13] ****ERROR**** File C:\windows\Microsoft.NET\assembly\GAC_32\System.EnterpriseServices\v4.0_4.0.0.0__b03f5f7f11d50a3a\System.EnterpriseServices.Wrapper.dll is not installed on the system ... [07/04/12,21:45:15] ****ERROR**** Verification failed for product .NET Framework 4 Client That file does seem to be on disc, but somewhere else: c:/windows/Microsoft.NET/Framework/v4.0.30319/System.EnterpriseServices.Wra… Not sure how serious it is that that's misplaced. Have other folks seen this? - Dan

1 0

[PATCH] winepulse.drv: Add PulseAudio driver
by Joerg-Cyril.Hoehle＠t-systems.com 04 Jul '12

04 Jul '12

Hi, >If you don't paste all the mmdevapi tests with the fix in my git tree, it's a >regression. I don't want to see held_Frames because it's a shadow buffer. >If winmm and dsound don't work without it, they're wrong, fix those. I won't comment on the two individual pulseaudio drivers. Unfortunately, none of the two authors bothered to point me to a verbose log of the mmdevapi tests for my scrutiny. The tests should pass like they do on native. Regarding latency, I'm not aware of a flaw in Wine's winmm. HOWEVER, Wine's DSound needs a patch. Here the reason. To recap, latency is somehow related to the "distance" between what one hears and the PCM frames sent to the device. GetPosition is the only API call in that area in both winmm and mmdevapi. Buffer size is a completely different thing. I find it useful to have in mind a cascade of audio filters performing some buffering each. Then it's obvious that an app only sees the frontmost buffer size and knows nothing about the others. High latencies imply that there must be some large buffer space(s), somewhere. Period is an artificial entity related to how often the frontmost buffer is drained to feed the next buffer in the chain, in case that happens regularly. This is not guaranteed nor necessarily documented. You get the idea: regardless of arbitrary (front) buffer sizes and periods not worth this name, try and send a stable flow of data to play music without glitches and let explosions be heard ASAP. So far the situation, now the issue. It appears easier to write code that provides (frontmost) buffer space behaviour similar to native than one that provides similar latency. Alas, behaviour observable on MS-Windows differs from what we get with PulseAudio and some ALSA devices. Tests seem to indicate a latency of around 30-70ms from mmdevapi with MS-Windows. That good enough for games. OTOH if ALSA or PA gives us 2 seconds on Linux, that's a big cause of trouble. The authors of MS apps could never test them in an environment with such high latency. What happens with software not tested? It happens not to work. Two solutions: - Reduce actual latency or - lie about latency. Lying about latency causes apps to loose the ability to sync audio and video. However, given a choice between loss of lip sync and possible crashes or other weird behaviour because an app is executed in a environment that its developers never experienced, I consider the loss of lip sync to be less worrying. We designed the winealsa driver to accomodate an arbitrary latency and accept a large variation of periods. (Some bug report from Jack users seem to imply that it should accept even larger periods, e.g. 150ms, yet still pretend to use 10ms on the mmdevapi side). We choose to have the winealsa driver not lie about latency. That should be left to high level APIs. DirectSound is built around the "Direct" HW (lack of) abstraction: a circular buffer of samples is played by a DAC. The converted signal is immediately sent to the speaker. Hence: 1. GetPosition information in the DSound abstraction translates to a "playpos" -- well known. 2. No provision is made for additional buffering. The playpos must lie within the circular buffer. The reported free (writable) space shall never cross the playpos -- with subtle consequences. Wine's DSound needs a patch to ensure this second property. Given a 80ms primary buffer, Wine's DSound must not pretend its playpos is 2 seconds late! I think Wine should try and reduce latency but lie if that does not suffice. What's needed? 1. Have DSound always use a buffer large enough for typical situations (I believe 100-200ms). 2. Clamp the reported position such that it won't leave the (virtual) primary buffer. As a result, DSound must pretend to play even when Wine is solely pumping the huge 2s of cascading SW audio buffers. 3. Work on reducing the latency of the cascading audio filters. That device-level work is independent on the DSound one. Native's 30-70ms has the benefit that it nicely fits within typical DSound primary buffer sizes. PulseAudio's 2s does not (yet). I believe a total 80ms latency would be acceptable with games. Further areas of investigation and effort: - Maybe clamp latency in the mmdevapi drivers after all? After all, huge latencies are known to cause dead-locks in apps and we don't know what all apps or libraries built atop mmdevapi or winmm expect. (If yes, to what value? 150% buffer size?) - Try and find MS setups with huge latencies. USB headphones are said to be candidates, but I've yet to see an excellent and trustworthy report about what happens in that case. - Work on reducing the cascades of audio buffers - Work on further decoupling mmdevapi periods from UNIX audio API ones - Research the trade-off between glitch free playback and buffering in UNIX. The current settings in Wine's audio are IMHO not good enough yet. Regards, Jörg Höhle

2 1

Re: [1/4] gdiplus: Add a test for font and graphics using different base units. Resend.
by Vincent Povirk 04 Jul '12

04 Jul '12

> +todo_wine > + expectf(102.499985, bounds.Width); Please don't hard-code string widths in your tests. It's not likely or desirable to get them exactly right.

2 1

New Wine Gecko 1.7-beta1
by Jacek Caban 04 Jul '12

04 Jul '12

Hi all, It's a bit sooner for another Gecko update than usually. This is caused by the fact, that many existing bugs in our MSHTML are blocked by another bugs, which require Gecko changes for proper fix. These changes enable using Gecko's cycle collector to understand our objects. This solves quite a few problems. The update also contains merged Firefox 14 beta changes. To test it, download Wine Gecko [1] and install it as usually [2]. You will also need the attached patch. The patch contains not only Gecko API changes, but also partial (but substantial enough for test to be meaningful) support for cycle collector in MSHTML. You may find some bugs already fixed by this patch. Any help with testing will be appreciated. Thanks, Jacek [1] http://sourceforge.net/projects/wine/files/Wine%20Gecko/1.7-beta1/ [2] http://wiki.winehq.org/Gecko

1 0

Re: League of Legends patches
by Dan Kegel 04 Jul '12

04 Jul '12

On Tue, Jul 3, 2012 at 1:08 AM, Andrea Canciani <ranma42(a)gmail.com> wrote: > I tried to follow the http://wiki.winehq.org/SubmittingPatches > guidelines, but this is my first submission to wine, so I guess the > patches might need further improvement. > Please point out any issues that need fixing. On an administrivial note, you should send one patch per email to wine-patches. Do you think you could write a test that (semi-)reliably causes the deadlock you're fixing? > The patches have been tested by Dan Kegel (in CC) and are currently > being used by many MacOSX and Linux users to run LoL. Our emails may have crossed - with these patches applied, dlls/ieframe/tests/webbrowser.ok seems to crash here: Unhandled exception: privileged instruction in 32-bit code (0x02786624). Backtrace: =>0 0x02786624 (0x0032fcd8) 1 0x688e22db func_webbrowser+0x3ca() [dlls/ieframe/tests/webbrowser.c:3382] Otherwise the tests seem to pass. Thanks, Dan

3 3

Re: [3/4] gdiplus: Add some tests for GdipGetPropertyItemSize and GdipGetPropertyItem. Take 2.
by Marvin 04 Jul '12

04 Jul '12

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

2 1

Re: [4/4] gdiplus: Add a bunch of tests for TIFF image properties. Resend.
by Marvin 04 Jul '12

04 Jul '12

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

1 0

Re: [PATCH 3/4] vbscript: Fixed function return crossing for loop
by Marvin 03 Jul '12

03 Jul '12

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=19753 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: [PATCH 4/4] jscript: Added more apply and call tests
by Marvin 03 Jul '12

03 Jul '12

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

1 0

Re: [PATCH 2/4] vbscript: Fixed 'exit do' statements crossing for in loops
by Marvin 03 Jul '12

03 Jul '12

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=19752 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Wine on the front page of Reddit
by Caleb Hearon 03 Jul '12

03 Jul '12

Hi all, In a conversation between myself (MaggotBrain on the forums) and Jarrex, I expressed some frustration that Blizzard had been banning Diablo III users who used Wine. He posted some of the excerpts I sent him from WineHQ, and it quickly got to 17 pages on Blizzard's forums. http://us.battle.net/d3/en/forum/topic/5978861022 Then it made the front page of Reddit with 25k votes! http://www.reddit.com/r/gaming/comments/vyc4z/linux_users_permanently_banne… It's likely a mistake on Blizzard's part. But I thought this might be interesting to some people here. It's amazing that this conversation I had while at work led to more widespread awareness of Wine (one of the top comments on Reddit asks what Wine is). And Wine hackers- great work, I've just finished playing 2 hours of Diablo III and it runs great. -Caleb

2 1

Re: [1/2] kernel32/tests: Improve some traces for the CreateTimerQueueTimer() tests.
by Marvin 03 Jul '12

03 Jul '12

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=19733 Your paranoid android. === WVISTAADM (32 bit sync) === sync.c:924: Test failed: DeleteTimerQueueTimer sync.c:925: Test failed: ChangeTimerQueueTimer

2 1

user32: Add test for ComboBox repositioning behavior (try 2).
by Sergey Guralnik 03 Jul '12

03 Jul '12

Alexandre Julliard writes: >> @@ -5807,6 +5814,13 @@ static void test_combobox_messages(void) >> log_all_parent_messages--; >> ok_sequence(WmKeyDownComboSeq, "WM_KEYDOWN/VK_DOWN on a >> ComboBox", FALSE); >> >> + SetWindowLongPtr(combo, GWLP_WNDPROC, >> (LONG_PTR)combobox_hook_proc_2); >> + SendMessage(combo, CB_SETCURSEL, 0, 0); >> + SetWindowPos(combo, 0, 0, 0, 120, 130, >> SWP_NOZORDER|SWP_NOMOVE); >> + ret = SendMessage(combo, CB_GETCURSEL, 0, 0); >> +todo_wine >> + ok(ret == 1, "expected 1, got %ld\n", ret); >> + > > There's no reason to test just WM_SIZE when we have support for > testing > full sequences. Once again, what are you trying to test? I only try to test that combobox should send WM_SIZE to it's wndproc during WM_WINDOWPOSCHANGED handling. -- Sergey

2 1

Fwd: [GSoC Mentors] [Announce] 2012 GSoC Mentor Midterm Evaluations 9 July - 13 July
by Austin English 02 Jul '12

02 Jul '12

A friendly reminder for GSOC people. If you won't be able to submit your evaluation during the specified timeline, or have any other questions, please contact me privately. Cheers, Austin ---------- Forwarded message ---------- From: Carol Smith <carols(a)google.com> Date: Fri, Jun 29, 2012 at 11:41 AM Subject: [GSoC Mentors] [Announce] 2012 GSoC Mentor Midterm Evaluations 9 July - 13 July To: Google Summer of Code Mentors List <google-summer-of-code-mentors-list(a)googlegroups.com> Hi GSoC 2012 Mentors and Org Admins, Per the program timeline [0], starting 9 July you will will be able to submit an evaluation of your student(s)' progress on their projects thus far. Here's some important info on midterm evaluations for those not familiar: Midterm evaluations are done via Melange [1]. Starting at 19:00 UTC on 9 July you will be able to submit an evaluation for your student(s).You can find the evaluation links on your dashboard under 'Evaluations', one for each student you are a mentor (or co-mentor) for. If you are curious about who can see evaluations after they are submitted, please check out the FAQ on Evaluations [2]. I have also pre-published the evaluation questions below in this email so you can prepare. The deadline is 19:00 UTC on 13 July. You may not submit your evaluation before or after the evaluation window. Please ask your org admin to submit your evaluation for you if you absolutely cannot submit yours during the time allotted. Primary mentors, co-mentors, and org admins may all submit evaluations of their students.**Students must have an evaluation on file from both themselves *and* their mentors in order to receive their midterm payments.** There is no excuse for missing the submission of a student's evaluation. You must submit an evaluation for every student you are the primary mentor for this year. You must fill out the entire survey in one session as there's no auto-save in Melange. You may submit, modify, and resubmit an evaluation as many times as you choose up until the deadline. Please note that failing a student at the midterm evaluation means *this student is immediately removed from the program.* There is no way to fail a student at the midterm and have the student continue with the program to try to "make it up" at the final. If your student fails, your student is out. You might find the FLOSS manual on GSoC evaluations [3] helpful as well. There's some excellent wisdom in there from your fellow mentors and org admins on the evaluation process. Finally, a reminder: This year we will not allow any mentor who misses an evaluation deadline to attend the mentor summit (assuming no one else submits the evaluation on the mentor's behalf before the deadline either). Also, any org that misses 2 or more evaluation deadlines (for the midterm, final, or midterm and final combined) will not be invited to attend the mentor summit this year. Please let me know directly if you have questions or concerns. [0] - http://www.google-melange.com/gsoc/events/google/gsoc2012 [1] - http://google-melange.com [2] - http://www.google-melange.com/document/show/gsoc_program/google/gsoc2012/fa… [3] - http://www.booki.cc/gsoc-mentoring/evaluations/ Cheers, Carol ------------ How many years have you been a mentor for Google Summer of Code (Total - this doesn’t have to be consecutive)? This is my first year 2-3 years More than 3 years If you answered 2-3 years or more than 3 years, what years did you participate? How many years have you been a student in Google Summer of Code? I have never been a student 1 year 2 years 3+ years If you answered 1 year, 2 years, or 3+ years, what years did you participate? At what point did you first make contact with your student? Before the program was announced After my organization was selected to participate in Google Summer of Code During the student application/acceptance phase of the program During the community bonding period After the start of coding How often do you and your student interact? Daily Every few days Weekly Every few weeks Monthly How do you communicate with your student? (Check all that apply) Voice (phone, Skype, etc.) IM/IRC Private emails Mailing Lists Student blog updates In-person meeting(s) Google+ Hangout Other Of the communication methods listed above, which do you use most frequently? How much time do you spend on Google Summer of Code per week (take into consideration your interactions with your student as well as time working with your org and on your own). 1-5 hours 5-10 hours 10-15 hours 15-20 hours More than 20 hours per week How many time zones apart from your student are you? Less than 3 3-6 More than 6 How often do you require status updates from your student? Daily Every few days Weekly Only when explicitly asked for by me Please rate the quality of your interactions and communications with the student (consider his/her communication with you as well as your responses). Very Good (Student is regularly responsive and the quality of communication is high) Good (Student is somewhat responsive and the quality of communication is ok) Bad (Student is only somewhat or not at all responsive and the quality of communication is low) Please rate the quality of the student’s interactions with your organization and community. Very Good (Student is regularly responsive and the quality of communication is high) Good (Student is somewhat responsive and the quality of communication is ok) Bad (Student is only somewhat or not at all responsive and the quality of communication is low) Is your student on track to complete his/her project? The student has already completed his/her project He/she is ahead of schedule He/she is on schedule He/she is behind schedule Please rate the quality of code/work the student has produced thus far. Amongst the best people I’ve ever worked with Solid-quality performance Good performance Mediocre performance Disappointing or not performing at all Give an example of a very good or very bad interaction you had with your student. What one thing could you do to improve the quality of interaction and communications with your student? Anything else you’d like to tell us or suggestions on how we could improve the program? -- You received this message because you are subscribed to the Google Groups "Google Summer of Code Mentors List" group. To post to this group, send email to google-summer-of-code-mentors-list(a)googlegroups.com. To unsubscribe from this group, send email to google-summer-of-code-mentors-list+unsubscribe(a)googlegroups.com. For more options, visit this group at http://groups.google.com/group/google-summer-of-code-mentors-list?hl=en. -- -Austin

1 0

[1/2] comctl32: Fix message sequence when right-clicking a treeview
by Daniel Jelinski 02 Jul '12

02 Jul '12

Resend from last month

1 0

Re: msvcp90: Fix __cdecl functions definitions that returns structures
by Alexandre Julliard 02 Jul '12

02 Jul '12

Piotr Caban <piotr(a)codeweavers.com> writes: > In code produced by gcc callee removes hidden return argument. In code > produced by MS Visual Studio caller is responsible for removing the > hidden argument. That's only for structures larger than 8 bytes, which isn't the case at least for _Getcoll. Some tests would probably be a good idea. -- Alexandre Julliard julliard(a)winehq.org

2 2

Integer overflow problem in string::compare()
by Dan Kegel 02 Jul '12

02 Jul '12

Hi Piotr, could you review the patch in http://bugs.winehq.org/show_bug.cgi?id=31085 ? If you like it, I'll submit it to wine-patches. It passes winetestbot and gets pulsen and stepmania past the splash screen abort. Why are there no tests for msvcp60? If you think it's appropriate, I can add the fix and test for msvcp60 as well. Thanks for all your hard work! - Dan

2 2

Re: [PATCH] wined3d: Add a switch to preserve vertex buffer copy in system memory.
by Henri Verbeet 02 Jul '12

02 Jul '12

In general we don't do hacks like this.

2 1

Re: user32: Add test for ComboBox repositioning behavior (try 2).
by Alexandre Julliard 02 Jul '12

02 Jul '12

Sergey Guralnik <serhio(a)etersoft.ru> writes: > @@ -5807,6 +5814,13 @@ static void test_combobox_messages(void) > log_all_parent_messages--; > ok_sequence(WmKeyDownComboSeq, "WM_KEYDOWN/VK_DOWN on a ComboBox", FALSE); > > + SetWindowLongPtr(combo, GWLP_WNDPROC, (LONG_PTR)combobox_hook_proc_2); > + SendMessage(combo, CB_SETCURSEL, 0, 0); > + SetWindowPos(combo, 0, 0, 0, 120, 130, SWP_NOZORDER|SWP_NOMOVE); > + ret = SendMessage(combo, CB_GETCURSEL, 0, 0); > +todo_wine > + ok(ret == 1, "expected 1, got %ld\n", ret); > + There's no reason to test just WM_SIZE when we have support for testing full sequences. Once again, what are you trying to test? -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [7/7] msxml3: Add support for bin.hex element type encoding
by Marvin 02 Jul '12

02 Jul '12

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

2 1

Re: quartz/tests: Add COM aggregation test for NullRenderer. (2nd try)
by Marvin 02 Jul '12

02 Jul '12

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=19686 Your paranoid android. === WNT4WSSP6 (32 bit misc) === misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL

2 1

Re: [2/5] gdiplus: Add a test for 'value' pointer in the PropertyItem structure.
by Marvin 02 Jul '12

02 Jul '12

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

2 1

Re: [5/5] gdiplus: Add some tests for GdipGetPropertySize and GdipGetAllPropertyItems.
by Marvin 02 Jul '12

02 Jul '12

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

1 0

Re: [PATCH 05/13] quartz/tests: Add COM aggregation test for NullRenderer.
by Marvin 01 Jul '12

01 Jul '12

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=19684 Your paranoid android. === WNT4WSSP6 (32 bit misc) === misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL misc.c:60: Test failed: CoCreateInstance failed with 80040154 misc.c:61: Test failed: pUnkOuter is NULL === W2KPROSP4 (32 bit misc) === misc.c:72: Test failed: CoCreateInstance returned 80040154 misc.c:78: Test failed: CoCreateInstance returned 80040154 misc.c:79: Test failed: pUnkInner is NULL misc.c:72: Test failed: CoCreateInstance returned 80040154 misc.c:78: Test failed: CoCreateInstance returned 80040154 misc.c:79: Test failed: pUnkInner is NULL

1 0

Re: Dmitry Timoshkov : gdiplus: Implement GdipGetPropertyItem.
by Dmitry Timoshkov 01 Jul '12

01 Jul '12

Alexandre Julliard <julliard(a)winehq.org> wrote: > Module: wine > Branch: master > Commit: 670445b22b9ce027bc35c40359499447ef4f8632 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=670445b22b9ce027bc35c4035… > > Author: Dmitry Timoshkov <dmitry(a)baikal.ru> > Date: Thu Jun 28 13:26:49 2012 +0900 > > gdiplus: Implement GdipGetPropertyItem. With this patch committed property reading support should be mostly complete, as well as support for multiframe images (on gdiplus side). -- Dmitry.

1 0

Re: [PATCH 2/2] qedit/tests: Add COM aggregation test for MediaDet.
by Marvin 30 Jun '12

30 Jun '12

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=19677 Your paranoid android. === WNT4WSSP6 (32 bit mediadet) === mediadet.c:149: Test failed: CoCreateInstance failed: 80040154 mediadet: unhandled exception c0000005 at 00401587 === W2KPROSP4 (32 bit mediadet) === mediadet.c:149: Test failed: CoCreateInstance failed: 80040154 mediadet: unhandled exception c0000005 at 00401587

2 1

Release schedule
by Alexandre Julliard 30 Jun '12

30 Jun '12

Folks, Because of various travels and vacations (and just to keep you on your toes ;-), during the month of July I'm planning to make releases on Tuesdays instead of Fridays. So 1.5.8 will be on July 3rd, and then I'll be out for the rest of that week. Subsequent releases should happen on July 17th and 31st, and I'll also be out the first 10 days of August. And if you find this confusing, just take some vacations too ;-) -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: include/basetsd.h: Fix int64 to int truncation warnings when compiling with a 64-bit PSDK compiler.
by Michael Stefaniuc 30 Jun '12

30 Jun '12

Dmitry, afair the fix is to disable the int truncation warnings in MSVC. For whatever reason they seem to enable those bogus warnings. bye michael On 06/28/2012 12:12 PM, Dmitry Timoshkov wrote: > --- > include/basetsd.h | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/include/basetsd.h b/include/basetsd.h > index 698dd5c..4a59897 100644 > --- a/include/basetsd.h > +++ b/include/basetsd.h > @@ -153,12 +153,12 @@ typedef unsigned int UHALF_PTR, *PUHALF_PTR; > > static inline ULONG32 HandleToULong(const void *h) > { > - return (ULONG_PTR)h; > + return (ULONG32)(ULONG_PTR)h; > } > > static inline LONG32 HandleToLong(const void *h) > { > - return (LONG_PTR)h; > + return (LONG32)(LONG_PTR)h; > } > > static inline void *ULongToHandle(ULONG32 ul) > @@ -173,32 +173,32 @@ static inline void *LongToHandle(LONG32 l) > > static inline ULONG32 PtrToUlong(const void *p) > { > - return (ULONG_PTR)p; > + return (ULONG32)(ULONG_PTR)p; > } > > static inline LONG32 PtrToLong(const void *p) > { > - return (LONG_PTR)p; > + return (LONG32)(LONG_PTR)p; > } > > static inline UINT32 PtrToUint(const void *p) > { > - return (UINT_PTR)p; > + return (UINT32)(UINT_PTR)p; > } > > static inline INT32 PtrToInt(const void *p) > { > - return (INT_PTR)p; > + return (INT32)(INT_PTR)p; > } > > static inline UINT16 PtrToUshort(const void *p) > { > - return (ULONG_PTR)p; > + return (UINT16)(ULONG_PTR)p; > } > > static inline INT16 PtrToShort(const void *p) > { > - return (LONG_PTR)p; > + return (INT16)(LONG_PTR)p; > } > > static inline void *IntToPtr(INT32 i)

3 5

Re: windowscodecs: test for TIFFSetWarningHandlerExt
by Alexandre Julliard 29 Jun '12

29 Jun '12

Daniel Lehman <dlehman(a)esri.com> writes: > Fails to build on RHEL4 with libtiff 3.6.1 > > Alternative to this is a configure test for libtiff 3.8.0, which is when the function first appears I don't think this needs a configure test. Just define the prototype explicitly instead of using typeof. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [1/5] d3dx9: Implement D3DXLoadVolumeFromMemory.
by Matteo Bruni 29 Jun '12

29 Jun '12

2012/6/27 Józef Kucia <joseph.kucia(a)gmail.com>: > --- Hi Józef, > +void copy_simple_data(const BYTE *src, UINT srcpitch, SIZE src_size, const PixelFormatDesc *srcformat, > + BYTE *dest, UINT destpitch, SIZE dst_size, const PixelFormatDesc *destformat, D3DCOLOR colorkey) DECLSPEC_HIDDEN; > + You are not actually using this function in the patch. > + if (dst_box->Left > dst_box->Right || dst_box->Right > desc.Width) > + return D3DERR_INVALIDCALL; > + if (dst_box->Top > dst_box->Bottom || dst_box->Bottom > desc.Height) > + return D3DERR_INVALIDCALL; > + if (dst_box->Front > dst_box->Back || dst_box->Back > desc.Depth) > + return D3DERR_INVALIDCALL; > + > + dst_width = dst_box->Right - dst_box->Left; > + dst_height = dst_box->Bottom - dst_box->Top; > + dst_depth = dst_box->Back - dst_box->Front; > + if (!dst_width || !dst_height || !dst_depth) > + return D3DERR_INVALIDCALL; You can avoid the last check by comparing e.g. (dst_box->Left >= dst_box->Right) above, instead of using ">". > + if (src_box->Left & (src_format_desc->block_width - 1) > + || src_box->Top & (src_format_desc->block_height - 1) > + || (src_box->Right & (src_format_desc->block_width - 1) > + && src_width != desc.Width) > + || (src_box->Bottom & (src_format_desc->block_height - 1) > + && src_height != desc.Height)) > + { > + WARN("Source box (%u, %u, %u, %u) is misaligned\n", > + src_box->Left, src_box->Top, src_box->Right, src_box->Bottom); > + return D3DXERR_INVALIDDATA; > + } I'm not quite sure of the right and bottom misalignment checks. If the tests confirm that, of course that's okay. > + for (slice = 0; slice < src_depth; slice++) > + { > + src_addr = src_memory; > + src_addr += slice * src_slice_pitch; Shouldn't you take into account src_box->Front?

2 1

Re: user32: Add test for ComboBox repositioning behavior (try 2).
by Marvin 29 Jun '12

29 Jun '12

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=19658 Your paranoid android. === W7PRO (32 bit msg) === msg.c:8219: Test failed: did not get zero count as expected (1).

2 1

Re: gdiplus: Add a test to show that GdipCloneImage is not supposed to increase refcount of the source image stream, but clone it instead.
by Vincent Povirk 29 Jun '12

29 Jun '12

I think it's most likely not keeping the input stream in the new image at all. It's probably also worth testing SelectActiveFrame, GetPropertyItem, and GetImageRawFormat with clones of images. Maybe they don't work.

2 1

Re: wbemprox: Add a Win32_LogicalDisk class stub.
by Hans Leidekker 28 Jun '12

28 Jun '12

On Thu, 2012-06-28 at 21:53 +0300, John Yani wrote: > +static void fill_logicaldisk( struct table *table ) > +{ > + static const WCHAR caption[] = > + {'C',':',0}; > + static const WCHAR description[] = > + {'L','o','c','a','l',' ','F','i','x','e','d',' ','D','i','s','k',0}; > + static const WCHAR device_id[] = > + {'C',':',0}; > + struct record_logicaldisk *rec; > + FIXME("returns a fake logical disks table\n"); > + if (!(table->data = heap_alloc( sizeof(*rec) ))) > + { > + return; > + } > + rec = (struct record_logicaldisk *)(table->data); > + rec->caption = caption; > + rec->description = description; > + rec->device_id = device_id; > + table->num_rows = 1; > + TRACE("created %u row(s)\n", table->num_rows); That's better. Does NFS actually query the Caption and Description properties?

2 4

Re: user32: Add a test for ComboBox WM_WINDOWPOSCHANGED handling.
by Alexandre Julliard 28 Jun '12

28 Jun '12

Sergey Guralnik <serhio(a)etersoft.ru> writes: > @@ -5807,6 +5830,25 @@ static void test_combobox_messages(void) > log_all_parent_messages--; > ok_sequence(WmKeyDownComboSeq, "WM_KEYDOWN/VK_DOWN on a ComboBox", FALSE); > > + wp.hwnd = combo; > + wp.hwndInsertAfter = HWND_TOP; > + wp.cx = wp.cy = wp.x = wp.y = 50; > + wp.flags = 0; > + flush_sequence(); > + SendMessage(combo, WM_WINDOWPOSCHANGED, 0, (LPARAM)&wp); It doesn't make much sense to send this message directly. What are you trying to test? -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/2] gdiplus: Increase refcount of the source stream when loading a bitmap instead of cloning it.
by Vincent Povirk 28 Jun '12

28 Jun '12

This looks good to me. (I didn't think it was necessary to comment, but this is an important fix and I was surprised it was still New.)

1 0

Unused msvcp60 functions
by Francois Gouget 28 Jun '12

28 Jun '12

I get the following warnings when compiling msvcp60: main.c:108:51: warning: ‘std_BADOFF_func’ defined but not used [-Wunused-function] misc.c:66:40: warning: ‘mutex_mutex_lock’ defined but not used [-Wunused-function] misc.c:73:40: warning: ‘mutex_mutex_unlock’ defined but not used [-Wunused-function] misc.c:80:40: warning: ‘mutex_mutex_ctor’ defined but not used [-Wunused-function] misc.c:87:40: warning: ‘mutex_mutex_dtor’ defined but not used [-Wunused-function] misc.c:175:58: warning: ‘set_new_handler_reset’ defined but not used [-Wunused-function] Indeed msvcp60.spec contains no _BADOFF, _Mutex or set_new_handler function. I don't see them in my 'database' of exported Windows APIs for msvcp60 but maybe the Windows dlls I built it from where not entirely up to date. Still it looks like these were added in later revisions and thus can be removed from the above files. misc.c:127:50: warning: ‘wctype’ defined but not used [-Wunused-function] This one is actually forwarded to msvcp90 according to msvcp60.spec, thus making this implementation redundant. So it should either be removed or msvcp60.spec should be modified. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ "Only wimps use tape backup: _real_ men just upload their important stuff on ftp, and let the rest of the world mirror it ;)" -- Linus Torvalds

2 3

Re: windowscodecs: Make sure that stream is not reused once the decoder is initialized.
by Vincent Povirk 28 Jun '12

28 Jun '12

Looks good to me.

1 0

Re: kernel32/tests: Remove an unused variable in the sync tests.
by Marvin 28 Jun '12

28 Jun '12

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

1 0

Re: [4/5] d3dx9: Implement D3DXCreateVolumeTextureFromFileInMemoryEx.
by Matteo Bruni 28 Jun '12

28 Jun '12

2012/6/27 Józef Kucia <joseph.kucia(a)gmail.com>: > --- > + for (mip_level = 0; mip_level < mip_levels; mip_level++) > + { > + hr = calculate_dds_surface_size(src_info, width, height, &src_pitch, &mip_level_size); > + if (FAILED(hr)) return hr; ... > + hr = D3DXLoadVolumeFromMemory(volume, palette, NULL, pixels, src_info->Format, > + src_pitch, mip_level_size, NULL, &box, filter, color_key); mip_level_size would be the source slice pitch, right? I'd just call it like that.

1 0

Re: [2/2] msvcp90: Sync spec files
by Owen Rudge 28 Jun '12

28 Jun '12

On 28/06/2012 11:33, Owen Rudge wrote: > --- > dlls/msvcp60/msvcp60.spec | 638 > ++++++++++++++++++++++---------------------- Sorry, I ran the make_specfiles script without realising that msvcp60.spec shouldn't have been updated. I've resent the patch without these changes. -- Owen Rudge http://www.owenrudge.net/

1 0

Re: [2/2] msvcp90: Sync spec files
by Piotr Caban 28 Jun '12

28 Jun '12

First patch looks correct. On 06/28/12 12:33, Owen Rudge wrote: > dlls/msvcp60/msvcp60.spec | 638 > ++++++++++++++++++++++---------------------- You can't update msvcp60.spec file this way (most of functions there should not be forwarded to msvcp90).

1 0

Re: po: add missing url tag to nl.po
by Francois Gouget 28 Jun '12

28 Jun '12

On Wed, 27 Jun 2012, Austin English wrote: > http://bugs.winehq.org/show_bug.cgi?id=31038 "Noot: Het wordt aanbevolen om distributiepakketten te gebruiken. Zie <a " -"href=http://wiki.winehq.org/Gecko voor details." +"href=http://wiki.winehq.org/Gecko</a> voor details." Note that this is missing a lot more than just </a>. This should be: "Noot: Het wordt aanbevolen om distributiepakketten te gebruiken. Zie <a " "href=\"http://wiki.winehq.org/Gecko\">http://wiki.winehq.org/Gecko</a> voor details." Fortunately it looks like nl is the only such case. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ A polar bear is a cartesian bear after a coordinate transform.

1 0

Re: [1/5] windowscodecs: Implement MetadataHandler_GetValue. Take 2. Resend.
by Marvin 28 Jun '12

28 Jun '12

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=19631 Your paranoid android. === WVISTAADM (32 bit metadata) === Failure running script in VM: The specified guest user must be logged in interactively to perform this operation

1 0

Re: [4/5] gdiplus: Add some tests for GdipGetPropertyItemSize and GdipGetPropertyItem. Take 2.
by Marvin 28 Jun '12

28 Jun '12

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=19625 Your paranoid android. === W7PROX64 (64 bit image) === image.c:2817: Test failed: 0: expected property size 17 or 36, got 44 image.c:2817: Test failed: 2: expected property size 144 or 0, got 152 image.c:2817: Test failed: 3: expected property size 20 or 0, got 28 === TEST64_W7SP1 (64 bit image) === image.c:2817: Test failed: 0: expected property size 17 or 36, got 44 image.c:2817: Test failed: 2: expected property size 144 or 0, got 152 image.c:2817: Test failed: 3: expected property size 20 or 0, got 28

2 1

Re: [5/5] gdiplus: Add a bunch of tests for TIFF image properties. Resend.
by Marvin 28 Jun '12

28 Jun '12

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=19626 Your paranoid android. === W7PROX64 (64 bit image) === image.c:2818: Test failed: 0: expected property size 17 or 36, got 44 image.c:2818: Test failed: 2: expected property size 144 or 0, got 152 image.c:2818: Test failed: 3: expected property size 20 or 0, got 28 === TEST64_W7SP1 (64 bit image) === image.c:2818: Test failed: 0: expected property size 17 or 36, got 44 image.c:2818: Test failed: 2: expected property size 144 or 0, got 152 image.c:2818: Test failed: 3: expected property size 20 or 0, got 28

1 0

Re: [3/4] gdiplus: Add some tests for GdipGetPropertyItemSize and GdipGetPropertyItem.
by Marvin 28 Jun '12

28 Jun '12

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=19594 Your paranoid android. === W7PROX64 (64 bit image) === image.c:2811: Test failed: 0: expected property size 17 or 36, got 44 image.c:2811: Test failed: 2: expected property size 144 or 0, got 152 image.c:2811: Test failed: 3: expected property size 20 or 0, got 28 === TEST64_W7SP1 (64 bit image) === image.c:2811: Test failed: 0: expected property size 17 or 36, got 44 image.c:2811: Test failed: 2: expected property size 144 or 0, got 152 image.c:2811: Test failed: 3: expected property size 20 or 0, got 28

3 4

Re: wbemprox: Add a Win32_LogicalDisk class stub.
by Hans Leidekker 27 Jun '12

27 Jun '12

On Wed, 2012-06-27 at 14:41 +0300, John Yani wrote: > +static void fill_logicaldisk( struct table *table ) > +{ > + static const WCHAR device_id[] = {'0',0}; > + struct record_logicaldisk *rec; > + FIXME("returns a fake logical disks table\n"); > + if (!(table->data = heap_alloc( sizeof(*rec) ))) > + { > + return; > + } > + rec = (struct record_logicaldisk *)(table->data); > + rec->device_id = heap_strdupW( device_id ); > + table->num_rows = 1; > + TRACE("created %u rows\n", table->num_rows); > +} There's no need to create a copy of device_id. Just assign it to rec->device_id and remove COL_FLAG_DYNAMIC from the column definition.

1 0

ODBC on 64bit system
by Daniel Jelinski 27 Jun '12

27 Jun '12

Hello all, Is it possible to run a 32bit application using 64bit ODBC drivers? I already checked that standard 32bit wine compilation does not work - but will it work with WoW64? I am trying to get an application to connect to SQL Server via tdsodbc, but unfortunately tdsodbc:i386 in ubuntu 11.10 is broken... Regards, Daniel

1 0

Re: [1/5] windowscodecs: Implement IWICBitmapScaler::GetSize.
by Dmitry Timoshkov 27 Jun '12

27 Jun '12

Vincent Povirk <madewokherd(a)gmail.com> wrote: > static HRESULT WINAPI BitmapScaler_GetSize(IWICBitmapScaler *iface, > UINT *puiWidth, UINT *puiHeight) > { > - FIXME("(%p,%p,%p): stub\n", iface, puiWidth, puiHeight); > + BitmapScaler *This = impl_from_IWICBitmapScaler(iface); > + TRACE("(%p,%p,%p)\n", iface, puiWidth, puiHeight); > > - return E_NOTIMPL; > + if (!This->source) > + return WINCODEC_ERR_WRONGSTATE; > + > + *puiWidth = This->width; > + *puiHeight = This->height; > + > + return S_OK; This patch and remaning patches in the sequence completely miss any parameter checking. -- Dmitry.

2 1

Re: d3dx9_36 [patch 2 of 3]: Implementation of D3DXSHScale
by Matteo Bruni 27 Jun '12

27 Jun '12

2012/6/27 Nozomi Kodama <nozomi.kodama(a)yahoo.com>: > -@ stub D3DXSHScale(ptr long ptr ptr) +@ stdcall D3DXSHScale(ptr long ptr ptr) ... +FLOAT* WINAPI D3DXSHScale(FLOAT *out, UINT order, CONST FLOAT *a, CONST FLOAT scale) The last parameter doesn't match between the function prototype and the spec entry. Also, apparently MSDN agrees with your prototype, while the docs in the last DirectX SDK have a pointer there. I guess the tests demonstrate that the parameter is actually a plain float... + for (i = 0; i < 100; i++) + { + a[i] = (FLOAT)i; + b[i] = (FLOAT)i; + } No need to cast. + D3DXSHScale(b, order, a, 5.0f); Not important, but you could also verify that the returned pointer is == b. +/* D3DXSHScale does not modify the elements of the array after the order * order-th element */ Can you please indent those comments? + return; +} That's not needed at all. Also, while you're at it, you should probably merge the two for loops in the test, like Francois did in http://www.winehq.org/pipermail/wine-patches/2012-June/115464.html

1 0

Re: wbemprox: Add a Win32_LogicalDisk class stub.
by Hans Leidekker 27 Jun '12

27 Jun '12

On Wed, 2012-06-27 at 11:34 +0300, John Yani wrote: > +static void fill_logicaldisk( struct table *table ) > +{ > + static const WCHAR fmtW[] = {'%','u',0}; > + WCHAR device_id[11]; > + struct record_logicaldisk *rec; > + UINT num_rows = 1; > + FIXME("returns a fake logical disks table\n"); > + if (!(table->data = heap_alloc( sizeof(*rec) * num_rows ))) > + { > + return; > + } > + rec = (struct record_networkadapter *)(table->data); This should be (struct record_logicaldisk *)(table->data); > + sprintfW( device_id, fmtW, 0 ); > + rec->device_id = heap_strdupW( device_id ); > + > + TRACE("created %u rows\n", num_rows); > + table->num_rows = num_rows; > +} You don't need the num_rows variable since it's a fixed value. You also don't need to build the device id, it can just be a static string.

1 0

Re: [4/4] gdiplus: Add a bunch of tests for TIFF image properties.
by Marvin 27 Jun '12

27 Jun '12

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=19595 Your paranoid android. === W7PROX64 (64 bit image) === image.c:2812: Test failed: 0: expected property size 17 or 36, got 44 image.c:2812: Test failed: 2: expected property size 144 or 0, got 152 image.c:2812: Test failed: 3: expected property size 20 or 0, got 28 === TEST64_W7SP1 (64 bit image) === image.c:2812: Test failed: 0: expected property size 17 or 36, got 44 image.c:2812: Test failed: 2: expected property size 144 or 0, got 152 image.c:2812: Test failed: 3: expected property size 20 or 0, got 28

1 0

Re: d3dx9_36 [patch 3 of 3]: Implementation of D3DXSHMultiply2
by Marvin 27 Jun '12

27 Jun '12

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

1 0

Re: d3dx9_36 [patch 2 of 3]: Implementation of D3DXSHScale
by Marvin 27 Jun '12

27 Jun '12

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

1 0

Re: [PATCH 4/4] qedit: Support COM aggregation for SampleGrabber.
by Marvin 26 Jun '12

26 Jun '12

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=19578 Your paranoid android. === WNT4WSSP6 (32 bit mediadet) === mediadet.c:148: Test failed: CoCreateInstance failed: 80040154 mediadet.c:356: Test failed: CoCreateInstance failed: 80040154 mediadet: unhandled exception c0000005 at 00401639 === W2KPROSP4 (32 bit mediadet) === mediadet.c:148: Test failed: CoCreateInstance failed: 80040154 mediadet.c:356: Test failed: CoCreateInstance failed: 80040154 mediadet: unhandled exception c0000005 at 00401639

2 1

Re: [PATCH 3/4] qedit/tests: Add COM aggregation test for MediaDet.
by Michael Stefaniuc 26 Jun '12

26 Jun '12

Argh! Please skip this patch, it is incomplete and just skips the rest of the tests on Wine. I'll resubmit it when I implement COM aggregation for MediaDet. Patch 4/4 can be still applied as it doesn't depends on this. bye michael On 06/26/2012 11:53 PM, Michael Stefaniuc wrote: > --- > dlls/qedit/tests/mediadet.c | 7 +++++++ > 1 files changed, 7 insertions(+), 0 deletions(-) > > diff --git a/dlls/qedit/tests/mediadet.c b/dlls/qedit/tests/mediadet.c > index 1d9a2f1..1d75680 100644 > --- a/dlls/qedit/tests/mediadet.c > +++ b/dlls/qedit/tests/mediadet.c > @@ -132,6 +132,7 @@ static BOOL init_tests(void) > static void test_mediadet(void) > { > HRESULT hr; > + struct unk_impl unk_obj = {{&unk_vtbl}, 19, NULL}; > IMediaDet *pM = NULL; > BSTR filename = NULL; > LONG nstrms = 0; > @@ -141,6 +142,12 @@ static void test_mediadet(void) > int flags; > int i; > > + /* COM aggregation */ > + hr = CoCreateInstance(&CLSID_MediaDet, &unk_obj.IUnknown_iface, CLSCTX_INPROC_SERVER, > + &IID_IUnknown, (void**)&unk_obj.inner_unk); > + todo_wine ok(hr == S_OK, "CoCreateInstance failed: %08x\n", hr); > + if (hr != S_OK) return; > + > /* test.avi has one video stream. */ > hr = CoCreateInstance(&CLSID_MediaDet, NULL, CLSCTX_INPROC_SERVER, > &IID_IMediaDet, (LPVOID*)&pM);

1 0

Re: [PATCH 3/4] qedit/tests: Add COM aggregation test for MediaDet.
by Marvin 26 Jun '12

26 Jun '12

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=19577 Your paranoid android. === WNT4WSSP6 (32 bit mediadet) === mediadet.c:148: Test failed: CoCreateInstance failed: 80040154 mediadet.c:356: Test failed: CoCreateInstance failed: 80040154 === W2KPROSP4 (32 bit mediadet) === mediadet.c:148: Test failed: CoCreateInstance failed: 80040154 mediadet.c:356: Test failed: CoCreateInstance failed: 80040154

1 0

Re: kernel32: Change LCMapStringW to LCMapStringEx and call it from LCMapStringW (try 2)
by Alexandre Julliard 26 Jun '12

26 Jun '12

André Hentschel <nerv(a)dawncrow.de> writes: > @@ -2625,6 +2641,28 @@ INT WINAPI LCMapStringW(LCID lcid, DWORD flags, LPCWSTR src, INT srclen, > } > > /************************************************************************* > + * LCMapStringW (KERNEL32.@) > + * > + * See LCMapStringA. > + */ > +INT WINAPI LCMapStringW(LCID lcid, DWORD flags, LPCWSTR src, INT srclen, > + LPWSTR dst, INT dstlen) > +{ > + WCHAR locale[LOCALE_NAME_MAX_LENGTH]; > + > + TRACE("(0x%04x,0x%08x,%s,%d,%p,%d)\n", > + lcid, flags, debugstr_wn(src, srclen), srclen, dst, dstlen); > + > + if (!LCIDToLocaleName(lcid, locale, LOCALE_NAME_MAX_LENGTH, 0)) > + { > + SetLastError(ERROR_INVALID_PARAMETER); > + return 0; > + } I don't think it makes sense to do name conversion for a parameter that isn't used. If we use it someday it will most likely be to look up a conversion table directly from the lcid. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: windowscodecs: Implement MetadataHandler_GetValue.
by Vincent Povirk 26 Jun '12

26 Jun '12

> + case VT_I8: > + case VT_UI8: > + case VT_R8: > + return v1->u.uhVal.QuadPart - v2->u.uhVal.QuadPart; Won't this only compare the low part on x86?

2 1

Re: [PATCH 3/5] secur32: Handle incomplete messages in schan_InitializeSecurityContextW().
by Marvin 26 Jun '12

26 Jun '12

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=19560 Your paranoid android. === WNT4WSSP6 (32 bit schannel) === schannel.c:702: Test failed: Got unexpected status 0x80090308.

1 0

Re: urlmon: Do not stop binding when OnDataAvailable() returns E_PENDING
by Jacek Caban 26 Jun '12

26 Jun '12

On 06/26/12 00:02, Andrew Talbot wrote: > Changelog: > urlmon: Do not stop binding when OnDataAvailable() returns E_PENDING. > > diff --git a/dlls/urlmon/binding.c b/dlls/urlmon/binding.c > index 58aed20..42eca4b 100644 > --- a/dlls/urlmon/binding.c > +++ b/dlls/urlmon/binding.c > @@ -1097,7 +1097,7 @@ static void report_data(Binding *This, DWORD bscf, ULONG progress, ULONG progres > > hres = IBindStatusCallback_OnDataAvailable(This->callback, bscf, progress, > &formatetc, &stgmed); > - if(hres != S_OK) { > + if(FAILED(hres) && hres != E_PENDING) { > if(This->download_state != END_DOWNLOAD) { > This->download_state = END_DOWNLOAD; > IBindStatusCallback_OnProgress(This->callback, progress, progress_max, Please add a test case. You may modify one of existing binding tests to use E_PENDING. Jacek

1 0

Is it safe yet for developers to upgrade to Ubuntu 12.04?
by Erich E. Hoover 26 Jun '12

26 Jun '12

I know that for a while there were some packaging problems that meant that upgrading to 12.04 made compiling 32-bit Wine on a 64-bit PC extremely difficult. Is this still the case? I'd like to upgrade to the new OS version, but I'm not willing to sacrifice the ability to build Wine in order to do so. Thanks in advance! Erich Hoover ehoover(a)mines.edu

6 5

Re: [PATCH] opengl32: Disable wglGetProcAddress for core GL 1.0/1.1 functions
by Marvin 26 Jun '12

26 Jun '12

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

2 1

change wording of ok()-messages in winspool-tests?
by Julian Rüger 26 Jun '12

26 Jun '12

Hi Francois and list, [/nitpick-mode on] I'm preparing a patch to fix typos/spelling/grammar in dlls/winspool.drv/tests/info.c, where I stumbled across repeated messages like ok( !retval, "function result wrong! False expected\n"); or ok( size == exact, "Parameter size wrong! %d expected got %d\n", exact, size); Do you think it's worth changing these? (I think "Wrong function result: 'false' expected\n", and "Wrong parameter size: expected %d, got %d\n" would be better) Do we have guidelines for strings like these? For example: start with a captial letter, no punctuation at the end... What tests have "good" style I can use as reference? Best regards, Julian PS: Am I bike shedding here? Does anyone care about stuff like this? Sorry for wasting your time ;)

3 3

Building WoW64 packages
by Hilko Bengen 25 Jun '12

25 Jun '12

Hi, after the effort on the Debian wine packages has progressed to a point where it's likely that we'll release wheezy with a stable wine version that is not completely out of date, I'd like to take things a step further. I recently spent some time looking at ways how we could build Debian packages for a shared WoW64 setup similar to what is described in http://wiki.winehq.org/Wine64. We will have to use separate build environments because having 32-bit and 64-bit -dev packages installed at the same time is not possible. Installing a 64-bit package that contains the build tools into a 32-bit environment should be possible, though. >From looking at the build system, things happen in a different way when I run configure for the 32-bit components with --wine64=...: 1. Manpages are not installed (WOW64_DISABLE) 2. --disable-fonts is implied 3. --disable-server is implied, so wineserver is not built. (The 64-bit wineserver is used instead, right?) 4. --disable-tools is implied because the existing build tools are being used. Did I miss anything? Leaving parts out is easy to understand, but how do the build tools that have been built for the x86_64 architecture affect the 32-bit components that are being built with them? Are different paths encoded into libwine? Cheers, -Hilko

2 1

Re: [PATCH 2/2] opengl32: Check for valid context in wglGetProcAddress
by Chris Robinson 25 Jun '12

25 Jun '12

On Sunday, June 24, 2012 10:42:11 PM Roderick Colenbrander wrote: > @@ -797,7 +797,7 @@ static void test_getprocaddress(HDC hdc) > /* Temporarily disable the context, so we can see that we can't > retrieve functions now. */ wglMakeCurrent(hdc, NULL); > func = wglGetProcAddress("glActiveTextureARB"); > - todo_wine ok(func == NULL, "Function lookup without a context passed, > expected a failure; last error %#x\n", GetLastError()); > + ok(func == NULL, "Function lookup without a context passed, expected a > failure; last error %#x\n", GetLastError()); > wglMakeCurrent(hdc, ctx); > } Does this apply to all functions, or just the ones MS's DLL doesn't implement? I could imagine wlgGetProcAddress still returning functions that are in opengl32.dll without worrying about the driver.

2 1

Re: [3/5] gdiplus: Implement GdipGetPropertyIdList.
by Nikolay Sivov 25 Jun '12

25 Jun '12

On 6/25/2012 14:30, Dmitry Timoshkov wrote: > + > + return hr == S_OK ? Ok : hresult_to_status(hr); > } > No need for special case for S_OK here.

1 0

Re: [3/3] d3drm: Implement IDirect3DRMMeshBuilderX_GetClassName
by Marvin 24 Jun '12

24 Jun '12

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

1 0

Re: [2/3] d3drm: Implement IDirect3DRMLight_GetClassName
by Marvin 24 Jun '12

24 Jun '12

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

1 0

Incompatibility in Kernel32
by robert.van.herk＠serioustoys.com 23 Jun '12

23 Jun '12

Hi group, As I was trying write a fix for msi.dll, I stumbled upon an incompatibility in the kernel. If you open an existing file, FILE_SHARE_DELETE doesn't seem to work in Wine. In Windows, it does. Attached is a minimal test case. I feel hesitation towards starting to hack in kernel32 myself. Though this bug seems to be blocking me to implement msi.dll completely compatible with the MS implementation... Shall I file a bug report? Regards, Robert

4 6

MSVCP60.dll
by John Emmas 22 Jun '12

22 Jun '12

Firstly, I'm not a Linux user. I'm a Windows programmer but I have a passing knowledge of Linux (and several friends who are Linux programmers). I write a Windows application which gets launched as a child process by a popular Linux DAW. My program was first written many years ago when the (then) current version of Wine was about v0.9.58. Over a hundred people are using my program with old versions of Wine and I've had no complaints so far. Recently however, two customers tried to use it with Wine v1.5.35. In both cases the program crashed - apparently because some particular function wasn't found in MSVCP60.dll (at the moment, we don't know which function). Both customers went to a web site called WineTricks from where they were able to install an alternative version of MSVCP60.dll. My app now gets past the place where it previously crashed although it now crashes further on... :-( It looks as if the MSVC6 runtime module(s) that now come with Wine aren't the same as the ones from v0.9.58. Either they're different versions of the Microsoft DLLs or maybe they're now being implemented by somebody else, perhaps as part of Wine development? Either way, they don't work with my particular app any more. I still have Visual C++6 installed on an old computer, together with both the source code and the (InstallShield) packager for my original app. So is there anything I can do to make this work? For example, do I need to link against a particular version of the MSVC6 runtime? Or do I need to configure my packager so that it specifically distributes the runtime modules that my app linked to? I never did that in the past - because whichever runtime came with Wine seemed to work. Any advice? John

4 7

Re: [1/2] windowscodecs: Implement GetPreview and GetThumbnail in the TIFF decoder.
by Vincent Povirk 22 Jun '12

22 Jun '12

According to MSDN, GetPreview isn't supported on any decoder except WMPhoto, and if that's true it should probably be returning WINCODEC_ERR_UNSUPPORTEDOPERATION. And why bother with GetThumbnail if you're not going to do it properly? At least check whether a thumbnail exists in the image before returning something.

2 1

Re: d3dx9_36: Implentation of D3DXSHDot
by Alexandre Julliard 22 Jun '12

22 Jun '12

Nozomi Kodama <nozomi.kodama(a)yahoo.com> writes: > +static void test_D3DXSHDot(void) > +{ > + unsigned int i; > + FLOAT a[64], b[64], got; > + CONST FLOAT expected[] = > + { 0.0f, 0.0f, 17.0f, 222.0f, 1300.0f, 5050.0f, 15225.0f, 38612.0f, 86352.0f, }; > + > + for (i = 0; i < 64; i++) > + { > + a[i] = (FLOAT)i; > + b[i] = (FLOAT)i + 0.5f; > + } You should start with a non-zero value to tests orders 0 and 1 properly. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: d3dx9_36 [patch 4 of 4, try 2]: Save multiplications
by Alexandre Julliard 22 Jun '12

22 Jun '12

Nozomi Kodama <nozomi.kodama(a)yahoo.com> writes: > @@ -1971,11 +1971,13 @@ FLOAT *WINAPI D3DXFloat16To32Array(FLOAT *pout, CONST D3DXFLOAT16 *pin, UINT n) > > FLOAT* WINAPI D3DXSHAdd(FLOAT *out, UINT order, const FLOAT *a, const FLOAT *b) > { > - UINT i; > + UINT i, sq_order; > > TRACE("out %p, order %u, a %p, b %p\n", out, order, a, b); > > - for (i = 0; i < order * order; i++) > + sq_order = order * order; > + > + for (i = 0; i < sq_order; i++) > out[i] = a[i] + b[i]; There's no reason to do that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: wintrust/tests: Fix build with MSVC.
by Dmitry Timoshkov 22 Jun '12

22 Jun '12

Thomas Faber <thfabba(a)gmx.de> wrote: > FIELD_OFFSET isn't constant "enough". The problem is not with FIELD_OFFSET, but with applying shift and mask operations to a constant. -- Dmitry.

3 3

Re: d3dx9_36 [patch 1 of 4, try 2]: Implemetation of D3DXSHDot
by Dmitry Timoshkov 22 Jun '12

22 Jun '12

Hello, Please follow http://wiki.winehq.org/SubmittingPatches instructions for naming patches when sending a patch series. Usually 'git format-patch -n' will do it for you. Otherwise testbot won't recognize the series. -- Dmitry.

1 0

Re: d3dx9_36 [patch 3 of 4, try 2]: Implemetation of D3DXSHMultiply2
by Marvin 22 Jun '12

22 Jun '12

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

1 0

Re: d3dx9_36 [patch 1 of 4, try 2]: Implemetation of D3DXSHScale
by Marvin 22 Jun '12

22 Jun '12

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

1 0

re: [PATCH 2/5] ddraw: Use wined3d_get_adapter_display_mode() in CreateSurface().
by Dan Kegel 21 Jun '12

21 Jun '12

Henri wrote: - hr = wined3d_device_get_display_mode(ddraw->wined3d_device, 0, &mode); - if (FAILED(hr)) + if (FAILED(hr = wined3d_get_adapter_display_mode(ddraw->wined3d, WINED3DADAPTER_DEFAULT, &mode))) Seems like a step back in readability to combine setting and testing hr. - Dan

1 0

Examples of AJ silently improving patches?
by Dan Kegel 21 Jun '12

21 Jun '12

AJ sometimes kindly fixes problems in patches while committing them (I guess it's easier than rejecting the patch for small issues). I wonder if these silent improvements would be worth collecting as a sort of style-guide-by-example. Anyone remember any good examples? I've started a list at http://wiki.winehq.org/StyleExamples

4 3

Re: [PATCH 1/7] ntdll: Add support for junction point creation.
by Hans Leidekker 21 Jun '12

21 Jun '12

On Wed, 2012-06-20 at 17:26 -0600, Erich E. Hoover wrote: > + TRACE("Linking %s to %s\n", unix_src.Buffer, unix_dest.Buffer); > + if (rmdir( unix_src.Buffer ) < 0) > + { > + status = FILE_GetNtStatus(); > + goto cleanup; > + } > + if (symlink( unix_dest.Buffer, unix_src.Buffer ) < 0) > + { > + status = FILE_GetNtStatus(); > + goto cleanup; > + } > + status = STATUS_SUCCESS; This is not an atomic operation since you need two Unix calls. So you would need locking or rollback to deal with possible races. Should permissions and ownership be preserved on the directory when a junction is removed?

2 2

[1/5] windowscodecs: Implement PixelFormatInfo_GetBitsPerPixel.
by Vincent Povirk 21 Jun '12

21 Jun '12

1 0

Re: kernel32: Add a test for deleting an opened for reading file. Take 2.
by Eric Pouech 21 Jun '12

21 Jun '12

one tricky question... could it be that on windows this test behaves differently on ntfs vs FAT filesystem ? or what would give re-opening the file after the deletefile call ? A+ 2012/6/21, Dmitry Timoshkov <dmitry(a)baikal.ru>: > Previous version had a typo in file name. > --- > dlls/kernel32/tests/file.c | 26 ++++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c > index 8768b99..0a662d9 100644 > --- a/dlls/kernel32/tests/file.c > +++ b/dlls/kernel32/tests/file.c > @@ -3236,6 +3236,31 @@ static void test_CreatFile(void) > > DeleteFile(file_name); > } > +static void test_delete_file(void) > +{ > + char temp_path[MAX_PATH], temp_file[MAX_PATH]; > + HANDLE hfile; > + int ret; > + > + GetTempPathA(MAX_PATH, temp_path); > + GetTempFileName(temp_path, "tst", 0, temp_file); > + > + SetLastError(0xdeadbeef); > + hfile = CreateFile(temp_file, GENERIC_READ, FILE_SHARE_DELETE | > FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, 0); > + ok(hfile != INVALID_HANDLE_VALUE, "CreateFile error %d\n", > GetLastError()); > + > + SetLastError(0xdeadbeef); > + ret = DeleteFile(temp_file); > +todo_wine > + ok(ret, "DeleteFile error %d\n", GetLastError()); > + > + SetLastError(0xdeadbeef); > + ret = CloseHandle(hfile); > + ok(ret, "CloseHandle error %d\n", GetLastError()); > + ret = DeleteFile(temp_file); > +todo_wine > + ok(!ret, "DeleteFile should fail\n"); > +} > > START_TEST(file) > { > @@ -3257,6 +3282,7 @@ START_TEST(file) > test_CreateFileW(); > test_DeleteFileA(); > test_DeleteFileW(); > + test_delete_file(); > test_MoveFileA(); > test_MoveFileW(); > test_FindFirstFileA(); > -- > 1.7.11 > > > > -- -- Eric Pouech

1 0

Re: wintrust/tests: Fix build with MSVC.
by Dmitry Timoshkov 21 Jun '12

21 Jun '12

Thomas Faber <thfabba(a)gmx.de> wrote: > Empty braces are no valid initializer. > FIELD_OFFSET isn't constant "enough". > + /* add 'jmp ExitProcess' instruction */ > + count = 2; > + bin.text_section[count++] = 0xFF; > + bin.text_section[count++] = 0x25; > + bin.text_section[count++] = EXIT_PROCESS&0xFF; > + bin.text_section[count++] = (EXIT_PROCESS>>8)&0xFF; > + bin.text_section[count++] = (EXIT_PROCESS>>16)&0xFF; > + bin.text_section[count++] = (EXIT_PROCESS>>24)&0xFF; This part of the patch obviously is not related to the patch description. -- Dmitry.

1 0

Re: kernel32: Add a test for deleting an opened for reading file.
by Marvin 21 Jun '12

21 Jun '12

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=19407 Your paranoid android. === WNT4WSSP6 (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === W2KPROSP4 (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === WXPPROSP3 (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === W2K3R2SESP2 (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === WVISTAADM (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === W2K8SE (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === W7PRO (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === W7PROX64 (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === TEST64_W7SP1 (32 bit file) === file.c:3255: Test failed: DeleteFile error 2 === W7PROX64 (64 bit file) === file.c:3255: Test failed: DeleteFile error 2 === TEST64_W7SP1 (64 bit file) === file.c:3255: Test failed: DeleteFile error 2

1 0

Re: windowscodecs: Create a IWICMetadataReader for a TIFF frame. Take 3.
by Dmitry Timoshkov 21 Jun '12

21 Jun '12

Dmitry Timoshkov <dmitry(a)baikal.ru> wrote: > --- > dlls/windowscodecs/tiffformat.c | 75 +++++++++++++++++++++++++++++++++++++---- > 1 file changed, 69 insertions(+), 6 deletions(-) Just a note, that once this patch is committed plus today's IFD metadata reader improvements the infrastructure for metadata handling in TIFF files should be all done. WICExplorer is able to recognize and show complete metadata listings for every frame in a multiframe TIFF file (native riched20 is needed for nicer visualization). -- Dmitry.

1 0

Re: [PATCH] winspool/tests: Fix failures, when the spooler service is not running
by Marvin 20 Jun '12

20 Jun '12

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=19360 Your paranoid android. === WINEBUILD (build) === Can't determine base name

1 0

Re: gdiplus: Implement GdipImageSelectActiveFrame. Take 2.
by Vincent Povirk 20 Jun '12

20 Jun '12

Looks good to me.

1 0

RE: msi:RECORD_StreamFromFile bug, help needed
by robert.van.herk＠serioustoys.com 20 Jun '12

20 Jun '12

> IStream is not exposed to the user, but an internal interface would probably end up looking a lot like IStream because we need read/write/seek operations, reference counting and the ability to create clones. So we might as well use IStream. Hmmm... Then I want exactly the same as SHCreateStreamOnFileEx, but the only difference is that the file should be opened with FILE_SHARE_DELETE. (I think). What is that policy on this? It seems overkill to copy istream.c from shlwapi, just because I want to change 1 flag. On the other hand, I don't want to expose more functions in the shlwapi header, as that would make it different from the Windows implementation... Regards, Robert

2 1

Re: mmcndmgr: Implement IMMCVersionInfo (try 5)
by Marvin 20 Jun '12

20 Jun '12

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=19325 Your paranoid android. === WNT4WSSP6 (32 bit mmcndmgr) === mmcndmgr.c:40: Test failed: got 0x80040154, expected 0x00000000 === W2KPROSP4 (32 bit mmcndmgr) === mmcndmgr.c:40: Test failed: got 0x80040154, expected 0x00000000

1 0

Re: shell32: Make ShellExecute use native file association databases as fallback path (try 2)
by Francois Gouget 20 Jun '12

20 Jun '12

On Tue, 19 Jun 2012, Erik van Pienbroek wrote: [...] > Additionally a check is added to make sure no recursive loop is > triggered when the native file association database points back to > wine and the fallback path is triggered again This looks much better. [...] > @@ -1911,6 +1944,44 @@ static BOOL SHELL_execute( LPSHELLEXECUTEINFOW sei, SHELL_ExecuteW32 execfunc ) > strcatW(lpstrTmpFile, lpFile); > retval = (UINT_PTR)ShellExecuteW(sei_tmp.hwnd, sei_tmp.lpVerb, lpstrTmpFile, NULL, NULL, 0); > } > +#if defined(HAVE_FORK) && defined(HAVE_WAITPID) > + /* Integration with native file association database */ > + else if (!getenv("WINE_INSIDE_SHELLEXECUTE")) > + { [...] > + /* Prevent the situation where the native file association database > + * tries to run wine to execute a program and we trigger this fallback > + * path again and end up in a recursive loop */ > + setenv("WINE_INSIDE_SHELLEXECUTE", "1", 0); [...] + unsetenv("WINE_INSIDE_SHELLEXECUTE"); Setting WINE_INSIDE_SHELLEXECUTE here creates a race condition if two threads call ShellExecute() at the same time. So I think this should be set between the fork() and the exec() in SHELL_try_native_execute(). -- Francois Gouget <fgouget(a)codeweavers.com>

1 0

msi:RECORD_StreamFromFile bug, help needed
by robert.van.herk＠serioustoys.com 20 Jun '12

20 Jun '12

Hi all, I think I found a msi.dll, record.c, RECORD_StreamFromFile. This function calls GlobalAlloc, with a file size as an argument. However, I believe that this can easily fail (which it does in my case) when the file size is too large. In my case, my msi builder (WiX) calls MsiRecordSetStreamW, which calls static function RECORD_StreamFromFile. WiX calls MsiRecordSetStreamW on some file that seems to be 0x11b2acc5 bytes (296 megs) large. Trying to allocate the block of 296 megs causes the error: 036:Call msi.MsiRecordSetStreamW(00000003,00000002,0cc8078c L"C:\\users\\robert\\Temp\\3c5c43d5\\1bfe39fa\\#product.cab") ret=009a4c3a 0036:Call KERNEL32.CreateFileW(0cc8078c L"C:\\users\\robert\\Temp\\3c5c43d5\\1bfe39fa\\#product.cab",80000000,00000001,00000000,00000003,00000000,00000000) ret=202868cf 0036:Ret KERNEL32.CreateFileW() retval=00000100 ret=202868cf 0036:Call KERNEL32.GetFileSize(00000100,0033ef58) ret=202868ef 0036:Ret KERNEL32.GetFileSize() retval=11b2acc5 ret=202868ef 0036:Call KERNEL32.GlobalAlloc(00000000,11b2acc5) ret=202869fb 0036:Ret KERNEL32.GlobalAlloc() retval=00000000 ret=202869fb <--- fail I believe the call to GlobalAlloc stems from RECORD_StreamFromFile. This tries to copy the full contents of the file in memory, and then create an IStream from that memory using CreateStreamOnHGlobal: /* read the data in a file into an IStream */ static UINT RECORD_StreamFromFile(LPCWSTR szFile, IStream **pstm) { ... handle = CreateFileW(szFile, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, NULL); ... sz = GetFileSize(handle, &szHighWord); if( sz != INVALID_FILE_SIZE && szHig sz = GetFileSize(handle, &szHighWord); if( sz != INVALID_FILE_SIZE && szHighWord == 0 ) { hGlob = GlobalAlloc(GMEM_FIXED, sz); // <-- Fails in my case, because sz == 296 megs if( hGlob ) { BOOL r = ReadFile(handle, hGlob, sz, &read, NULL); if( !r ) { GlobalFree(hGlob); hGlob = 0; } } CloseHandle(handle); if( !hGlob ) return ERROR_FUNCTION_FAILED; // <-- hence, this error is raised. hr = CreateStreamOnHGlobal(hGlob, TRUE, pstm); ... } I want to fix this. I guess that apart from that the allocation fails, I have the feeling that copying this entire 296 megs into main memory is anyway a bit heavy, just for creating a stream. So I thought a better solution would be just to create a stream from the file. My first guess would be to use the SHCreateStreamOnFile function: HRESULT SHCreateStreamOnFile( __in LPCTSTR pszFile, __in DWORD grfMode, __out IStream **ppstm ); But then I was thinking: this seems like such an obvious thing to do, that it is almost suspicious. Was there any reason for copying the file contents to main memory and then create a memory stream? For instance, was the intention to have a non-mutable copy in memory, in case the backing file would be altered later on? I guess if that's the case, I could just create a copy of the original file in a temp dir, and mark it as STGM_DELETEONRELEASE. Before I wildly attempt a fix though, I would appreciate it if someone had an idea here... Kind regards, Robert van Herk

2 8

Re: [PATCH 1/4] hhctrl.ocx: Add HTML to Unicode decoding capability.
by André Hentschel 19 Jun '12

19 Jun '12

Am 18.06.2012 15:52, schrieb Erich E. Hoover: > Real Name: > Erich Hoover > > Description: > This patch adds the ability in HTML Help to convert HTML encoded > characters (e.g. ê) into the Unicode character equivalent. This > feature is needed by the table of contents and the index for > displaying international characters in some CHM files. With this > version of the patch the code for finding the appropriate symbol has > been split out into a separate function (requested by Michael > Stefaniuc) and the string comparison has been modified to use the > larger of the two lengths (the length of the symbol to check and the > length of the symbol given in the help file). > > Changelog: > hhctrl.ocx: Add HTML to Unicode decoding capability. You're adding dead code here, you should maybe merge Patch 1 & 2 -- Best Regards, André Hentschel

2 2

Re: shell32: add some tests for the Progman DDE interface
by Alexandre Julliard 19 Jun '12

19 Jun '12

Damjan Jovanovic <damjan.jov(a)gmail.com> writes: > Changelog: > * shell32: add some tests for the Progman DDE interface Please merge that with the existing tests, assuming that what you are testing is not already covered. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: ntdll : add ThreadQuerySetWin32StartAddress in NtQueryInformationThread
by Alexandre Julliard 19 Jun '12

19 Jun '12

Roman Dadkov <romand(a)etersoft.ru> writes: > @@ -63,7 +63,7 @@ struct reply_header > /* this is used to construct the generic_request union */ > struct request_max_size > { > - int pad[16]; /* the max request size is 16 ints */ > + int pad[18]; /* the max request size is 18 ints */ > }; You can't change that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: mmcndmgr: Implement IMMCVersionInfo (try 4)
by Alexandre Julliard 19 Jun '12

19 Jun '12

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > @@ -66,8 +179,6 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) > > switch (fdwReason) > { > - case DLL_WINE_PREATTACH: > - return FALSE; /* prefer native version */ Changing this may or may not be a good idea at this point, but it definitely shouldn't be silently sneaked in as part of a different patch. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/3] gdi32/tests: Make GetStockObject tests pass on East-Asian machines. (try 2)
by Akihiro Sagawa 19 Jun '12

19 Jun '12

On Tue, 19 Jun 2012 03:39:18 -0300, Bruno Jesus wrote: > Hi, judging by the subjects I'd say the patches were commited already, > you're trying to send them again or are they different patches? > > http://source.winehq.org/git/wine.git/commitdiff/afdc84cf0366c8e161004bf1e7… > http://source.winehq.org/git/wine.git/commitdiff/2bb416baa88c28f57bb0f65aa9… > http://source.winehq.org/git/wine.git/commitdiff/f503afd86ff9df9467eb1351cf… Hi Bruno, Please ignore these patches. They were sent over 48 hours ago. It seems there is a huge delay between Gmail and wine-patches. Actually, one of my original patches are missed `static' keyword as Dmitry spotted(*). Therefore, I added the keyword and sent series of patches again. But, during transfer (>48 hours), the original patch was updated by Alexandre, then committed. After a while, my modified version was delivered and tested although they have already been applied. * ... http://www.winehq.org/pipermail/wine-devel/2012-June/095956.html Akihiro Sagawa

2 2

Re: [website] Portuguese translation for release 1.4.1
by Lucas Zawacki 19 Jun '12

19 Jun '12

Disregard this patch. I added the wrong file to the commit and will resend. On Tue, Jun 19, 2012 at 12:01 PM, Jeremy Newman <jnewman(a)codeweavers.com> wrote: > This is just english. Did you mean for that to happen? > > No need to send the english, as if the xml does not exist, the site will > always fall back to the english version already. > > -N > > > On 06/19/2012 09:49 AM, Lucas Zawacki wrote: >> >> From: Lucas Zawacki<lfzawacki(a)gmail.com> >> >> --- >> news/pt/2012061501.xml | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) >> create mode 100644 news/pt/2012061501.xml >> >> diff --git a/news/pt/2012061501.xml b/news/pt/2012061501.xml >> new file mode 100644 >> index 0000000..7c0ff9f >> --- /dev/null >> +++ b/news/pt/2012061501.xml >> @@ -0,0 +1,13 @@ >> +<news> >> +<date>June 15, 2012</date> >> +<title>Wine 1.4.1 Released</title> >> +<body> >> + The Wine maintenance release 1.4.1 is now available. >> + <a href="{$root}/announce/1.4.1">What's new</a> in this release: >> +<ul> >> +<li>Translation updates.</li> >> +<li>Various bug fixes.</li> >> +</ul> >> +The source is<a >> href="http://prdownloads.sourceforge.net/wine/wine-1.4.1.tar.bz2">available >> now</a>. >> +Binary packages are in the process of being built, and will appear soon >> at their respective<a href="{$root}/download">download locations</a>. >> +</body></news>

1 0

Re: [5/5] gdiplus: Implement GdipImageSelectActiveFrame.
by Vincent Povirk 19 Jun '12

19 Jun '12

> + hr = IStream_Clone(image->stream, &stream); > + if (FAILED(hr)) That's odd. When does that happen? > + if (stat == Ok) > + { > + memcpy(&new_image->format, &codec->info.FormatID, sizeof(GUID)); > + free_image_data(image); > + if (image->type == ImageTypeBitmap) > + *(GpBitmap *)image = *(GpBitmap *)new_image; > + else if (image->type == ImageTypeMetafile) > + *(GpMetafile *)image = *(GpMetafile *)new_image; > + return Ok; > + } Why did you re-implement move_bitmap() ?

2 5

Re: d3dx9_36 [patch 1of 2]: Implementation of D3DXSHDot
by Matteo Bruni 19 Jun '12

19 Jun '12

2012/6/19 Nozomi Kodama <nozomi.kodama(a)yahoo.com>: +static void test_D3DXSHDot(void) +{ + unsigned int i; + FLOAT a[90], b[90], got; + CONST FLOAT expected[] = + { 0.0f, 0.0f, 17.0f, 222.0f, 1300.0f, 5050.0f, 15225.0f, 38612.0f, 86352.0f, 175500.0f, }; + + for (i = 0; i < 90; i++) + { + a[i] = (FLOAT)i; + b[i] = (FLOAT)i + 0.5f; + } + +/* D3DXSHDot computes by using order * order elements */ + for (i = 0; i < 9; i++) + { + got = D3DXSHDot(i, a, b); + ok(relative_error(got, expected[i]) < admitted_error, "order %d: expected %f, received %f\n", i, expected[i], got); + } + + return; +} + Making the "a" and "b" arrays 90 elements big doesn't make much sense. If your D3DXSHDot implementation is correct, for the i = 8 case you only need 8 * 8 = 64 elements. Also notice that the "expected" array has 10 elements, while you use only 9. Also please recheck formatting and whitespaces of this patch and patch 2/2.

1 0

Re: [appdb] Upgrade to xinha svn head to fix issues with xinha editors not appearing in Firefox 15
by Ricardo Filipe 19 Jun '12

19 Jun '12

2012/6/18 André Hentschel <nerv(a)dawncrow.de>: > http://bugs.winehq.org/show_bug.cgi?id=30820 > --- > xinha/Xinha.css | 229 +- > xinha/XinhaCore.js |12107 ++++++++++++++------ > xinha/contrib/php-xinha.php | 29 +- > xinha/examples/Extended.html | 299 - > xinha/examples/Newbie.html | 12 +- > xinha/examples/XinhaConfig.js | 149 +- > xinha/examples/custom.css | 40 - > xinha/examples/dynamic.css | 56 - > xinha/examples/ext_example-body.html | 202 - > xinha/examples/ext_example-dest.php | 23 - > xinha/examples/ext_example-menu.php | 331 - > xinha/examples/ext_example.html | 16 - > xinha/examples/full_example.css | 48 - > xinha/examples/full_example.js | 97 - > xinha/examples/simple_example.html | 190 +- > xinha/examples/stylist.css | 31 - > xinha/examples/testbed.html | 45 +- > xinha/htmlarea.js | 44 +- > xinha/lang/b5.js | 58 +- > xinha/lang/ch.js | 112 +- > xinha/lang/cz.js | 100 +- > xinha/lang/da.js | 202 +- > xinha/lang/de.js | 338 +- > xinha/lang/ee.js | 100 +- > xinha/lang/el.js | 110 +- > xinha/lang/es.js | 207 +- > xinha/lang/eu.js | 2 +- > xinha/lang/fa.js | 6 +- > xinha/lang/fi.js | 76 +- > xinha/lang/fr.js | 338 +- > xinha/lang/gb.js | 58 +- > xinha/lang/he.js | 128 +- > xinha/lang/hu.js | 128 +- > xinha/lang/it.js | 110 +- > xinha/lang/ja.js | 6 +- > xinha/lang/lt.js | 106 +- > xinha/lang/lv.js | 84 +- > xinha/lang/nb.js | 156 +- > xinha/lang/nl.js | 128 +- > xinha/lang/pl.js | 2 +- > xinha/lang/pt_br.js | 180 +- > xinha/lang/ro.js | 126 +- > xinha/lang/ru.js | 370 +- > xinha/lang/sh.js | 2 +- > xinha/lang/si.js | 100 +- > xinha/lang/sr.js | 2 +- > xinha/lang/vn.js | 112 +- > xinha/modules/ColorPicker/ColorPicker.js | 1494 ++- > xinha/modules/CreateLink/link.js | 190 +- > xinha/modules/Dialogs/dialog.js | 159 +- > xinha/modules/Dialogs/inline-dialog.js | 204 - > xinha/modules/Dialogs/panel-dialog.js | 1 + > xinha/modules/Dialogs/popupwin.js | 291 +- > xinha/modules/FullScreen/full-screen.js | 373 +- > xinha/modules/Gecko/Gecko.js | 1225 ++- > xinha/modules/Gecko/paraHandlerBest.js | 1196 ++- > xinha/modules/Gecko/paraHandlerDirty.js | 116 - > xinha/modules/GetHtml/DOMwalk.js | 519 +- > xinha/modules/GetHtml/TransformInnerHTML.js | 368 +- > xinha/modules/InsertImage/insert_image.html | 345 +- > xinha/modules/InsertImage/insert_image.js | 214 +- > xinha/modules/InsertTable/insert_table.html | 314 +- > xinha/modules/InsertTable/insert_table.js | 153 +- > xinha/modules/InternetExplorer/InternetExplorer.js | 1150 ++- > xinha/plugins/CSS/css.js | 74 - > xinha/plugins/CharCounter/char-counter.js | 106 - > xinha/plugins/CharacterMap/CharacterMap.css | 6 +- > xinha/plugins/CharacterMap/character-map.js | 68 - > xinha/plugins/CharacterMap/lang/de.js | 22 +- > .../CharacterMap/popups/select_character.html | 184 - > xinha/plugins/ContextMenu/1.pl | 38 - > xinha/plugins/ContextMenu/context-menu.js | 383 - > xinha/plugins/ContextMenu/lang/de.js | 100 +- > xinha/plugins/ContextMenu/lang/el.js | 94 +- > xinha/plugins/ContextMenu/lang/fr.js | 98 +- > xinha/plugins/ContextMenu/lang/he.js | 94 +- > xinha/plugins/ContextMenu/lang/nl.js | 94 +- > xinha/plugins/DoubleClick/double-click.js | 66 - > xinha/plugins/DynamicCSS/dynamiccss.js | 199 - > xinha/plugins/DynamicCSS/lang/de.js | 24 +- > xinha/plugins/DynamicCSS/lang/fr.js | 12 +- > xinha/plugins/FindReplace/find-replace.js | 42 - > xinha/plugins/FindReplace/fr_engine.js | 291 +- > xinha/plugins/FindReplace/lang/pt_br.js | 60 +- > xinha/plugins/FindReplace/popups/find_replace.html | 324 +- > xinha/plugins/Forms/forms.js | 354 - > xinha/plugins/Forms/popups/fieldset.html | 15 +- > xinha/plugins/Forms/popups/form.html | 21 +- > xinha/plugins/Forms/popups/input.html | 78 +- > xinha/plugins/Forms/popups/label.html | 15 +- > xinha/plugins/Forms/popups/select.html | 217 +- > xinha/plugins/Forms/popups/textarea.html | 41 +- > xinha/plugins/FullPage/full-page.js | 200 - > xinha/plugins/FullPage/lang/de.js | 36 +- > xinha/plugins/FullPage/lang/fr.js | 32 +- > xinha/plugins/FullPage/lang/he.js | 28 +- > xinha/plugins/FullPage/lang/ro.js | 28 +- > xinha/plugins/FullPage/popups/docprop.html | 282 +- > xinha/plugins/FullScreen/full-screen.js | 203 - > xinha/plugins/FullScreen/lang/de.js | 6 - > xinha/plugins/FullScreen/lang/fr.js | 5 - > xinha/plugins/FullScreen/lang/no.js | 6 - > xinha/plugins/FullScreen/lang/pl.js | 6 - > xinha/plugins/HtmlTidy/README | 104 - > xinha/plugins/HtmlTidy/html-tidy-config.cfg | 29 - > xinha/plugins/HtmlTidy/html-tidy-logic.php | 79 - > xinha/plugins/HtmlTidy/html-tidy.js | 105 - > xinha/plugins/HtmlTidy/img/html-tidy.gif | Bin 599 -> 0 bytes > xinha/plugins/HtmlTidy/lang/de.js | 7 - > xinha/plugins/HtmlTidy/lang/fr.js | 8 - > xinha/plugins/HtmlTidy/lang/ja.js | 8 - > xinha/plugins/HtmlTidy/lang/nb.js | 7 - > xinha/plugins/HtmlTidy/lang/nl.js | 5 - > xinha/plugins/InsertAnchor/insert-anchor.css | 1 - > xinha/plugins/InsertAnchor/insert-anchor.js | 76 - > xinha/plugins/InsertPicture/InsertPicture.php | 301 - > xinha/plugins/InsertPicture/img/btn_open.gif | Bin 173 -> 0 bytes > xinha/plugins/InsertPicture/img/nopic.gif | Bin 1537 -> 0 bytes > xinha/plugins/InsertPicture/insert-picture.js | 25 - > xinha/plugins/InsertPicture/lang/de.js | 15 - > xinha/plugins/InsertPicture/lang/fr.js | 22 - > xinha/plugins/InsertPicture/lang/ja.js | 25 - > xinha/plugins/InsertPicture/lang/nb.js | 22 - > xinha/plugins/InsertPicture/lang/sv.js | 35 - > xinha/plugins/InsertPicture/viewpicture.html | 46 - > xinha/plugins/InsertWords/insert-words.js | 70 - > xinha/plugins/Linker/dTree/api.html | 502 +- > xinha/plugins/Linker/dTree/dtree.css | 8 +- > xinha/plugins/Linker/dTree/dtree.js | 1037 ++- > xinha/plugins/Linker/dTree/example01.html | 104 +- > xinha/plugins/Linker/dialog.html | 210 +- > xinha/plugins/Linker/linker.js | 385 - > xinha/plugins/Linker/scan.php | 220 +- > xinha/plugins/ListType/lang/de.js | 24 +- > xinha/plugins/ListType/list-type.js | 137 - > xinha/plugins/QuickTag/lang/pt_br.js | 26 +- > xinha/plugins/QuickTag/popups/quicktag.html | 706 +- > xinha/plugins/QuickTag/quick-tag.js | 53 - > xinha/plugins/QuickTag/tag-lib.js | 381 +- > xinha/plugins/SpellChecker/aspell_setup.php | 246 +- > xinha/plugins/SpellChecker/lang/cz.js | 52 +- > xinha/plugins/SpellChecker/lang/da.js | 52 +- > xinha/plugins/SpellChecker/lang/de.js | 52 +- > xinha/plugins/SpellChecker/lang/he.js | 54 +- > xinha/plugins/SpellChecker/lang/hu.js | 52 +- > xinha/plugins/SpellChecker/lang/nl.js | 54 +- > xinha/plugins/SpellChecker/lang/ro.js | 52 +- > xinha/plugins/SpellChecker/readme-tech.html | 228 +- > xinha/plugins/SpellChecker/spell-check-logic.cgi | 420 +- > xinha/plugins/SpellChecker/spell-check-logic.php | 340 +- > .../plugins/SpellChecker/spell-check-savedicts.php | 74 +- > xinha/plugins/SpellChecker/spell-check-ui.html | 250 +- > xinha/plugins/SpellChecker/spell-check-ui.js | 846 +- > xinha/plugins/SpellChecker/spell-checker.js | 73 - > xinha/plugins/Stylist/stylist.js | 372 - > xinha/plugins/SuperClean/dialog.html | 14 +- > xinha/plugins/SuperClean/lang/de.js | 3 +- > xinha/plugins/SuperClean/super-clean.js | 211 - > xinha/plugins/SuperClean/tidy.php | 358 +- > xinha/plugins/TableOperations/lang/cz.js | 158 +- > xinha/plugins/TableOperations/lang/da.js | 161 +- > xinha/plugins/TableOperations/lang/de.js | 162 +- > xinha/plugins/TableOperations/lang/el.js | 158 +- > xinha/plugins/TableOperations/lang/fi.js | 136 +- > xinha/plugins/TableOperations/lang/fr.js | 154 +- > xinha/plugins/TableOperations/lang/he.js | 158 +- > xinha/plugins/TableOperations/lang/it.js | 158 +- > xinha/plugins/TableOperations/lang/ja.js | 6 +- > xinha/plugins/TableOperations/lang/nb.js | 158 +- > xinha/plugins/TableOperations/lang/nl.js | 158 +- > xinha/plugins/TableOperations/lang/pl.js | 6 +- > xinha/plugins/TableOperations/lang/ro.js | 158 +- > xinha/plugins/TableOperations/lang/ru.js | 6 +- > xinha/plugins/TableOperations/lang/sv.js | 6 +- > .../TableOperations/popups/merge_cells.html | 11 +- > xinha/plugins/TableOperations/table-operations.js | 893 -- > xinha/plugins/UnFormat/popups/unformat.html | 85 - > xinha/plugins/UnFormat/un-format.js | 65 - > xinha/popups/about.html | 296 - > xinha/popups/blank.html | 3 +- > xinha/popups/editor_help.html | 122 +- > xinha/popups/popup.js | 239 +- > xinha/popups/select_color.html | 716 +- > xinha/release-notes.txt | 476 +- > xinha/skins/blue-look/skin.css | 14 +- > xinha/skins/blue-metallic/skin.css | 7 + > xinha/skins/green-look/skin.css | 10 +- > xinha/skins/inditreuse/skin.css | 10 +- > xinha/skins/titan/skin.css | 10 +- > xinha/skins/xp-blue/skin.css | 10 +- > xinha/skins/xp-green/skin.css | 10 +- > 191 files changed, 22579 insertions(+), 20386 deletions(-) > delete mode 100644 xinha/examples/Extended.html > delete mode 100644 xinha/examples/custom.css > delete mode 100644 xinha/examples/dynamic.css > delete mode 100644 xinha/examples/ext_example-body.html > delete mode 100644 xinha/examples/ext_example-dest.php > delete mode 100644 xinha/examples/ext_example-menu.php > delete mode 100644 xinha/examples/ext_example.html > delete mode 100644 xinha/examples/full_example.css > delete mode 100644 xinha/examples/full_example.js > delete mode 100644 xinha/examples/stylist.css > delete mode 100644 xinha/modules/Dialogs/inline-dialog.js > mode change 100644 => 100755 xinha/modules/FullScreen/lang/sv.js > delete mode 100644 xinha/modules/Gecko/paraHandlerDirty.js > delete mode 100644 xinha/plugins/CSS/css.js > delete mode 100644 xinha/plugins/CharCounter/char-counter.js > delete mode 100644 xinha/plugins/CharacterMap/character-map.js > mode change 100644 => 100755 xinha/plugins/CharacterMap/lang/sv.js > delete mode 100644 xinha/plugins/CharacterMap/popups/select_character.html > delete mode 100755 xinha/plugins/ContextMenu/1.pl > delete mode 100644 xinha/plugins/ContextMenu/context-menu.js > delete mode 100644 xinha/plugins/DoubleClick/double-click.js > delete mode 100644 xinha/plugins/DynamicCSS/dynamiccss.js > delete mode 100644 xinha/plugins/FindReplace/find-replace.js > mode change 100644 => 100755 xinha/plugins/FindReplace/lang/sv.js > delete mode 100644 xinha/plugins/Forms/forms.js > delete mode 100644 xinha/plugins/FullPage/full-page.js > delete mode 100644 xinha/plugins/FullScreen/full-screen.js > delete mode 100644 xinha/plugins/FullScreen/lang/de.js > delete mode 100644 xinha/plugins/FullScreen/lang/fr.js > delete mode 100644 xinha/plugins/FullScreen/lang/no.js > delete mode 100644 xinha/plugins/FullScreen/lang/pl.js > delete mode 100644 xinha/plugins/HtmlTidy/README > delete mode 100644 xinha/plugins/HtmlTidy/html-tidy-config.cfg > delete mode 100755 xinha/plugins/HtmlTidy/html-tidy-logic.php > delete mode 100644 xinha/plugins/HtmlTidy/html-tidy.js > delete mode 100644 xinha/plugins/HtmlTidy/img/html-tidy.gif > delete mode 100644 xinha/plugins/HtmlTidy/lang/de.js > delete mode 100644 xinha/plugins/HtmlTidy/lang/fr.js > delete mode 100644 xinha/plugins/HtmlTidy/lang/ja.js > delete mode 100644 xinha/plugins/HtmlTidy/lang/nb.js > delete mode 100644 xinha/plugins/HtmlTidy/lang/nl.js > delete mode 100755 xinha/plugins/InsertAnchor/insert-anchor.js > delete mode 100644 xinha/plugins/InsertPicture/InsertPicture.php > delete mode 100644 xinha/plugins/InsertPicture/img/btn_open.gif > delete mode 100644 xinha/plugins/InsertPicture/img/nopic.gif > delete mode 100644 xinha/plugins/InsertPicture/insert-picture.js > delete mode 100644 xinha/plugins/InsertPicture/lang/de.js > delete mode 100644 xinha/plugins/InsertPicture/lang/fr.js > delete mode 100644 xinha/plugins/InsertPicture/lang/ja.js > delete mode 100644 xinha/plugins/InsertPicture/lang/nb.js > delete mode 100644 xinha/plugins/InsertPicture/lang/sv.js > delete mode 100644 xinha/plugins/InsertPicture/viewpicture.html > delete mode 100755 xinha/plugins/InsertWords/insert-words.js > delete mode 100644 xinha/plugins/Linker/linker.js > mode change 100644 => 100755 xinha/plugins/ListType/lang/sv.js > delete mode 100644 xinha/plugins/ListType/list-type.js > delete mode 100644 xinha/plugins/QuickTag/quick-tag.js > delete mode 100644 xinha/plugins/SpellChecker/spell-checker.js > delete mode 100644 xinha/plugins/Stylist/stylist.js > delete mode 100644 xinha/plugins/SuperClean/super-clean.js > mode change 100644 => 100755 xinha/plugins/TableOperations/lang/sv.js > mode change 100644 => 100755 xinha/plugins/TableOperations/popups/merge_cells.html > delete mode 100644 xinha/plugins/TableOperations/table-operations.js > delete mode 100644 xinha/plugins/UnFormat/popups/unformat.html > delete mode 100644 xinha/plugins/UnFormat/un-format.js > delete mode 100644 xinha/popups/about.html > > > -- > > Best Regards, André Hentschel > > > > The xinha editor does not show up in chrome either for a long time now... could it be related to this? I even opened a bug report on chromium but it still wasn't looked at. Maybe this solves it too...

1 0

'Expensive' expressions in loop limits (was: Implementation of D3DXSHScale)
by Dan Kegel 19 Jun '12

19 Jun '12

Nozomi wrote: + for (i = 0; i < order * order; i++) I might have written int n = order * order; for (i=0; i < n; i++) to avoid repeating the multiplication every time around the loop, even though multiplication is cheap nowadays, and -O1 will optimize it out anyway. Staying in the habit of avoiding 'expensive' operations in loop limits might still be a good idea, since the optimizer can't always save you. Or is that considered ugly these days? D3DXSHAdd() has the same code, which is where this came from, probably.

2 1

Re: d3dx9_36: Implementation of a test for D3DFMT_A8B8G8R8
by Józef Kucia 19 Jun '12

19 Jun '12

I've also added a tests for this pixel format in the patch, which was submitted yesterday: http://www.winehq.org/pipermail/wine-patches/2012-June/115268.html

1 0

Re: [PATCH 2/3] gdi32: Fix East-Asian font parameters. (resend)
by Marvin 19 Jun '12

19 Jun '12

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

1 0

Re: [PATCH 1/3] gdi32/tests: Make GetStockObject tests pass on East-Asian machines. (try 2)
by Marvin 19 Jun '12

19 Jun '12

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

1 0

Re: windowscodecs: Create a IWICMetadataReader for a TIFF frame. Take 3.
by Vincent Povirk 19 Jun '12

19 Jun '12

I'm still a little worried about seeking the stream behind libtiff's back, but otherwise it looks good.

2 1

Disabling joysticks with registry keys and joy.cpl
by Lucas Zawacki 19 Jun '12

19 Jun '12

Hello all, disabling an specific joystick device is an issue that is raised from time to time in this list. I'm planning on implementing it as a registry key like 'DirectInput/Disable controllername'. However I'd like the ability to disable specific joysticks on Linux, like the /dev/input/js* version, while keeping the /dev/input/event* one being enumerated. One option for doing this would be having a "Disable USB Controller" -> "driver" key with options being: "linux", "linuxinput", and "all". My only doubt is how would you guys want to have this option in joy.cpl? I think the correct thing to do would be to disable it with "all", but my use case includes a user disabling the repeated joysticks on Linux through the interface and if he's only able to disable all drivers at once that would be confusing and less useful. The bottom line is: can I add platform dependent code in joy.cpl for this purpose? If that's not an option, do you have some other way to do through joy.cpl or should it be registry only? Attached is an example patch of the approach for joystick_linuxinput.c Cheers, Lucas

2 2

Re: d3dx9_36: Reckonize A8B8G8R8 format
by Stefan Dösinger 18 Jun '12

18 Jun '12

Hi, Please add the format to the dds tests in dlls/d3dx9_36/tests/surface.c > { 32, 0x0000ffff, 0xffff0000, 0x00000000, 0x00000000, D3DFMT_G16R16 }, > + { 32, 0xff, 0xff00, 0xff0000, 0xff000000, D3DFMT_A8B8G8R8 }, The other 32 bit formats in this table have their bitmasks padded to 32 bit with leading zeroes. Stefan Am Freitag, 15. Juni 2012, 18:18:46 schrieb Nozomi Kodama: This patch solves the color problem appearing in bug 30786. Nozomi

1 1

Re: comctl32: hotkey control needs to invalidate itself when painting for some apps
by Dmitry Timoshkov 18 Jun '12

18 Jun '12

Dan Kegel <dank(a)kegel.com> wrote: > @@ -190,6 +190,8 @@ HOTKEY_Paint(HOTKEY_INFO *infoPtr, HDC hdc) > HOTKEY_Refresh(infoPtr, hdc); > else { > PAINTSTRUCT ps; > + /* In some apps, the hotkey doesn't draw unless we invalidate it here */ > + InvalidateRect(infoPtr->hwndSelf, NULL, FALSE); It doesn't make sense to invalidate before painting in the paint handler. -- Dmitry.

2 1

Re: [PATCH] winepulse.drv: Add PulseAudio driver
by Chris Robinson 18 Jun '12

18 Jun '12

On Friday, June 15, 2012 3:39:33 PM Andrew Eikum wrote: > One thing to note is that PulseAudio has an absurdly high default > latency (two seconds), and due to its poor API there's no (easy) way > for us to control it. The latency is likely caused by PulseAudio setting a much larger tlength (and minreq) than what you asked for. This happens all the time for me when the server has been running. The trick I've found to dealing with this in OpenAL Soft is to essentially ignore what PulseAudio sets and do your free space calculations so that only the size of the buffer you requested is used. Something like: size_t writable_size(pulse_data *data) { uint buffer_size = our requested buffer size in bytes; uint update_size = our expected period size in bytes; /* This is signed since it could come out negative */ ssize_t len; pa_threaded_mainloop_lock(data->loop); len = pa_stream_writable_size(data->stream) - data->attr.tlength + buffer_size; pa_threaded_mainloop_unlock(data->loop); if(len < 0) return 0; len -= len%update_size; return len; } The update_size is only relevant if mmdevapi updates in period-sized chunks. If it doesn't, you can just remove it and the rounding. The tlength should be kept updated using pa_stream_set_buffer_attr_callback, in case the server tries to increase it more during runtime. When creating the stream, I set minreq to the period size, and tlength to the requested buffer size. I also set the flags PA_STREAM_INTERPOLATE_TIMING, PA_STREAM_AUTO_TIMING_UPDATE, and PA_STREAM_ADJUST_LATENCY. The most important part is to not rely on PA_STREAM_EARLY_REQUESTS or the write callback to tell you when some space is free for writing, since as noted above, PulseAudio can set unreasonably large values. This doesn't completely fix the latency problem, as I'll still get better performance using ALSA's dmix, but it helps immensely.

2 4

comctl32: hotkey control needs to invalidate itself when painting for some apps
by Dan Kegel 18 Jun '12

18 Jun '12

In at least one app, the hotkey control doesn't draw itself. Invalidating its own window during HOTKEY_Paint gets the control to draw properly, and makes the app happy. Fixes http://bugs.winehq.org/show_bug.cgi?id=30486 (Unlike the patch attached to that bug, this one doesn't also ask the background to be redrawn again.)

2 1

Re: windowscodecs: Create a IWICMetadataReader for a TIFF frame. Take 2.
by Vincent Povirk 18 Jun '12

18 Jun '12

This looks better, but I still think you should use WICPersistOptionNoCacheStream. Some of your failure cases in create_metadata_reader will return a reference to an IWICMetadataReader, which seems wrong.

2 1

Re: windowscodecs: Create a IWICMetadataReader when loading a TIFF frame.
by Vincent Povirk 18 Jun '12

18 Jun '12

> + hr = IWICPersistStream_LoadEx(persist, This->parent->stream, NULL, persist_options); > + if (FAILED(hr)) > + ERR("IWICPersistStream_LoadEx error %#x\n", hr); We can't use that stream because we provide separate seek and read/write methods to libtiff, and libtiff may rely on the stream's position to not change between operations. This illustrates a design flaw in WIC. Normally, we would just clone the stream, but IWICStream doesn't implement the Clone method. I think we are probably supposed to create a new IWICStream based on the given stream, but the resulting stream won't be thread-safe with ours. I think for now it's OK to re-use this stream as long as you seek it back to where it was when you're done, but we may have to change that later. Also, LoadEx should (but does not yet) save the stream so it can create an IWICFastMetadataEncoder. If you re-use the bitmap decoder's stream, you should also use the WICPersistOptionNoCacheStream flag when you create the metadata reader to make sure it won't do that. Your GetReaderByIndex method should check whether This->metadata_reader has been set and return an error if not. Otherwise, the patch looks good to me.

2 4

Re: atl80: prefer native, if present
by Alexandre Julliard 18 Jun '12

18 Jun '12

Daniel Jelinski <djelinski1(a)gmail.com> writes: > From 7243b3594c9c0b13770ede1ab32d4dd1329b1dd4 Mon Sep 17 00:00:00 2001 > From: Daniel Jelinski <djelinski1(a)gmail.com> > Date: Sun, 17 Jun 2012 17:37:15 +0200 > Subject: atl80: prefer native, if present > > --- > dlls/atl80/atl80.c | 11 +++++++++++ > 1 files changed, 11 insertions(+), 0 deletions(-) There's no reason to do that, most of the dll is already implemented. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: [PATCH] winepulse.drv: Add PulseAudio driver
by Darryl Slack 18 Jun '12

18 Jun '12

Well after fighting with pulseaudio after all this time I have tried this patch with wine 1.5.6 with a Presonus StudioLive 24.4.2 Fire-wire controller within wine, to control software known as Virtual DJ with timecoded vinyl records. With Andrew's patch, there are stalled buffer errors, and I wasn't able to measure latency very well because the controller just kept on reporting 'stalled' I did also try martaans driver, which was added to wine, and latency was much lower, and the controller worked well within Linux, in fact the first time i've actually been able to use it with pulseaudio. I was measuring latency much lower than I expected, 2-4ms. With Andrew's Driver I was not even able to play a timecode from start to finish without 'stalled' buffers. Please Take the time to QA this driver for things just other than 'cos we have it' and consider Martaan's driver as well, because there is a massive improvement in latency over Andrews, and applications requiring fire-wire work much nicer with pulseaudio. My Wine build is 1.5.4, Gentoo 3.4.2-r1. Thanks Darryl

1 0

Re: mshtml: Implement IHTMLScriptElement_put_src
by Jacek Caban 18 Jun '12

18 Jun '12

André, On 06/15/12 23:45, André Hentschel wrote: > --- > dlls/mshtml/htmlscript.c | 14 ++++++++++++-- > dlls/mshtml/tests/dom.c | 6 ++++++ > 2 files changed, 18 insertions(+), 2 deletions(-) > > diff --git a/dlls/mshtml/htmlscript.c b/dlls/mshtml/htmlscript.c > index 376db7b..1714cfe 100644 > --- a/dlls/mshtml/htmlscript.c > +++ b/dlls/mshtml/htmlscript.c > @@ -101,8 +101,18 @@ static HRESULT WINAPI HTMLScriptElement_Invoke(IHTMLScriptElement *iface, DISPID > static HRESULT WINAPI HTMLScriptElement_put_src(IHTMLScriptElement *iface, BSTR v) > { > HTMLScriptElement *This = impl_from_IHTMLScriptElement(iface); > - FIXME("(%p)->(%s)\n", This, debugstr_w(v)); > - return E_NOTIMPL; > + nsAString src_str; > + nsresult nsres; > + > + TRACE("(%p)->(%s)\n", This, debugstr_w(v)); > + > + nsAString_Init(&src_str, v); > + nsres = nsIDOMHTMLScriptElement_SetSrc(This->nsscript, &src_str); > + if (NS_FAILED(nsres)) > + ERR("SetSrc failed: %08x\n", nsres); > + nsAString_Finish (&src_str); > + > + return S_OK; > } > Unfortunately it's not so easy. That won't work this way. Jacek

1 0

Re: [PATCH 2/5] include/ddk: Add DVD_DESCRIPTOR_HEADER descriptor structure (try 2)
by Alexandre Julliard 18 Jun '12

18 Jun '12

Alexandre Goujon <ale.goujon(a)gmail.com> writes: > @@ -113,7 +113,16 @@ typedef struct DVD_READ_STRUCTURE { > > /* From 0 to 4 */ > UCHAR LayerNumber; > -}DVD_READ_STRUCTURE, * PDVD_READ_STRUCTURE; > +} DVD_READ_STRUCTURE, *PDVD_READ_STRUCTURE; > + > +typedef struct _DVD_DESCRIPTOR_HEADER { > + USHORT Length; > + UCHAR Reserved[2]; > +#if !defined(_midl) > + UCHAR Data[0]; > +#endif > +} DVD_DESCRIPTOR_HEADER, *PDVD_DESCRIPTOR_HEADER; > +C_ASSERT(sizeof(DVD_DESCRIPTOR_HEADER) == 4); Zero-size arrays are not portable and should be avoided. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [6/6] wbemprox: Add tests for IWbemServices.
by Marvin 18 Jun '12

18 Jun '12

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=19208 Your paranoid android. === WNT4WSSP6 (32 bit services) === services.c:38: Test failed: failed to create IWbemLocator interface 80040154 services: unhandled exception c0000005 at 004019EE

1 0

Re: [2/2] comdlg32/tests: Test file name selection on init
by Marvin 17 Jun '12

17 Jun '12

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=19196 Your paranoid android. === WNT4WSSP6 (32 bit filedlg) === Timeout === W2KPROSP4 (32 bit filedlg) === Timeout === WXPPROSP3 (32 bit filedlg) === filedlg: unhandled exception c0000005 at 7C809E32 === W2K3R2SESP2 (32 bit filedlg) === filedlg: unhandled exception c0000005 at 77E63110 === WVISTAADM (32 bit filedlg) === filedlg: unhandled exception c0000005 at 76A75031 === W2K8SE (32 bit filedlg) === filedlg: unhandled exception c0000005 at 76285DBE === W7PRO (32 bit filedlg) === filedlg.c:1158: Test failed: The EM_GETSEL message returned with no selection

1 0

Re: [PATCH 5/5] kernel32/tests: Add a IOCTL_DVD_READ_STRUCTURE (DvdManufacturerDescriptor) test
by Nikolay Sivov 17 Jun '12

17 Jun '12

On 6/17/2012 13:11, Alexandre Goujon wrote: > > case DvdManufacturerDescriptor: > + nt_desc.manf->Header.Length = 0x0802; > + nt_desc.manf->Header.Reserved[0] = 0; > + nt_desc.manf->Header.Reserved[1] = 0; Does this length mean something? Looks a bit dirty to leave hardcoded things like that, but maybe I don't know details.

2 1

Re: [PATCH 2/5] include/ddk: Add DVD_DESCRIPTOR_HEADER descriptor structure (try 2)
by Nikolay Sivov 17 Jun '12

17 Jun '12

On 6/17/2012 13:11, Alexandre Goujon wrote: > Superseeds the patch of the same name (#87003) > Definition and assert taken from Windows Driver Kit (WDK) 8 Consumer Preview > --- > include/ddk/ntddcdvd.h | 12 +++++++++++- > 1 file changed, 11 insertions(+), 1 deletion(-) > > diff --git a/include/ddk/ntddcdvd.h b/include/ddk/ntddcdvd.h > index 5313496..d0178c7 100644 > --- a/include/ddk/ntddcdvd.h > +++ b/include/ddk/ntddcdvd.h > @@ -113,7 +113,16 @@ typedef struct DVD_READ_STRUCTURE { > > /* From 0 to 4 */ > UCHAR LayerNumber; > -}DVD_READ_STRUCTURE, * PDVD_READ_STRUCTURE; > +} DVD_READ_STRUCTURE, *PDVD_READ_STRUCTURE; > + > +typedef struct _DVD_DESCRIPTOR_HEADER { > + USHORT Length; > + UCHAR Reserved[2]; > +#if !defined(_midl) > + UCHAR Data[0]; > +#endif > +} DVD_DESCRIPTOR_HEADER, *PDVD_DESCRIPTOR_HEADER; > +C_ASSERT(sizeof(DVD_DESCRIPTOR_HEADER) == 4); This looks wrong, correct name for it __midl, and this also needs a check for __WIDL__ probably.

2 1

Re: [PATCH 1/3] gdi32/tests: Make GetStockObject tests pass on East-Asian machines.
by Dmitry Timoshkov 17 Jun '12

17 Jun '12

Akihiro Sagawa <sagawa.aki(a)gmail.com> wrote: > - static const struct test_data > + const int font[] = > { > - int id, weight, height, dpi; > + ANSI_FIXED_FONT, ANSI_VAR_FONT, SYSTEM_FONT, DEVICE_DEFAULT_FONT, DEFAULT_GUI_FONT > + /* SYSTEM_FIXED_FONT, OEM_FIXED_FONT */ > + }; > + const struct test_data > + { Please add back 'static' to test_data and add it to font. -- Dmitry.

1 0

Re: [PATCH] winepulse.drv: Add PulseAudio driver
by Maarten Lankhorst 16 Jun '12

16 Jun '12

What happened to the stamped capture buffers? Why is held_frames added? This doesn't look like an attempt to get my driver in, it looks like a rewrite by someone who doesn't understand why I made the design decisions I made for correctness. If you don't paste all the mmdevapi tests with the fix in my git tree, it's a regression. I don't want to see held_Frames because it's a shadow buffer. If winmm and dsound don't work without it, they're wrong, fix those. The logic I kept for timestamping audio buffers was removed which just makes timestamping less accurate on capture buffers. Was this driver even tested on low latency? There was a reason I kept track of all packets in a linked list, no need to do held_frames logic. I genuinely do want a good driver to go in, but this just isn't it. There is also no reason to have an INCREASED period size for exclusive compared to shared. I'm currently not handling exclusive because there are no good tests for it. and "hooray_cb" ? If this is going to be accepted (likely, sadly) I want to have this added to the commit: Rejected-by: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> ~Maarten PS: Still sad that you chose to reject my original driver not based on any merits.

1 0

Re: [PATCH] sfnt2fnt: Fix broken .fon files on !x86 architectures
by Alexandre Julliard 15 Jun '12

15 Jun '12

Hilko Bengen <bengen(a)hilluzination.de> writes: > @@ -46,6 +46,8 @@ > > #include "pshpack1.h" > > +#include <endian.h> endian.h is not portable, you should use the endianness defines from the Wine headers instead. -- Alexandre Julliard julliard(a)winehq.org

4 9

Re: [PATCH (try2)] winepulse.drv: Add PulseAudio driver
by Alexandre Julliard 15 Jun '12

15 Jun '12

Andrew Eikum <aeikum(a)codeweavers.com> writes: > The configure.ac changes and parts of the driver itself were written by > Maarten Lankhorst. It doesn't work here, it's apparently using the driver even though PulseAudio is not running on this box: ../../../tools/runtest -q -P wine -M amstream.dll -T ../../.. -p amstream_test.exe.so amstream.c && touch amstream.ok err:pulse:AudioClient_Initialize PulseAudio no longer running? err:quartz:DSoundRender_create Cannot create Direct Sound object (88890010) fixme:ole:CoCreateInstance no instance created for interface {56a86895-0ad4-11ce-b03a-0020af0ba770} of class {79376820-07d0-11cf-a24d-0020afd79767}, hres is 0x88890010 amstream.c:274: Test failed: IAMMultiMediaStream_AddMediaStream returned: 88890010 -- Alexandre Julliard julliard(a)winehq.org

3 4

Re: [PATCH 1/4] DSS Test Suite: CryptAcquireContext tests for base DSS
by Alexandre Julliard 15 Jun '12

15 Jun '12

Marek Chmiel <kcmark(a)gmail.com> writes: > From 748332765355ae099ab24cf1402bed86610984e7 Mon Sep 17 00:00:00 2001 > From: Marek Chmiel <kcmark(a)gmail.com> > Date: Fri, 1 Jun 2012 22:38:03 -0500 > Subject: [PATCH 1/4] Add CryptAcquireContext tests for base DSS CSP. > > --- > configure | 1 + > configure.ac | 1 + > dlls/dssenh/tests/Makefile.in | 7 +++ > dlls/dssenh/tests/dssenh.c | 119 +++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 128 insertions(+) > create mode 100644 dlls/dssenh/tests/Makefile.in > create mode 100644 dlls/dssenh/tests/dssenh.c You are adding 4 almost identical copies of that code. Try to find a better way. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 2/2 resend] advapi32: Pass service name as argv[0] to ServiceMain
by Alexandre Julliard 15 Jun '12

15 Jun '12

Jacek Caban <jacek(a)codeweavers.com> writes: > Resend with tests. > > --- > dlls/advapi32/service.c | 3 +-- > programs/services/tests/service.c | 4 ++++ > 2 files changed, 5 insertions(+), 2 deletions(-) You will now get the name twice for autostart services. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH 1/4] hhctrl.ocx: Add HTML to Unicode decoding capability (try 4).
by Alexandre Julliard 15 Jun '12

15 Jun '12

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > + while(min <= max) > + { > + const char *encoded_symbol; > + > + pos = (min+max)/2; > + encoded_symbol = html_encoded_symbols[pos].html_code; > + dir = strncmp(encoded_symbol, amp, len); You need to check the length, strncmp is not enough. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: mmcndmgr: Implement IMMCVersionInfo (try 3)
by Marvin 15 Jun '12

15 Jun '12

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=19090 Your paranoid android. === WNT4WSSP6 (32 bit mmcndmgr) === mmcndmgr.c:40: Test failed: got 0x80040154, expected 0x00000000 === W2KPROSP4 (32 bit mmcndmgr) === mmcndmgr.c:40: Test failed: got 0x80040154, expected 0x00000000

1 0

Re: ru.po: improved an Russian translation
by Oleg Yarigin 15 Jun '12

15 Jun '12

Hi, Julian! > 1. Patches should be sent to wine-patches(a)winehq.org, not wine-devel. Thank you a lot, I didn't know these address! > I don't think that's a good idea... Afaik, the general guideline is to > not change formatting and punctuation (from the English original), > unless there's a rule in the Russian language that specifically > dictates it. In Russian there is different from English punctuation. We create a tool for humans, not machines, don't we? So, we must improve a punctuation according to a language's rules. Instead, I send my improves to Nikolay, as you recommended. > A normal, UTF-8 formatted textfile as an attachment ... should suffice, > you shouldn't need to pack it. > I guess an archive could also confuse the automated scripts I packed it to decrease a size. But if it isn't good for merging, I willn't do it. > Please understand this as constructive criticism, it will help getting > your patch accepted and not cause extra work for the maintainer > (Alexandre). It looks like you invested a lot of effort, thanks for > that. No, that's OK. There isn't something bad in constructive criticism, I think. --- Best regards, Oleg. P. S. I didn't know how to post messages into pipemail, so I replied privetly. There is a copy of my answer.

1 0

Re: [05/11] windowscodecs: Add support for more types of IFD fields. Resend.
by Alexandre Julliard 15 Jun '12

15 Jun '12

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > @@ -650,30 +672,43 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, > SWAP_ULONG(count); > type = entry->type; > SWAP_USHORT(type); > + item->value.vt = tag_to_vt(type); > value = entry->value; > SWAP_ULONG(value); > > switch (type) > { > + case IFD_BYTE: > + case IFD_SBYTE: > + if (count == 1) > + { > + item->value.u.bVal = *(BYTE *)&value; > + break; > + } > + FIXME("loading multiple byte fields is not implemented\n"); > + break; This seems to contradict the endianness support. Are you really supposed to byte-swap arrays of bytes? -- Alexandre Julliard julliard(a)winehq.org

2 1

RFC: comdlg32: Fix null lpstrFile uses in FILEDLG95_OnOpen
by Bruno Jesus 15 Jun '12

15 Jun '12

Hi, I have written a patch to fix bug 17229 and would like to request help to complete the test part. The test creates a common dialog and select a filename (file does not need to exist), it was copied from other tests in the same file (comdlg32/tests/filedlg.c). It's not possible to have a default file name because lpstrFile is used to do this and it's NULL. To fill a filename in the common dialog control that appears I have used EnumChildWindows to find the textbox and set the filename in it. The main problem is how to detect that a HWND is a textbox? Currently I'm sending SetWindowText to every control and that does work but I think the patch may be rejected by that. The patch can be downloaded at: https://testbot.winehq.org/JobDetails.pl?Key=19074 The problem is in the line 56. Thanks in advance, Bruno

1 0

Re: [PATCH 1/4] hhctrl.ocx: Add HTML to Unicode decoding capability (try 4).
by Alexandre Julliard 14 Jun '12

14 Jun '12

"Erich E. Hoover" <ehoover(a)mymail.mines.edu> writes: > Real Name: > Erich Hoover > > Description: > This patch adds the ability in HTML Help to convert HTML encoded > characters (e.g. ê) into the Unicode character equivalent. This > feature is needed by the table of contents and the index for > displaying international characters in some CHM files. As of version > 3 of the patch the decoding is done manually by parsing the HTML > characters instead of using the web browser control, the search is a > now a binary search, and it also includes some additional cleanup. It > is important to note that HTML Help only supports characters within > the ANSI code pages, so support for multi-byte characters is not > necessary. I don't see why you'd want to use ANSI code pages for this, since they will be converted to Unicode anyway. -- Alexandre Julliard julliard(a)winehq.org

2 4

Re: msvcrt/tests: Fix portuguese locale test on NT4
by Marvin 14 Jun '12

14 Jun '12

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

2 1

Re: msvcrt: Fix the name of the Portuguese locale alias.
by Bruno Jesus 14 Jun '12

14 Jun '12

On Wed, Jun 13, 2012 at 2:47 PM, Francois Gouget <fgouget(a)free.fr> wrote: > ... > dlls/msvcrt/locale.c | 2 +- > dlls/msvcrt/tests/locale.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > ... > - "portugese", "ptb", > + "portuguese", "ptb", Just a side note. Although I'm not sure I think portuguese is "ptg", brazilian portuguese is "ptb". So the correct may be: > + "portuguese", "ptg", A Portugal user would be able to check that, or maybe it's easier to write a testcase. If I'm wrong I ask apologies in advance, Bruno

3 2

DllMain() seems unused in ntoskrnl.exe
by Francois Gouget 14 Jun '12

14 Jun '12

The DllMain() function in ntoskrnl.exe appears to be unused: making it static causes a compiler warning to that effect and removing it entirely does not cause a compilation failure. Should it be removed or is there a build issue with ntoskrnl.exe? diff --git a/dlls/ntoskrnl.exe/ntoskrnl.c b/dlls/ntoskrnl.exe/ntoskrnl.c index 2b84297..ffeb2f8 100644 --- a/dlls/ntoskrnl.exe/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/ntoskrnl.c @@ -1711,30 +1711,6 @@ VOID WINAPI IoInitializeRemoveLockEx(PIO_REMOVE_LOCK lock, ULONG tag, } /***************************************************** - * DllMain - */ -BOOL WINAPI DllMain( HINSTANCE inst, DWORD reason, LPVOID reserved ) -{ - static void *handler; - LARGE_INTEGER count; - - switch(reason) - { - case DLL_PROCESS_ATTACH: - DisableThreadLibraryCalls( inst ); -#ifdef __i386__ - handler = RtlAddVectoredExceptionHandler( TRUE, vectored_handler ); -#endif - KeQueryTickCount( &count ); /* initialize the global KeTickCount */ - break; - case DLL_PROCESS_DETACH: - RtlRemoveVectoredExceptionHandler( handler ); - break; - } - return TRUE; -} - -/***************************************************** * Ke386IoSetAccessProcess (NTOSKRNL.EXE.@) */ BOOLEAN WINAPI Ke386IoSetAccessProcess(PEPROCESS *process, ULONG flag) -- 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: [11/11] windowscodecs: Implement GetMetadataFormat. Resend.
by Dmitry Timoshkov 14 Jun '12

14 Jun '12

Please ignore this patch, it doesn't really belong to this series, and correct implementation should use MetadataReaderInfo interface instead. -- Dmitry.

1 0

Re: [4/5] windowscodecs: Add test for IWICMetadataReaderInfo.
by Marvin 13 Jun '12

13 Jun '12

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

1 0

Re: [4/5] windowscodecs: Add test for IWICMetadataReaderInfo.
by Marvin 13 Jun '12

13 Jun '12

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

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