August 2011 - Wine-Commits - list.winehq.org

Maarten Lankhorst : ntdll: Use FUTEX_PRIVATE_FLAG for critical section futexes when available.
by Alexandre Julliard Aug. 4, 2011

Aug. 4, 2011

Module: wine Branch: master Commit: 57493fdd8c76e9ec3f621be8aaf4e04d9d0ade9c URL: http://source.winehq.org/git/wine.git/?a=commit;h=57493fdd8c76e9ec3f621be8a… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Wed Aug 3 20:27:43 2011 +0200 ntdll: Use FUTEX_PRIVATE_FLAG for critical section futexes when available. --- dlls/ntdll/critsection.c | 13 +++++++++++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/critsection.c b/dlls/ntdll/critsection.c index 2f1abcf..fb69b31 100644 --- a/dlls/ntdll/critsection.c +++ b/dlls/ntdll/critsection.c @@ -61,14 +61,17 @@ static inline void small_pause(void) #ifdef __linux__ +static int wait_op = 128; /*FUTEX_WAIT|FUTEX_PRIVATE_FLAG*/ +static int wake_op = 129; /*FUTEX_WAKE|FUTEX_PRIVATE_FLAG*/ + static inline int futex_wait( int *addr, int val, struct timespec *timeout ) { - return syscall( SYS_futex, addr, 0/*FUTEX_WAIT*/, val, timeout, 0, 0 ); + return syscall( SYS_futex, addr, wait_op, val, timeout, 0, 0 ); } static inline int futex_wake( int *addr, int val ) { - return syscall( SYS_futex, addr, 1/*FUTEX_WAKE*/, val, NULL, 0, 0 ); + return syscall( SYS_futex, addr, wake_op, val, NULL, 0, 0 ); } static inline int use_futexes(void) @@ -78,6 +81,12 @@ static inline int use_futexes(void) if (supported == -1) { futex_wait( &supported, 10, NULL ); + if (errno == ENOSYS) + { + wait_op = 0; /*FUTEX_WAIT*/ + wake_op = 1; /*FUTEX_WAKE*/ + futex_wait( &supported, 10, NULL ); + } supported = (errno != ENOSYS); } return supported;

1 0

Alexandre Julliard : gdi32: Update the coordinates in convert_bitmapinfo to reflect the position in the destination bitmap .
by Alexandre Julliard Aug. 4, 2011

Aug. 4, 2011

Module: wine Branch: master Commit: c309883a702bcb48db818e889cf122172a240fad URL: http://source.winehq.org/git/wine.git/?a=commit;h=c309883a702bcb48db818e889… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Aug 3 20:26:44 2011 +0200 gdi32: Update the coordinates in convert_bitmapinfo to reflect the position in the destination bitmap. --- dlls/gdi32/bitblt.c | 2 +- dlls/gdi32/dib.c | 4 ++-- dlls/gdi32/dibdrv/dc.c | 10 +++++++--- dlls/gdi32/gdi_private.h | 2 +- 4 files changed, 11 insertions(+), 7 deletions(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index c32302b..e8e2c88 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -212,7 +212,7 @@ BOOL nulldrv_StretchBlt( PHYSDEV dst_dev, struct bitblt_coords *dst, dst_info->bmiHeader.biWidth = src->visrect.right - src->visrect.left; if ((ptr = HeapAlloc( GetProcessHeap(), 0, get_dib_image_size( dst_info )))) { - err = convert_bitmapinfo( src_info, bits.ptr, &src->visrect, dst_info, ptr ); + err = convert_bitmapinfo( src_info, bits.ptr, src, dst_info, ptr ); if (bits.free) bits.free( &bits ); bits.ptr = ptr; bits.is_copy = TRUE; diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c index 4fc3f3a..d09d351 100644 --- a/dlls/gdi32/dib.c +++ b/dlls/gdi32/dib.c @@ -615,7 +615,7 @@ INT WINAPI SetDIBits( HDC hdc, HBITMAP hbitmap, UINT startscan, ptr = HeapAlloc( GetProcessHeap(), 0, get_dib_image_size( dst_info )); if (ptr) { - err = convert_bitmapinfo( src_info, src_bits.ptr, &src.visrect, dst_info, ptr ); + err = convert_bitmapinfo( src_info, src_bits.ptr, &src, dst_info, ptr ); { if (src_bits.free) src_bits.free( &src_bits ); src_bits.ptr = ptr; @@ -1134,7 +1134,7 @@ INT WINAPI GetDIBits( else dst_info->bmiHeader.biHeight = -src.height; - convert_bitmapinfo( src_info, src_bits.ptr, &src.visrect, dst_info, bits ); + convert_bitmapinfo( src_info, src_bits.ptr, &src, dst_info, bits ); if (src_bits.free) src_bits.free( &src_bits ); } else lines = abs(height); diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index 7e45772..17b2966 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -260,7 +260,7 @@ void copy_dib_color_info(dib_info *dst, const dib_info *src) } } -DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, const RECT *src_rect, +DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, struct bitblt_coords *src, const BITMAPINFO *dst_info, void *dst_bits ) { dib_info src_dib, dst_dib; @@ -271,11 +271,15 @@ DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, const RECT if ( !init_dib_info_from_bitmapinfo( &dst_dib, dst_info, dst_bits, 0 ) ) return ERROR_BAD_FORMAT; - ret = dst_dib.funcs->convert_to( &dst_dib, &src_dib, src_rect ); + ret = dst_dib.funcs->convert_to( &dst_dib, &src_dib, &src->visrect ); /* We shared the color tables, so there's no need to free the dib_infos here */ - if(!ret) return ERROR_BAD_FORMAT; + + /* update coordinates, the destination rectangle is always stored at 0,0 */ + src->x -= src->visrect.left; + src->y -= src->visrect.top; + offset_rect( &src->visrect, -src->visrect.left, -src->visrect.top ); return ERROR_SUCCESS; } diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index f0fb825..0111951 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -340,7 +340,7 @@ extern void DC_UpdateXforms( DC * dc ) DECLSPEC_HIDDEN; extern int bitmap_info_size( const BITMAPINFO * info, WORD coloruse ) DECLSPEC_HIDDEN; extern int DIB_GetBitmapInfo( const BITMAPINFOHEADER *header, LONG *width, LONG *height, WORD *planes, WORD *bpp, DWORD *compr, DWORD *size ) DECLSPEC_HIDDEN; -extern DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, const RECT *src_rect, +extern DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, struct bitblt_coords *src, const BITMAPINFO *dst_info, void *dst_bits ) DECLSPEC_HIDDEN;

1 0

Alexandre Julliard : gdi32: Use the visrect coordinates in dibdrv_PutImage.
by Alexandre Julliard Aug. 4, 2011

Aug. 4, 2011

Module: wine Branch: master Commit: 2ba1a437038f8f92c24cb3a2fcd2433b24874a4b URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ba1a437038f8f92c24cb3a2f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Aug 3 19:59:46 2011 +0200 gdi32: Use the visrect coordinates in dibdrv_PutImage. --- dlls/gdi32/dibdrv/dc.c | 17 ++++++----------- 1 files changed, 6 insertions(+), 11 deletions(-) diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index a8d197b..7e45772 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -513,7 +513,6 @@ static DWORD dibdrv_PutImage( PHYSDEV dev, HBITMAP hbitmap, HRGN clip, BITMAPINF { dib_info *dib, stand_alone; DWORD ret; - RECT rect; POINT origin; dib_info src_dib; HRGN total_clip, saved_clip = NULL; @@ -564,12 +563,8 @@ static DWORD dibdrv_PutImage( PHYSDEV dev, HBITMAP hbitmap, HRGN clip, BITMAPINF init_dib_info_from_bitmapinfo( &src_dib, info, bits->ptr, 0 ); - rect.left = dst->x; - rect.top = dst->y; - rect.right = dst->x + dst->width; - rect.bottom = dst->y + dst->height; - origin.x = src->x; - origin.y = src->y; + origin.x = src->visrect.left; + origin.y = src->visrect.top; if (hbitmap) { @@ -583,7 +578,7 @@ static DWORD dibdrv_PutImage( PHYSDEV dev, HBITMAP hbitmap, HRGN clip, BITMAPINF rop2 = ((rop >> 16) & 0xf) + 1; } - if (total_clip == NULL) dib->funcs->copy_rect( dib, &rect, &src_dib, &origin, rop2 ); + if (total_clip == NULL) dib->funcs->copy_rect( dib, &dst->visrect, &src_dib, &origin, rop2 ); else { clip_data = get_wine_region( total_clip ); @@ -591,10 +586,10 @@ static DWORD dibdrv_PutImage( PHYSDEV dev, HBITMAP hbitmap, HRGN clip, BITMAPINF { RECT clipped_rect; - if (intersect_rect( &clipped_rect, &rect, clip_data->rects + i )) + if (intersect_rect( &clipped_rect, &dst->visrect, clip_data->rects + i )) { - origin.x = src->x + clipped_rect.left - dst->x; - origin.y = src->y + clipped_rect.top - dst->y; + origin.x = src->visrect.left + clipped_rect.left - dst->visrect.left; + origin.y = src->visrect.top + clipped_rect.top - dst->visrect.top; dib->funcs->copy_rect( dib, &clipped_rect, &src_dib, &origin, rop2 ); } }

1 0

Frédéric Delanoy : cmd/tests: Fix "broken" handling when expected line is empty.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: bde872e7826b4e70e6bc5b2f73653641ef27cf99 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bde872e7826b4e70e6bc5b2f7… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Tue Aug 2 22:55:37 2011 +0200 cmd/tests: Fix "broken" handling when expected line is empty. --- programs/cmd/tests/batch.c | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/programs/cmd/tests/batch.c b/programs/cmd/tests/batch.c index 84d7f39..c4ccaab 100644 --- a/programs/cmd/tests/batch.c +++ b/programs/cmd/tests/batch.c @@ -152,6 +152,16 @@ static const char *compare_line(const char *out_line, const char *out_end, const static const char space_cmd[] = {'@','s','p','a','c','e','@'}; static const char or_broken_cmd[] = {'@','o','r','_','b','r','o','k','e','n','@'}; + /* Corner case where exp_line directly starts with @or_broken@, on Windowses */ + if(broken(1) + && exp_line+sizeof(or_broken_cmd) <= exp_end + && !memcmp(exp_line, or_broken_cmd, sizeof(or_broken_cmd))) { + if(out_line == out_end) + return NULL; + else + exp_ptr = exp_line + sizeof(or_broken_cmd); + } + while(exp_ptr < exp_end) { if(*exp_ptr == '@') { if(exp_ptr+sizeof(pwd_cmd) <= exp_end

1 0

Francois Gouget : net: Remove a debug trace.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: 89dcf229d05d20ee17aff5712d59ba30efe0d1a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=89dcf229d05d20ee17aff5712… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Aug 3 17:37:12 2011 +0200 net: Remove a debug trace. --- programs/net/net.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/programs/net/net.c b/programs/net/net.c index 815eadf..92f6ea7 100644 --- a/programs/net/net.c +++ b/programs/net/net.c @@ -68,7 +68,6 @@ static BOOL net_use(int argc, char *argv[]) { status[i] = HeapAlloc(GetProcessHeap(), 0, 1024); LoadStringA(hmod, STRING_OK+i, status[i], 1024); - printf("i=%d desc=%s\n", i, status[i]); } do {

1 0

Mariusz Pluciński : shell32/tests: Add verification of known folders parsing names.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: fe0eab4bd1b0c3d78a9adbe4a1f275b6a65616ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=fe0eab4bd1b0c3d78a9adbe4a… Author: Mariusz Pluciński <vshader(a)gmail.com> Date: Wed Aug 3 18:11:58 2011 +0200 shell32/tests: Add verification of known folders parsing names. --- dlls/shell32/tests/shellpath.c | 884 +++++++++++++++++++++++++++++++++++----- 1 files changed, 777 insertions(+), 107 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=fe0eab4bd1b0c3d78a9ad…

1 0

Austin English : msvcrt/tests: Make sure to use return values.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: 1197b5e3e31a53734a179faed56046685489713c URL: http://source.winehq.org/git/wine.git/?a=commit;h=1197b5e3e31a53734a179faed… Author: Austin English <austinenglish(a)gmail.com> Date: Tue Aug 2 12:22:11 2011 -0500 msvcrt/tests: Make sure to use return values. --- dlls/msvcrt/tests/file.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/dlls/msvcrt/tests/file.c b/dlls/msvcrt/tests/file.c index 0231f72..e2bf578 100644 --- a/dlls/msvcrt/tests/file.c +++ b/dlls/msvcrt/tests/file.c @@ -358,17 +358,21 @@ static void test_asciimode(void) fp = fopen("ascii.tst", "r"); c= fgetc(fp); + ok(c == '0', "fgetc failed, expected '0', got '%c'\n", c); c= fgetc(fp); + ok(c == '\n', "fgetc failed, expected '\\n', got '%c'\n", c); fseek(fp,0,SEEK_CUR); for(i=1; i<10; i++) { ok((j = ftell(fp)) == i*3, "ftell fails in TEXT mode\n"); fseek(fp,0,SEEK_CUR); ok((c = fgetc(fp)) == '0'+ i, "fgetc after fseek failed in line %d\n", i); c= fgetc(fp); + ok(c == '\n', "fgetc failed, expected '\\n', got '%c'\n", c); } /* Show that fseek doesn't skip \\r !*/ rewind(fp); c= fgetc(fp); + ok(c == '0', "fgetc failed, expected '0', got '%c'\n", c); fseek(fp, 2 ,SEEK_CUR); for(i=1; i<10; i++) { ok((c = fgetc(fp)) == '0'+ i, "fgetc after fseek with pos Offset failed in line %d\n", i); @@ -376,6 +380,7 @@ static void test_asciimode(void) } fseek(fp, 9*3 ,SEEK_SET); c = fgetc(fp); + ok(c == '9', "fgetc failed, expected '9', got '%c'\n", c); fseek(fp, -4 ,SEEK_CUR); for(i= 8; i>=0; i--) { ok((c = fgetc(fp)) == '0'+ i, "fgetc after fseek with neg Offset failed in line %d\n", i);

1 0

Jörg Höhle : winmm: Functions that take an open HWAVE don't need StartDevicesThread.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: c7d0c093e5bc29da6163d2ef66542562ae7bafc2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c7d0c093e5bc29da6163d2ef6… Author: Jörg Höhle <hoehle(a)users.sourceforge.net> Date: Wed Jul 27 19:32:02 2011 +0200 winmm: Functions that take an open HWAVE don't need StartDevicesThread. --- dlls/winmm/waveform.c | 60 ------------------------------------------------- 1 files changed, 0 insertions(+), 60 deletions(-) diff --git a/dlls/winmm/waveform.c b/dlls/winmm/waveform.c index 8801900..918364b 100644 --- a/dlls/winmm/waveform.c +++ b/dlls/winmm/waveform.c @@ -2370,9 +2370,6 @@ UINT WINAPI waveOutClose(HWAVEOUT hWaveOut) TRACE("(%p)\n", hWaveOut); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveOut); if(!WINMM_ValidateAndLock(device)) @@ -2398,9 +2395,6 @@ UINT WINAPI waveOutPrepareHeader(HWAVEOUT hWaveOut, { TRACE("(%p, %p, %u)\n", hWaveOut, lpWaveOutHdr, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!lpWaveOutHdr || uSize < sizeof(WAVEHDR)) return MMSYSERR_INVALPARAM; @@ -2418,9 +2412,6 @@ UINT WINAPI waveOutUnprepareHeader(HWAVEOUT hWaveOut, { TRACE("(%p, %p, %u)\n", hWaveOut, lpWaveOutHdr, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!lpWaveOutHdr || uSize < sizeof(WAVEHDR)) return MMSYSERR_INVALPARAM; @@ -2443,9 +2434,6 @@ UINT WINAPI waveOutWrite(HWAVEOUT hWaveOut, WAVEHDR *header, UINT uSize) TRACE("(%p, %p, %u)\n", hWaveOut, header, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveOut); if(!WINMM_ValidateAndLock(device)) @@ -2510,9 +2498,6 @@ UINT WINAPI waveOutBreakLoop(HWAVEOUT hWaveOut) TRACE("(%p)\n", hWaveOut); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveOut); if(!WINMM_ValidateAndLock(device)) @@ -2532,9 +2517,6 @@ UINT WINAPI waveOutPause(HWAVEOUT hWaveOut) { TRACE("(%p)\n", hWaveOut); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - return WINMM_Pause((HWAVE)hWaveOut); } @@ -2545,9 +2527,6 @@ UINT WINAPI waveOutReset(HWAVEOUT hWaveOut) { TRACE("(%p)\n", hWaveOut); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - return WINMM_Reset((HWAVE)hWaveOut); } @@ -2561,9 +2540,6 @@ UINT WINAPI waveOutRestart(HWAVEOUT hWaveOut) TRACE("(%p)\n", hWaveOut); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveOut); if(!WINMM_ValidateAndLock(device)) @@ -2590,9 +2566,6 @@ UINT WINAPI waveOutGetPosition(HWAVEOUT hWaveOut, LPMMTIME lpTime, { TRACE("(%p, %p, %u)\n", hWaveOut, lpTime, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!uSize || !lpTime || uSize != sizeof(MMTIME)) return MMSYSERR_INVALPARAM; @@ -2650,9 +2623,6 @@ UINT WINAPI waveOutGetVolume(HWAVEOUT hWaveOut, DWORD *out) TRACE("(%p, %p)\n", hWaveOut, out); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!out) return MMSYSERR_INVALPARAM; @@ -2705,9 +2675,6 @@ UINT WINAPI waveOutSetVolume(HWAVEOUT hWaveOut, DWORD in) TRACE("(%p, %08x)\n", hWaveOut, in); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveOut); if(!WINMM_ValidateAndLock(device)) @@ -2764,9 +2731,6 @@ UINT WINAPI waveOutGetID(HWAVEOUT hWaveOut, UINT* lpuDeviceID) TRACE("(%p, %p)\n", hWaveOut, lpuDeviceID); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!lpuDeviceID) return MMSYSERR_INVALPARAM; @@ -3054,9 +3018,6 @@ UINT WINAPI waveInClose(HWAVEIN hWaveIn) TRACE("(%p)\n", hWaveIn); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveIn); if(!WINMM_ValidateAndLock(device)) @@ -3082,9 +3043,6 @@ UINT WINAPI waveInPrepareHeader(HWAVEIN hWaveIn, WAVEHDR* lpWaveInHdr, { TRACE("(%p, %p, %u)\n", hWaveIn, lpWaveInHdr, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!lpWaveInHdr || uSize < sizeof(WAVEHDR)) return MMSYSERR_INVALPARAM; @@ -3102,9 +3060,6 @@ UINT WINAPI waveInUnprepareHeader(HWAVEIN hWaveIn, WAVEHDR* lpWaveInHdr, { TRACE("(%p, %p, %u)\n", hWaveIn, lpWaveInHdr, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!lpWaveInHdr || uSize < sizeof(WAVEHDR)) return MMSYSERR_INVALPARAM; @@ -3161,9 +3116,6 @@ UINT WINAPI waveInReset(HWAVEIN hWaveIn) { TRACE("(%p)\n", hWaveIn); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - return WINMM_Reset((HWAVE)hWaveIn); } @@ -3177,9 +3129,6 @@ UINT WINAPI waveInStart(HWAVEIN hWaveIn) TRACE("(%p)\n", hWaveIn); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveIn); if(!WINMM_ValidateAndLock(device)) @@ -3208,9 +3157,6 @@ UINT WINAPI waveInStop(HWAVEIN hWaveIn) TRACE("(%p)\n", hWaveIn); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - device = WINMM_GetDeviceFromHWAVE((HWAVE)hWaveIn); if(!WINMM_ValidateAndLock(device)) @@ -3249,9 +3195,6 @@ UINT WINAPI waveInGetPosition(HWAVEIN hWaveIn, LPMMTIME lpTime, { TRACE("(%p, %p, %u)\n", hWaveIn, lpTime, uSize); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!uSize || !lpTime || uSize != sizeof(MMTIME)) return MMSYSERR_INVALPARAM; @@ -3269,9 +3212,6 @@ UINT WINAPI waveInGetID(HWAVEIN hWaveIn, UINT* lpuDeviceID) TRACE("(%p, %p)\n", hWaveIn, lpuDeviceID); - if(!WINMM_StartDevicesThread()) - return MMSYSERR_ERROR; - if(!lpuDeviceID) return MMSYSERR_INVALPARAM;

1 0

Dan Kegel : cmd: mkdir: Set errorlevel on invalid argument.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: 3cd864bdf8a4565442a57a05a91b372ae0a4ca70 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3cd864bdf8a4565442a57a05a… Author: Dan Kegel <dank(a)kegel.com> Date: Tue Aug 2 12:21:31 2011 -0700 cmd: mkdir: Set errorlevel on invalid argument. --- programs/cmd/builtins.c | 1 + programs/cmd/tests/test_builtins.cmd | 8 ++++---- programs/cmd/tests/test_builtins.cmd.exp | 8 ++++---- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c index 58050cf..7e5ceaf 100644 --- a/programs/cmd/builtins.c +++ b/programs/cmd/builtins.c @@ -537,6 +537,7 @@ void WCMD_create_dir (WCHAR *command) { if (!argN) break; if (!create_full_path(thisArg)) { WCMD_print_error (); + errorlevel = 1; } } } diff --git a/programs/cmd/tests/test_builtins.cmd b/programs/cmd/tests/test_builtins.cmd index be1eead..2cbff33 100644 --- a/programs/cmd/tests/test_builtins.cmd +++ b/programs/cmd/tests/test_builtins.cmd @@ -485,18 +485,18 @@ if exist foo (rmdir foo & echo dir created echo %ErrorLevel% rem Invalid chars mkdir ? -echo %ErrorLevel% +echo mkdir ? gives errorlevel %ErrorLevel% call :setError 0 mkdir ?\foo -echo %ErrorLevel% +echo mkdir ?\foo gives errorlevel %ErrorLevel% call :setError 0 mkdir foo\? -echo %ErrorLevel% +echo mkdir foo\? gives errorlevel %ErrorLevel% if exist foo (rmdir foo & echo ok, foo created ) else ( echo foo not created ) call :setError 0 mkdir foo\bar\? -echo %ErrorLevel% +echo mkdir foo\bar\? gives errorlevel %ErrorLevel% call :setError 0 if not exist foo ( echo bad, foo not created diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp index 8db75a7..f64150d 100644 --- a/programs/cmd/tests/test_builtins.cmd.exp +++ b/programs/cmd/tests/test_builtins.cmd.exp @@ -277,11 +277,11 @@ del /q * succeeded on file2.dat 0 dir created 0 -(a)todo_wine@1 -(a)todo_wine@1 -(a)todo_wine@1 +mkdir ? gives errorlevel 1 +mkdir ?\foo gives errorlevel 1 +mkdir foo\? gives errorlevel 1 @todo_wine(a)ok, foo created -(a)todo_wine@1 +mkdir foo\bar\? gives errorlevel 1 @todo_wine(a)ok, foo\bar created foo created bar created

1 0

Dan Kegel : cmd: mkdir: Handle multiple arguments.
by Alexandre Julliard Aug. 3, 2011

Aug. 3, 2011

Module: wine Branch: master Commit: 15ce9a5784eb36ad619071a66da8a01395246126 URL: http://source.winehq.org/git/wine.git/?a=commit;h=15ce9a5784eb36ad619071a66… Author: Dan Kegel <dank(a)kegel.com> Date: Tue Aug 2 13:02:19 2011 -0700 cmd: mkdir: Handle multiple arguments. --- programs/cmd/builtins.c | 13 +++++++++++-- programs/cmd/tests/test_builtins.cmd.exp | 6 +++--- programs/cmd/wcmd.h | 2 +- programs/cmd/wcmdmain.c | 2 +- 4 files changed, 16 insertions(+), 7 deletions(-) diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c index 49869ba..58050cf 100644 --- a/programs/cmd/builtins.c +++ b/programs/cmd/builtins.c @@ -523,13 +523,22 @@ static BOOL create_full_path(WCHAR* path) return ret; } -void WCMD_create_dir (void) { +void WCMD_create_dir (WCHAR *command) { + int argno = 0; + WCHAR *argN = command; if (param1[0] == 0x00) { WCMD_output (WCMD_LoadMessage(WCMD_NOARG)); return; } - if (!create_full_path(param1)) WCMD_print_error (); + /* Loop through all args */ + while (TRUE) { + WCHAR *thisArg = WCMD_parameter(command, argno++, &argN); + if (!argN) break; + if (!create_full_path(thisArg)) { + WCMD_print_error (); + } + } } /* Parse the /A options given by the user on the commandline diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp index 463b225..8db75a7 100644 --- a/programs/cmd/tests/test_builtins.cmd.exp +++ b/programs/cmd/tests/test_builtins.cmd.exp @@ -284,9 +284,9 @@ dir created @todo_wine(a)1 @todo_wine(a)ok, foo\bar created foo created -(a)todo_wine@bar created -(a)todo_wine@foobar created -(a)todo_wine@bar\baz created +bar created +foobar created +bar\baz created ----------- Testing rmdir ----------- 0 dir removed diff --git a/programs/cmd/wcmd.h b/programs/cmd/wcmd.h index 939e404..8ed7a01 100644 --- a/programs/cmd/wcmd.h +++ b/programs/cmd/wcmd.h @@ -56,7 +56,7 @@ void WCMD_choice (WCHAR *); void WCMD_clear_screen (void); void WCMD_color (void); void WCMD_copy (void); -void WCMD_create_dir (void); +void WCMD_create_dir (WCHAR *); BOOL WCMD_delete (WCHAR *); void WCMD_directory (WCHAR *); void WCMD_echo (const WCHAR *); diff --git a/programs/cmd/wcmdmain.c b/programs/cmd/wcmdmain.c index f3fa129..5dbf74c 100644 --- a/programs/cmd/wcmdmain.c +++ b/programs/cmd/wcmdmain.c @@ -1472,7 +1472,7 @@ void WCMD_execute (WCHAR *command, WCHAR *redirects, break; case WCMD_MD: case WCMD_MKDIR: - WCMD_create_dir (); + WCMD_create_dir (p); break; case WCMD_MOVE: WCMD_move ();

1 0