wine-commits April 2008

wine-commits@winehq.org

4 participants
1278 discussions

Michael Stefaniuc : fonts: Add the Romanian glyphs SsTt with "comma below" to the tahoma font.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: adc78836b1239dbb8747b6f63bdab1f5eeaf53a3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=adc78836b1239dbb8747b6f63… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Apr 16 00:47:05 2008 +0200 fonts: Add the Romanian glyphs SsTt with "comma below" to the tahoma font. --- fonts/tahoma.sfd | 134 ++++++++++++++++++++++++++++++++++++++++++++++++++---- fonts/tahoma.ttf | Bin 84400 -> 85116 bytes 2 files changed, 124 insertions(+), 10 deletions(-) diff --git a/fonts/tahoma.sfd b/fonts/tahoma.sfd index c62946f..27eb125 100644 --- a/fonts/tahoma.sfd +++ b/fonts/tahoma.sfd @@ -12,13 +12,13 @@ UnderlinePosition: -170 UnderlineWidth: 130 Ascent: 1638 Descent: 410 -XUID: [1021 135 1692684408 1110817] +XUID: [1021 135 1692684408 1110822] FSType: 0 OS2Version: 2 OS2_WeightWidthSlopeOnly: 0 OS2_UseTypoMetrics: 1 CreationTime: 1169210120 -ModificationTime: 1206566066 +ModificationTime: 1208293008 PfmFamily: 33 TTFWeight: 400 TTFWidth: 5 @@ -94,7 +94,7 @@ NameList: Adobe Glyph List DisplaySize: 16 AntiAlias: 1 FitToEm: 1 -WinInfo: 0 28 11 +WinInfo: 336 28 11 BeginPrivate: 10 BlueValues 27 [-30 0 1120 1177 1522 1522] OtherBlues 11 [-426 -426] @@ -108,7 +108,7 @@ SnapStemV 9 [184 203] ExpansionFactor 4 0.06 EndPrivate TeXData: 1 0 0 327680 163840 109226 571904 -1048576 109226 783286 444596 497025 792723 393216 433062 380633 303038 157286 324010 404750 52429 2506097 1059062 262144 -BeginChars: 65537 425 +BeginChars: 65537 430 StartChar: .notdef Encoding: 0 0 0 Width: 2048 @@ -9179,8 +9179,62 @@ Fore 936 1671 l 1 EndSplineSet EndChar +StartChar: uni0326 +Encoding: 806 806 425 +Width: 1118 +Flags: W +HStem: -411 276<447 524> +VStem: 361 308 +Fore +361 -411 m 1 + 447 -135 l 1 + 669 -135 l 1 + 524 -411 l 5 + 361 -411 l 1 +EndSplineSet +EndChar +StartChar: Scommaaccent +Encoding: 536 536 426 +Width: 1141 +Flags: W +HStem: -411 276<447 524> -17 164<338.667 837.867> 1369 163<435.993 753.733> +VStem: 89 206<1023.33 1289.2> 361 308 886 211<181.933 591> +Fore +Refer: 425 806 S 1 0 0 1 0 0 2 +Refer: 52 83 N 1 0 0 1 0 0 2 +EndChar +StartChar: uni021A +Encoding: 538 538 427 +Width: 1196 +Flags: W +HStem: -410 276<482 559> 0 21<498 701> 1318 171<3 498 701 1198> +VStem: 396 308 498 203<0 1318> +Fore +Refer: 425 806 S 1 0 0 1 35 1 2 +Refer: 53 84 N 1 0 0 1 0 0 2 +EndChar +StartChar: scommaaccent +Encoding: 537 537 428 +Width: 913 +Flags: W +HStem: -410 276<373 450> -41 154<298.856 587.533> 981 154<342.2 643.267> +VStem: 76 178<740.8 952.933> 287 308 696 189<143.6 347.133> +Fore +Refer: 425 806 S 1 0 0 1 -74 1 2 +Refer: 84 115 N 1 0 0 1 0 0 2 +EndChar +StartChar: uni021B +Encoding: 539 539 429 +Width: 684 +Flags: W +HStem: -410 276<303 380> 0 154<426 719> 977 143<20 155 340 719> +VStem: 155 185<274.336 977 1120 1438> 217 308 +Fore +Refer: 425 806 S 1 0 0 1 -144 1 2 +Refer: 85 116 N 1 0 0 1 0 0 2 +EndChar EndChars -BitmapFont: 9 425 7 2 1 +BitmapFont: 9 430 7 2 1 BDFChar: 0 0 9 1 7 0 6 rdo`RJqEt% BDFChar: 1 32 3 0 0 0 0 @@ -10031,8 +10085,18 @@ BDFChar: 423 371 5 0 4 -2 4 OH>QcE"Eid BDFChar: 424 175 5 0 3 7 7 n,NFg +BDFChar: 425 806 5 3 3 -1 -1 +J,fQL +BDFChar: 426 536 5 0 4 -2 6 +G^t<`#RKo"+92BA +BDFChar: 427 538 5 0 4 -2 6 +p`L\%+<Vd,+92BA +BDFChar: 428 537 4 0 2 -2 4 +@":Kb^]6V" +BDFChar: 429 539 3 0 1 -2 5 +JA?\cJ3X*b EndBitmapFont -BitmapFont: 11 425 9 2 1 FontForge +BitmapFont: 11 430 9 2 1 FontForge BDFStartProperties: 36 FONT 1 "-FontForge-Tahoma-Normal-R-Normal--11-80-96-96-P-62-ISO10646-1" FONTBOUNDINGBOX 1 "15 11 0 -2" @@ -10918,8 +10982,18 @@ BDFChar: 423 371 6 0 4 -2 5 LkpkCLj0IJ BDFChar: 424 175 6 0 5 9 9 qu?]s +BDFChar: 425 806 6 2 3 -2 -1 +5_&h7 +BDFChar: 426 536 6 0 4 -3 7 +G^t<`#RC_*!$FN, +BDFChar: 427 538 6 0 4 -3 7 +p`L\%+<VdL!$FN, +BDFChar: 428 537 5 0 3 -3 5 +E.G_X&E!O25QCca +BDFChar: 429 539 4 0 2 -3 7 +J:QR.J:N/X!'l&7 EndBitmapFont -BitmapFont: 12 425 10 2 1 FontForge +BitmapFont: 12 430 10 2 1 FontForge BDFStartProperties: 36 FONT 1 "-FontForge-Tahoma-Normal-R-Normal--12-90-96-96-P-70-ISO10646-1" FONTBOUNDINGBOX 1 "16 12 0 -2" @@ -11601,8 +11675,18 @@ BDFChar: 423 371 7 0 5 -2 6 KS5#3KT(!Y$ig8- BDFChar: 424 175 7 0 6 9 9 rVuou +BDFChar: 425 806 7 2 3 -2 -1 +5_&h7 +BDFChar: 426 536 7 0 5 -3 8 +G_CTtGQ\/&GQ89t +BDFChar: 427 538 7 0 6 -3 8 +rX]VP&.fBa&-*7a +BDFChar: 428 537 5 0 3 -3 6 +E.EIH&.mb"+@#o, +BDFChar: 429 539 4 0 3 -3 8 +5X=g(5X7S"0E<4\ EndBitmapFont -BitmapFont: 13 425 10 3 1 FontForge +BitmapFont: 13 430 10 3 1 FontForge BDFStartProperties: 36 FONT 1 "-FontForge-Tahoma-Normal-R-Normal--13-100-96-96-P-73-ISO10646-1" FONTBOUNDINGBOX 1 "15 13 0 -3" @@ -12284,8 +12368,18 @@ BDFChar: 423 371 7 0 5 -2 6 KS5#3KT(!Y$ig8- BDFChar: 424 175 7 0 6 10 10 rVuou +BDFChar: 425 806 7 2 3 -2 -1 +5_&h7 +BDFChar: 426 536 7 0 6 -3 8 +I"Hm!HiaA$HiO^3 +BDFChar: 427 538 8 0 6 -3 8 +rX]VP&.fBa&-*7a +BDFChar: 428 537 6 0 4 -3 6 +E/9$`#`*F"+CG0L +BDFChar: 429 539 4 0 3 -3 8 +5X=g(5X7S"0E<4\ EndBitmapFont -BitmapFont: 15 425 12 3 1 FontForge +BitmapFont: 15 430 12 3 1 FontForge BDFStartProperties: 36 FONT 1 "-FontForge-Tahoma-Normal-R-Normal--15-110-96-96-P-85-ISO10646-1" FONTBOUNDINGBOX 1 "19 15 0 -3" @@ -12967,8 +13061,18 @@ BDFChar: 423 371 8 0 6 -2 7 JqAT+JqA`'"9ni+ BDFChar: 424 175 8 0 7 12 12 rr<$! +BDFChar: 425 806 8 2 4 -3 -2 +@))aB +BDFChar: 426 536 8 0 6 -3 10 +4@4RK5WAlJ!ebdu#T*OA +BDFChar: 427 538 9 0 8 -3 10 +s*t@T#QP,1#QP,1#QP,1#QP,1#QOi)#QP\A +BDFChar: 428 537 7 0 5 -3 7 +G_CTT(^,M:!"^ga +BDFChar: 429 539 5 0 4 -3 9 +5X>*05X7S"5WA`T0E;(Q EndBitmapFont -BitmapFont: 16 425 13 3 1 FontForge +BitmapFont: 16 430 13 3 1 FontForge BDFStartProperties: 36 FONT 1 "-FontForge-Tahoma-Normal-R-Normal--16-120-96-96-P-93-ISO10646-1" FONTBOUNDINGBOX 1 "21 16 0 -3" @@ -13650,5 +13754,15 @@ BDFChar: 423 371 9 0 7 -3 8 JUrB'JUrA>4TY`g BDFChar: 424 175 9 0 8 13 14 !!*%L +BDFChar: 425 806 9 2 4 -4 -2 +?pK5B +BDFChar: 426 536 9 0 7 -3 11 +5!a^LJ8bu!!<Ie"!!jDA +BDFChar: 427 538 9 0 8 -3 11 +s*t@T#QP,1#QP,1#QP,1#QP,1#QP,1!!!9)(]XO9 +BDFChar: 428 537 7 0 5 -3 8 +G_CTtGQ\/&GQ8:/ +BDFChar: 429 539 5 0 4 -3 11 +5X7U05X7S"5X7Ro!"^ga EndBitmapFont EndSplineFont diff --git a/fonts/tahoma.ttf b/fonts/tahoma.ttf index e54313c..08ce349 100644 Binary files a/fonts/tahoma.ttf and b/fonts/tahoma.ttf differ

1 0

Dan Hipschman : qmgr/tests: Fix minor potential memory leaks.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: cacd1c448e633852c77bfa87572aa12895d7e86a URL: http://source.winehq.org/git/wine.git/?a=commit;h=cacd1c448e633852c77bfa875… Author: Dan Hipschman <dsh(a)linux.ucla.edu> Date: Tue Apr 15 11:08:48 2008 -0700 qmgr/tests: Fix minor potential memory leaks. --- dlls/qmgr/tests/enum_jobs.c | 3 +++ dlls/qmgr/tests/job.c | 6 ++++++ 2 files changed, 9 insertions(+), 0 deletions(-) diff --git a/dlls/qmgr/tests/enum_jobs.c b/dlls/qmgr/tests/enum_jobs.c index 2b73631..f95404c 100644 --- a/dlls/qmgr/tests/enum_jobs.c +++ b/dlls/qmgr/tests/enum_jobs.c @@ -209,6 +209,7 @@ static void test_Next_walkList_2(void) if(hres != S_OK) { skip("Unable to get file from test_enumJobs\n"); + HeapFree(GetProcessHeap(), 0, jobs); return; } ok(fetched == test_jobCountB, "Next returned the incorrect number of jobs: %08x\n", hres); @@ -219,6 +220,8 @@ static void test_Next_walkList_2(void) if (jobs[i]) IBackgroundCopyFile_Release(jobs[i]); } + + HeapFree(GetProcessHeap(), 0, jobs); } /* Test Next Error conditions */ diff --git a/dlls/qmgr/tests/job.c b/dlls/qmgr/tests/job.c index 52d4e82..36201f8 100644 --- a/dlls/qmgr/tests/job.c +++ b/dlls/qmgr/tests/job.c @@ -416,6 +416,8 @@ static void test_CompleteLocalURL(void) if (!urlA || !urlB) { skip("Unable to allocate memory for URLs\n"); + HeapFree(GetProcessHeap(), 0, urlA); + HeapFree(GetProcessHeap(), 0, urlB); return; } @@ -428,6 +430,8 @@ static void test_CompleteLocalURL(void) if (hres != S_OK) { skip("Unable to add file to job\n"); + HeapFree(GetProcessHeap(), 0, urlA); + HeapFree(GetProcessHeap(), 0, urlB); return; } @@ -435,6 +439,8 @@ static void test_CompleteLocalURL(void) if (hres != S_OK) { skip("Unable to add file to job\n"); + HeapFree(GetProcessHeap(), 0, urlA); + HeapFree(GetProcessHeap(), 0, urlB); return; }

1 0

Andrew Talbot : dmime: Remove unused variables.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: a73ab629e9cc5877060411d38cf82405009dbe91 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a73ab629e9cc5877060411d38… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Tue Apr 15 21:33:30 2008 +0100 dmime: Remove unused variables. --- dlls/dmime/graph.c | 7 +++---- dlls/dmime/performance.c | 3 +-- dlls/dmime/segment.c | 16 ++++++---------- 3 files changed, 10 insertions(+), 16 deletions(-) diff --git a/dlls/dmime/graph.c b/dlls/dmime/graph.c index 7d0363a..79ff823 100644 --- a/dlls/dmime/graph.c +++ b/dlls/dmime/graph.c @@ -108,7 +108,6 @@ static HRESULT WINAPI IDirectMusicGraphImpl_IDirectMusicGraph_StampPMsg (LPDIREC static HRESULT WINAPI IDirectMusicGraphImpl_IDirectMusicGraph_InsertTool (LPDIRECTMUSICGRAPH iface, IDirectMusicTool* pTool, DWORD* pdwPChannels, DWORD cPChannels, LONG lIndex) { ICOM_THIS_MULTI(IDirectMusicGraphImpl, GraphVtbl, iface); - HRESULT hr; struct list* pEntry = NULL; struct list* pPrevEntry = NULL; LPDMUS_PRIVATE_GRAPH_TOOL pIt = NULL; @@ -142,14 +141,14 @@ static HRESULT WINAPI IDirectMusicGraphImpl_IDirectMusicGraph_InsertTool (LPDIRE pNewTool->pTool = pTool; pNewTool->dwIndex = lIndex; IDirectMusicTool8_AddRef(pTool); - hr = IDirectMusicTool8_Init(pTool, iface); + IDirectMusicTool8_Init(pTool, iface); list_add_tail (pPrevEntry->next, &pNewTool->entry); #if 0 DWORD dwNum = 0; - hr = IDirectMusicTool8_GetMediaTypes(pTool, &dwNum); + IDirectMusicTool8_GetMediaTypes(pTool, &dwNum); #endif - + return DS_OK; } diff --git a/dlls/dmime/performance.c b/dlls/dmime/performance.c index 9730104..ad3d740 100644 --- a/dlls/dmime/performance.c +++ b/dlls/dmime/performance.c @@ -222,8 +222,7 @@ static HRESULT WINAPI IDirectMusicPerformance8Impl_Init (LPDIRECTMUSICPERFORMANC This->pDirectSound = pDirectSound; IDirectSound_AddRef(This->pDirectSound); } else { - HRESULT hr; - hr = DirectSoundCreate8(NULL, (LPDIRECTSOUND8*) &This->pDirectSound, NULL); + DirectSoundCreate8(NULL, (LPDIRECTSOUND8*) &This->pDirectSound, NULL); if (!This->pDirectSound) return DSERR_NODRIVER; if (NULL != hWnd) { diff --git a/dlls/dmime/segment.c b/dlls/dmime/segment.c index a90bd4c..6b7243a 100644 --- a/dlls/dmime/segment.c +++ b/dlls/dmime/segment.c @@ -822,9 +822,9 @@ static HRESULT IDirectMusicSegment8Impl_IPersistStream_ParseTrackForm (LPPERSIST /*ICOM_THIS_MULTI(IDirectMusicSegment8Impl, PersistStreamVtbl, iface);*/ HRESULT hr = E_FAIL; DMUS_PRIVATE_CHUNK Chunk; - DWORD StreamSize, StreamCount, ListSize[3], ListCount[3]; + DWORD StreamSize, StreamCount, ListSize[3]; LARGE_INTEGER liMove; /* used when skipping chunks */ - + DMUS_IO_TRACK_HEADER track_hdr; DMUS_IO_TRACK_EXTRAS_HEADER track_xhdr; IDirectMusicTrack* pTrack = NULL; @@ -869,7 +869,6 @@ static HRESULT IDirectMusicSegment8Impl_IPersistStream_ParseTrackForm (LPPERSIST IStream_Read (pStm, &Chunk.fccID, sizeof(FOURCC), NULL); TRACE_(dmfile)(": LIST chunk of type %s", debugstr_fourcc(Chunk.fccID)); ListSize[0] = Chunk.dwSize - sizeof(FOURCC); - ListCount[0] = 0; if (Chunk.fccID == track_hdr.fccType && 0 == track_hdr.ckid) { LPSTREAM pClonedStream = NULL; @@ -982,7 +981,7 @@ static HRESULT IDirectMusicSegment8Impl_IPersistStream_ParseTrackList (LPPERSIST HRESULT hr = E_FAIL; DMUS_PRIVATE_CHUNK Chunk; - DWORD StreamSize, StreamCount, ListSize[3], ListCount[3]; + DWORD StreamSize, ListSize[3], ListCount[3]; LARGE_INTEGER liMove; /* used when skipping chunks */ if (pChunk->fccID != DMUS_FOURCC_TRACK_LIST) { @@ -1002,7 +1001,6 @@ static HRESULT IDirectMusicSegment8Impl_IPersistStream_ParseTrackList (LPPERSIST IStream_Read (pStm, &Chunk.fccID, sizeof(FOURCC), NULL); TRACE_(dmfile)(": RIFF chunk of type %s", debugstr_fourcc(Chunk.fccID)); StreamSize = Chunk.dwSize - sizeof(FOURCC); - StreamCount = 0; switch (Chunk.fccID) { case DMUS_FOURCC_TRACK_FORM: { TRACE_(dmfile)(": TRACK form\n"); @@ -1192,14 +1190,13 @@ static HRESULT IDirectMusicSegment8Impl_IPersistStream_LoadWave (LPPERSISTSTREAM static HRESULT WINAPI IDirectMusicSegment8Impl_IPersistStream_Load (LPPERSISTSTREAM iface, IStream* pStm) { ICOM_THIS_MULTI(IDirectMusicSegment8Impl, PersistStreamVtbl, iface); - + HRESULT hr; DMUS_PRIVATE_CHUNK Chunk; - DWORD StreamSize, StreamCount; + DWORD StreamSize; /*DWORD ListSize[3], ListCount[3];*/ LARGE_INTEGER liMove; /* used when skipping chunks */ - - + TRACE("(%p, %p): Loading\n", This, pStm); IStream_Read (pStm, &Chunk, sizeof(FOURCC)+sizeof(DWORD), NULL); TRACE_(dmfile)(": %s chunk (size = %d)", debugstr_fourcc (Chunk.fccID), Chunk.dwSize); @@ -1208,7 +1205,6 @@ static HRESULT WINAPI IDirectMusicSegment8Impl_IPersistStream_Load (LPPERSISTSTR IStream_Read (pStm, &Chunk.fccID, sizeof(FOURCC), NULL); TRACE_(dmfile)(": RIFF chunk of type %s", debugstr_fourcc(Chunk.fccID)); StreamSize = Chunk.dwSize - sizeof(FOURCC); - StreamCount = 0; switch (Chunk.fccID) { case DMUS_FOURCC_SEGMENT_FORM: { TRACE_(dmfile)(": segment form\n");

1 0

Andrew Talbot : comctl32: Remove unused variables.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 161d3322d697d330ac8e1a7f4fa2cfd3991162e0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=161d3322d697d330ac8e1a7f4… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Tue Apr 15 22:19:12 2008 +0100 comctl32: Remove unused variables. --- dlls/comctl32/syslink.c | 8 +++----- dlls/comctl32/toolbar.c | 3 --- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/dlls/comctl32/syslink.c b/dlls/comctl32/syslink.c index 02a0385..a1b8aff 100644 --- a/dlls/comctl32/syslink.c +++ b/dlls/comctl32/syslink.c @@ -967,16 +967,14 @@ static HFONT SYSLINK_SetFont (SYSLINK_INFO *infoPtr, HFONT hFont, BOOL bRedraw) */ static LRESULT SYSLINK_SetText (SYSLINK_INFO *infoPtr, LPCWSTR Text) { - int textlen; - /* clear the document */ SYSLINK_ClearDoc(infoPtr); - - if(Text == NULL || (textlen = lstrlenW(Text)) == 0) + + if(Text == NULL || *Text == 0) { return TRUE; } - + /* let's parse the string and create a document */ if(SYSLINK_ParseText(infoPtr, Text) > 0) { diff --git a/dlls/comctl32/toolbar.c b/dlls/comctl32/toolbar.c index 9991683..5746928 100644 --- a/dlls/comctl32/toolbar.c +++ b/dlls/comctl32/toolbar.c @@ -5778,7 +5778,6 @@ TOOLBAR_LButtonUp (HWND hwnd, WPARAM wParam, LPARAM lParam) POINT pt; INT nHit; INT nOldIndex = -1; - BOOL bSendMessage = TRUE; NMHDR hdr; NMMOUSE nmmouse; NMTOOLBARA nmtb; @@ -5865,8 +5864,6 @@ TOOLBAR_LButtonUp (HWND hwnd, WPARAM wParam, LPARAM lParam) if (btnPtr->fsStyle & BTNS_GROUP) { nOldIndex = TOOLBAR_GetCheckedGroupButtonIndex (infoPtr, nHit); - if (nOldIndex == nHit) - bSendMessage = FALSE; if ((nOldIndex != nHit) && (nOldIndex != -1)) infoPtr->buttons[nOldIndex].fsState &= ~TBSTATE_CHECKED;

1 0

Andrew Talbot : dinput: Fix handle leak.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 6657cfe70de94bf6ebd345800ef2eec8d4e04bd9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6657cfe70de94bf6ebd345800… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Tue Apr 15 22:54:03 2008 +0100 dinput: Fix handle leak. --- dlls/dinput/joystick_linux.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/dinput/joystick_linux.c b/dlls/dinput/joystick_linux.c index e9b8790..5e87a02 100644 --- a/dlls/dinput/joystick_linux.c +++ b/dlls/dinput/joystick_linux.c @@ -151,6 +151,8 @@ static INT find_joystick_devices(void) if ((fd = open(device_name, O_RDONLY)) < 0) continue; } + close(fd); + if (!(str = HeapAlloc(GetProcessHeap(), 0, len))) break; memcpy(str, device_name, len);

1 0

Andrew Talbot : browseui: Fix handle leak.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 160945e217bee92db266c8d8f19466978384f577 URL: http://source.winehq.org/git/wine.git/?a=commit;h=160945e217bee92db266c8d8f… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Tue Apr 15 22:11:26 2008 +0100 browseui: Fix handle leak. --- dlls/browseui/progressdlg.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/browseui/progressdlg.c b/dlls/browseui/progressdlg.c index c5465a3..c65a097 100644 --- a/dlls/browseui/progressdlg.c +++ b/dlls/browseui/progressdlg.c @@ -324,6 +324,7 @@ static HRESULT WINAPI ProgressDialog_StartProgressDialog(IProgressDialog *iface, hThread = CreateThread(NULL, 0, dialog_thread, &params, 0, NULL); WaitForSingleObject(params.hEvent, INFINITE); + CloseHandle(hThread); This->hwndDisabledParent = NULL; if (hwndParent && (dwFlags & PROGDLG_MODAL)) {

1 0

Rob Shearman : widl: Change alls calls to error in the parser to error_loc so that line number information is printed .
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 4b56ff7238b6a2bc12489694693d6ac3c301f775 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b56ff7238b6a2bc124896946… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 15 21:52:35 2008 +0100 widl: Change alls calls to error in the parser to error_loc so that line number information is printed. --- tools/widl/parser.y | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index fba941e..45471bb 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1190,7 +1190,7 @@ static expr_t *make_expr1(enum expr_type type, expr_t *expr) { expr_t *e; if (type == EXPR_ADDRESSOF && expr->type != EXPR_IDENTIFIER) - error("address-of operator applied to invalid expression\n"); + error_loc("address-of operator applied to invalid expression\n"); e = xmalloc(sizeof(expr_t)); e->type = type; e->ref = expr; @@ -1394,7 +1394,7 @@ static void set_type(var_t *v, type_t *type, const pident_t *pident, array_dims_ } } else if (!arr && ptr_attr) - error("%s: pointer attribute applied to non-pointer type\n", v->name); + error_loc("%s: pointer attribute applied to non-pointer type\n", v->name); } if (pident && pident->is_func) { @@ -1416,7 +1416,7 @@ static void set_type(var_t *v, type_t *type, const pident_t *pident, array_dims_ if (arr) LIST_FOR_EACH_ENTRY_REV(dim, arr, expr_t, entry) { if (sizeless) - error("%s: only the first array dimension can be unspecified\n", v->name); + error_loc("%s: only the first array dimension can be unspecified\n", v->name); if (dim->is_const) { @@ -1424,10 +1424,10 @@ static void set_type(var_t *v, type_t *type, const pident_t *pident, array_dims_ size_t size = type_memsize(v->type, &align); if (dim->cval <= 0) - error("%s: array dimension must be positive\n", v->name); + error_loc("%s: array dimension must be positive\n", v->name); if (0xffffffffuL / size < (unsigned long) dim->cval) - error("%s: total array size is too large\n", v->name); + error_loc("%s: total array size is too large\n", v->name); else if (0xffffuL < size * dim->cval) v->type = make_type(RPC_FC_LGFARRAY, v->type); else @@ -1453,10 +1453,10 @@ static void set_type(var_t *v, type_t *type, const pident_t *pident, array_dims_ atype = *ptype = duptype(*ptype, 0); if (atype->type == RPC_FC_SMFARRAY || atype->type == RPC_FC_LGFARRAY) - error("%s: cannot specify size_is for a fixed sized array\n", v->name); + error_loc("%s: cannot specify size_is for a fixed sized array\n", v->name); if (atype->type != RPC_FC_CARRAY && !is_ptr(atype)) - error("%s: size_is attribute applied to illegal type\n", v->name); + error_loc("%s: size_is attribute applied to illegal type\n", v->name); atype->type = RPC_FC_CARRAY; atype->size_is = dim; @@ -1464,7 +1464,7 @@ static void set_type(var_t *v, type_t *type, const pident_t *pident, array_dims_ ptype = &(*ptype)->ref; if (*ptype == NULL) - error("%s: too many expressions in size_is attribute\n", v->name); + error_loc("%s: too many expressions in size_is attribute\n", v->name); } ptype = &v->type; @@ -1482,14 +1482,14 @@ static void set_type(var_t *v, type_t *type, const pident_t *pident, array_dims_ else if (atype->type == RPC_FC_CARRAY) atype->type = RPC_FC_CVARRAY; else - error("%s: length_is attribute applied to illegal type\n", v->name); + error_loc("%s: length_is attribute applied to illegal type\n", v->name); atype->length_is = dim; } ptype = &(*ptype)->ref; if (*ptype == NULL) - error("%s: too many expressions in length_is attribute\n", v->name); + error_loc("%s: too many expressions in length_is attribute\n", v->name); } if (has_varconf && !last_array(v->type))

1 0

Rob Shearman : rpcrt4: Test and fix I_RpcExceptionFilter.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 4efff322911b224e727dc6df932c357d0e6a279a URL: http://source.winehq.org/git/wine.git/?a=commit;h=4efff322911b224e727dc6df9… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 15 21:50:14 2008 +0100 rpcrt4: Test and fix I_RpcExceptionFilter. --- dlls/rpcrt4/rpcrt4_main.c | 10 ++++++++-- dlls/rpcrt4/tests/rpc.c | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 49 insertions(+), 2 deletions(-) diff --git a/dlls/rpcrt4/rpcrt4_main.c b/dlls/rpcrt4/rpcrt4_main.c index 02e1dcd..0fddc28 100644 --- a/dlls/rpcrt4/rpcrt4_main.c +++ b/dlls/rpcrt4/rpcrt4_main.c @@ -944,8 +944,14 @@ int WINAPI I_RpcExceptionFilter(ULONG ExceptionCode) TRACE("0x%x\n", ExceptionCode); switch (ExceptionCode) { - case EXCEPTION_ACCESS_VIOLATION: - case EXCEPTION_ILLEGAL_INSTRUCTION: + case STATUS_DATATYPE_MISALIGNMENT: + case STATUS_BREAKPOINT: + case STATUS_ACCESS_VIOLATION: + case STATUS_ILLEGAL_INSTRUCTION: + case STATUS_PRIVILEGED_INSTRUCTION: + case STATUS_INSTRUCTION_MISALIGNMENT: + case STATUS_STACK_OVERFLOW: + case STATUS_POSSIBLE_DEADLOCK: return EXCEPTION_CONTINUE_SEARCH; default: return EXCEPTION_EXECUTE_HANDLER; diff --git a/dlls/rpcrt4/tests/rpc.c b/dlls/rpcrt4/tests/rpc.c index ace8aff..9fd9993 100644 --- a/dlls/rpcrt4/tests/rpc.c +++ b/dlls/rpcrt4/tests/rpc.c @@ -610,6 +610,46 @@ static void test_RpcStringBindingParseA(void) ok(options == NULL, "options was %p instead of NULL\n", options); } +static void test_I_RpcExceptionFilter(void) +{ + ULONG exception; + int retval; + int (WINAPI *pI_RpcExceptionFilter)(ULONG) = (void *)GetProcAddress(GetModuleHandle("rpcrt4.dll"), "I_RpcExceptionFilter"); + + if (!pI_RpcExceptionFilter) + { + skip("I_RpcExceptionFilter not exported\n"); + return; + } + + for (exception = 0; exception < STATUS_REG_NAT_CONSUMPTION; exception++) + { + /* skip over uninteresting bits of the number space */ + if (exception == 2000) exception = 0x40000000; + if (exception == 0x40000005) exception = 0x80000000; + if (exception == 0x80000005) exception = 0xc0000000; + + retval = pI_RpcExceptionFilter(exception); + switch (exception) + { + case STATUS_DATATYPE_MISALIGNMENT: + case STATUS_BREAKPOINT: + case STATUS_ACCESS_VIOLATION: + case STATUS_ILLEGAL_INSTRUCTION: + case STATUS_PRIVILEGED_INSTRUCTION: + case 0xc00000aa /* STATUS_INSTRUCTION_MISALIGNMENT */: + case STATUS_STACK_OVERFLOW: + case 0xc0000194 /* STATUS_POSSIBLE_DEADLOCK */: + ok(retval == EXCEPTION_CONTINUE_SEARCH, "I_RpcExceptionFilter(0x%x) should have returned %d instead of %d\n", + exception, EXCEPTION_CONTINUE_SEARCH, retval); + break; + default: + ok(retval == EXCEPTION_EXECUTE_HANDLER, "I_RpcExceptionFilter(0x%x) should have returned %d instead of %d\n", + exception, EXCEPTION_EXECUTE_HANDLER, retval); + } + } +} + START_TEST( rpc ) { trace ( " ** Uuid Conversion and Comparison Tests **\n" ); @@ -620,4 +660,5 @@ START_TEST( rpc ) test_towers(); test_I_RpcMapWin32Status(); test_RpcStringBindingParseA(); + test_I_RpcExceptionFilter(); }

1 0

Rob Shearman : ole32: Add tests for clipboard' s implementation of IDataObject_GetData.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 51158a2f9c54b4680e0b3d453e2b3ec90dd741c3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=51158a2f9c54b4680e0b3d453… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 15 18:29:41 2008 +0100 ole32: Add tests for clipboard's implementation of IDataObject_GetData. --- dlls/ole32/tests/clipboard.c | 45 ++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 45 insertions(+), 0 deletions(-) diff --git a/dlls/ole32/tests/clipboard.c b/dlls/ole32/tests/clipboard.c index c6371cd..e365ee8 100644 --- a/dlls/ole32/tests/clipboard.c +++ b/dlls/ole32/tests/clipboard.c @@ -58,6 +58,7 @@ typedef struct EnumFormatImpl { } EnumFormatImpl; static BOOL expect_DataObjectImpl_QueryGetData = TRUE; +static ULONG DataObjectImpl_GetData_calls = 0; static HRESULT EnumFormatImpl_Create(FORMATETC *fmtetc, UINT size, LPENUMFORMATETC *lplpformatetc); @@ -196,6 +197,8 @@ static HRESULT WINAPI DataObjectImpl_GetData(IDataObject* iface, FORMATETC *pfor { DataObjectImpl *This = (DataObjectImpl*)iface; + DataObjectImpl_GetData_calls++; + if(pformatetc->lindex != -1) return DV_E_LINDEX; @@ -326,6 +329,7 @@ static void test_get_clipboard(void) HRESULT hr; IDataObject *data_obj; FORMATETC fmtetc; + STGMEDIUM stgmedium; hr = OleGetClipboard(NULL); ok(hr == E_INVALIDARG, "OleGetClipboard(NULL) should return E_INVALIDARG instead of 0x%08x\n", hr); @@ -333,6 +337,8 @@ static void test_get_clipboard(void) hr = OleGetClipboard(&data_obj); ok(hr == S_OK, "OleGetClipboard failed with error 0x%08x\n", hr); + /* test IDataObject_QueryGetData */ + /* clipboard's IDataObject_QueryGetData shouldn't defer to our IDataObject_QueryGetData */ expect_DataObjectImpl_QueryGetData = FALSE; @@ -367,6 +373,45 @@ static void test_get_clipboard(void) expect_DataObjectImpl_QueryGetData = TRUE; + /* test IDataObject_GetData */ + + DataObjectImpl_GetData_calls = 0; + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + hr = IDataObject_GetData(data_obj, &fmtetc, &stgmedium); + ok(hr == S_OK, "IDataObject_GetData failed with error 0x%08x\n", hr); + ReleaseStgMedium(&stgmedium); + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + fmtetc.dwAspect = 0xdeadbeef; + hr = IDataObject_GetData(data_obj, &fmtetc, &stgmedium); + ok(hr == S_OK, "IDataObject_GetData failed with error 0x%08x\n", hr); + ReleaseStgMedium(&stgmedium); + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + fmtetc.dwAspect = DVASPECT_THUMBNAIL; + hr = IDataObject_GetData(data_obj, &fmtetc, &stgmedium); + ok(hr == S_OK, "IDataObject_GetData failed with error 0x%08x\n", hr); + ReleaseStgMedium(&stgmedium); + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + fmtetc.lindex = 256; + hr = IDataObject_GetData(data_obj, &fmtetc, &stgmedium); + todo_wine + ok(hr == DV_E_FORMATETC, "IDataObject_GetData should have failed with DV_E_FORMATETC instead of 0x%08x\n", hr); + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + fmtetc.cfFormat = CF_RIFF; + hr = IDataObject_GetData(data_obj, &fmtetc, &stgmedium); + ok(hr == DV_E_FORMATETC, "IDataObject_GetData should have failed with DV_E_FORMATETC instead of 0x%08x\n", hr); + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + fmtetc.tymed = TYMED_FILE; + hr = IDataObject_GetData(data_obj, &fmtetc, &stgmedium); + ok(hr == DV_E_TYMED, "IDataObject_GetData should have failed with DV_E_TYMED instead of 0x%08x\n", hr); + + ok(DataObjectImpl_GetData_calls == 6, "DataObjectImpl_GetData should have been called 6 times instead of %d times\n", DataObjectImpl_GetData_calls); + IDataObject_Release(data_obj); }

1 0

Rob Shearman : ole32: Fix OLEClipbrd_IDataObject_QueryGetData to conform to the tests.
by Alexandre Julliard 16 Apr '08

16 Apr '08

Module: wine Branch: master Commit: 2cd4117f81cb93da8a5b4ad13573d132e8118fc2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2cd4117f81cb93da8a5b4ad13… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 15 18:29:32 2008 +0100 ole32: Fix OLEClipbrd_IDataObject_QueryGetData to conform to the tests. --- dlls/ole32/clipboard.c | 33 ++++++--------------------------- dlls/ole32/tests/clipboard.c | 11 ++++++----- 2 files changed, 12 insertions(+), 32 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index ad15219..a74e71a 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -1312,42 +1312,21 @@ static HRESULT WINAPI OLEClipbrd_IDataObject_QueryGetData( IDataObject* iface, LPFORMATETC pformatetc) { - /* - * Declare "This" pointer - */ - OLEClipbrd *This = (OLEClipbrd *)iface; - TRACE("(%p, %p)\n", iface, pformatetc); - /* - * If we have a data source placed on the clipboard (via OleSetClipboard) - * simply delegate to the source object's QueryGetData - */ - if ( This->pIDataObjectSrc ) - { - return IDataObject_QueryGetData(This->pIDataObjectSrc, pformatetc); - } - if (!pformatetc) return E_INVALIDARG; -/* - if ( pformatetc->dwAspect != DVASPECT_CONTENT ) - return DV_E_DVASPECT; -*/ - if ( pformatetc->lindex != -1 ) - return DV_E_LINDEX; - /* TODO: Handle TYMED_IStorage media which were put on the clipboard - * by copying the storage into global memory. We must convert this - * TYMED_HGLOBAL back to TYMED_IStorage. - */ - if ( pformatetc->tymed != TYMED_HGLOBAL ) - return DV_E_TYMED; + if ( pformatetc->dwAspect != DVASPECT_CONTENT ) + return DV_E_FORMATETC; + + if ( pformatetc->lindex != -1 ) + return DV_E_FORMATETC; /* * Delegate to the Windows clipboard function IsClipboardFormatAvailable */ - return (IsClipboardFormatAvailable(pformatetc->cfFormat)) ? S_OK : DV_E_FORMATETC; + return (IsClipboardFormatAvailable(pformatetc->cfFormat)) ? S_OK : DV_E_CLIPFORMAT; } /************************************************************************ diff --git a/dlls/ole32/tests/clipboard.c b/dlls/ole32/tests/clipboard.c index aa2f622..c6371cd 100644 --- a/dlls/ole32/tests/clipboard.c +++ b/dlls/ole32/tests/clipboard.c @@ -226,7 +226,7 @@ static HRESULT WINAPI DataObjectImpl_QueryGetData(IDataObject* iface, FORMATETC BOOL foundFormat = FALSE; if (!expect_DataObjectImpl_QueryGetData) - todo_wine ok(0, "unexpected call to DataObjectImpl_QueryGetData\n"); + ok(0, "unexpected call to DataObjectImpl_QueryGetData\n"); if(pformatetc->lindex != -1) return DV_E_LINDEX; @@ -343,25 +343,26 @@ static void test_get_clipboard(void) InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); fmtetc.dwAspect = 0xdeadbeef; hr = IDataObject_QueryGetData(data_obj, &fmtetc); - todo_wine + ok(hr == DV_E_FORMATETC, "IDataObject_QueryGetData should have failed with DV_E_FORMATETC instead of 0x%08x\n", hr); + + InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); + fmtetc.dwAspect = DVASPECT_THUMBNAIL; + hr = IDataObject_QueryGetData(data_obj, &fmtetc); ok(hr == DV_E_FORMATETC, "IDataObject_QueryGetData should have failed with DV_E_FORMATETC instead of 0x%08x\n", hr); InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); fmtetc.lindex = 256; hr = IDataObject_QueryGetData(data_obj, &fmtetc); - todo_wine ok(hr == DV_E_FORMATETC, "IDataObject_QueryGetData should have failed with DV_E_FORMATETC instead of 0x%08x\n", hr); InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); fmtetc.cfFormat = CF_RIFF; hr = IDataObject_QueryGetData(data_obj, &fmtetc); - todo_wine ok(hr == DV_E_CLIPFORMAT, "IDataObject_QueryGetData should have failed with DV_E_CLIPFORMAT instead of 0x%08x\n", hr); InitFormatEtc(fmtetc, CF_TEXT, TYMED_HGLOBAL); fmtetc.tymed = TYMED_FILE; hr = IDataObject_QueryGetData(data_obj, &fmtetc); - todo_wine ok(hr == S_OK, "IDataObject_QueryGetData failed with error 0x%08x\n", hr); expect_DataObjectImpl_QueryGetData = TRUE;

1 0

Jump to page:

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-commits April 2008