wine-commits May 2011

wine-commits@winehq.org

2 participants
945 discussions

Nikolay Sivov : explorerframe: Get rid of interface pointer casts.
by Alexandre Julliard 20 May '11

20 May '11

Module: wine Branch: master Commit: 478316ceec8e5c2ecb177972a8f9184169fc531b URL: http://source.winehq.org/git/wine.git/?a=commit;h=478316ceec8e5c2ecb177972a… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed May 18 15:21:37 2011 +0400 explorerframe: Get rid of interface pointer casts. --- dlls/explorerframe/nstc.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/explorerframe/nstc.c b/dlls/explorerframe/nstc.c index 9628543..9a8c75b 100644 --- a/dlls/explorerframe/nstc.c +++ b/dlls/explorerframe/nstc.c @@ -495,7 +495,7 @@ static LRESULT create_namespacetree(HWND hWnd, CREATESTRUCTW *crs) ERR("Failed to get the System Image List.\n"); } - INameSpaceTreeControl_AddRef((INameSpaceTreeControl*)This); + INameSpaceTreeControl2_AddRef(&This->INameSpaceTreeControl2_iface); /* Subclass the treeview to get the keybord events. */ This->tv_oldwndproc = (WNDPROC)SetWindowLongPtrW(This->hwnd_tv, GWLP_WNDPROC, @@ -528,10 +528,10 @@ static LRESULT destroy_namespacetree(NSTC2Impl *This) RemovePropA(This->hwnd_tv, "PROP_THIS"); } - INameSpaceTreeControl_RemoveAllRoots((INameSpaceTreeControl*)This); + INameSpaceTreeControl2_RemoveAllRoots(&This->INameSpaceTreeControl2_iface); /* This reference was added in create_namespacetree */ - INameSpaceTreeControl_Release((INameSpaceTreeControl*)This); + INameSpaceTreeControl2_Release(&This->INameSpaceTreeControl2_iface); return TRUE; } @@ -1597,8 +1597,8 @@ HRESULT NamespaceTreeControl_Constructor(IUnknown *pUnkOuter, REFIID riid, void list_init(&nstc->roots); - ret = INameSpaceTreeControl_QueryInterface((INameSpaceTreeControl*)nstc, riid, ppv); - INameSpaceTreeControl_Release((INameSpaceTreeControl*)nstc); + ret = INameSpaceTreeControl2_QueryInterface(&nstc->INameSpaceTreeControl2_iface, riid, ppv); + INameSpaceTreeControl2_Release(&nstc->INameSpaceTreeControl2_iface); TRACE("--(%p)\n", ppv); return ret;

1 0

Thomas Mullaly : mshtml/tests: Added IDispatch event tests.
by Alexandre Julliard 20 May '11

20 May '11

Module: wine Branch: master Commit: ea2ab11a74df1e29707d7f888bd6b6ee4089f362 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea2ab11a74df1e29707d7f888… Author: Thomas Mullaly <tmullaly(a)codeweavers.com> Date: Sat May 7 22:56:52 2011 -0400 mshtml/tests: Added IDispatch event tests. --- dlls/mshtml/tests/htmldoc.c | 109 ++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 107 insertions(+), 2 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ea2ab11a74df1e29707d7…

1 0

Nikolay Sivov : comctl32/tests: Some tests for TVM_GETNEXTITEM and item toggle.
by Alexandre Julliard 20 May '11

20 May '11

Module: wine Branch: master Commit: f9ca298b21a79af22a26a150671526d6b364b673 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f9ca298b21a79af22a26a1506… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed May 18 15:14:33 2011 +0400 comctl32/tests: Some tests for TVM_GETNEXTITEM and item toggle. --- dlls/comctl32/tests/treeview.c | 46 ++++++++++++++++++++++++++++++++++++++++ 1 files changed, 46 insertions(+), 0 deletions(-) diff --git a/dlls/comctl32/tests/treeview.c b/dlls/comctl32/tests/treeview.c index 0b43e87..0437d65 100644 --- a/dlls/comctl32/tests/treeview.c +++ b/dlls/comctl32/tests/treeview.c @@ -1341,6 +1341,12 @@ static void test_expandnotify(void) the only way is to reset with all children removed */ ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "collapse after expand notifications", FALSE); + /* try to toggle child that doesn't have children itself */ + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, TVM_EXPAND, TVE_TOGGLE, (LPARAM)hChild); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "toggle node without children", TRUE); + DestroyWindow(hTree); /* test TVM_GETITEMRECT inside TVN_ITEMEXPANDED notification */ @@ -1350,6 +1356,7 @@ static void test_expandnotify(void) ret = TreeView_Select(hTree, hChild, TVGN_CARET); g_get_rect_in_expand = FALSE; ok(ret, "got %d\n", ret); + DestroyWindow(hTree); } @@ -1602,6 +1609,44 @@ static void test_TVS_CHECKBOXES(void) DestroyWindow(hTree); } +static void test_TVM_GETNEXTITEM(void) +{ + HTREEITEM item; + HWND hTree; + + hTree = create_treeview_control(0); + fill_tree(hTree); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_ROOT, 0); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_ROOT, (LPARAM)TVI_ROOT); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_ROOT, (LPARAM)hRoot); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_ROOT, (LPARAM)hChild); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_CHILD, 0); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_CHILD, (LPARAM)hRoot); + ok(item == hChild, "got %p, expected %p\n", item, hChild); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_CHILD, (LPARAM)TVI_ROOT); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_PARENT, 0); + ok(item == NULL, "got %p\n", item); + + item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_PARENT, (LPARAM)hChild); + ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + DestroyWindow(hTree); +} + START_TEST(treeview) { HMODULE hComctl32; @@ -1672,6 +1717,7 @@ START_TEST(treeview) test_delete_items(); test_htreeitem_layout(); test_TVS_CHECKBOXES(); + test_TVM_GETNEXTITEM(); if (!load_v6_module(&ctx_cookie, &hCtx)) {

1 0

Aric Stewart : usp10: If a feature changes a glyph reapply the feature to the new glyph.
by Alexandre Julliard 20 May '11

20 May '11

Module: wine Branch: master Commit: 9a6cf4a39135a9e3dc611bbbab9167f6e5a79b18 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9a6cf4a39135a9e3dc611bbba… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu May 19 11:15:43 2011 -0500 usp10: If a feature changes a glyph reapply the feature to the new glyph. --- dlls/usp10/shape.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/dlls/usp10/shape.c b/dlls/usp10/shape.c index 9263e19..0afb7aa 100644 --- a/dlls/usp10/shape.c +++ b/dlls/usp10/shape.c @@ -516,6 +516,9 @@ static INT GSUB_apply_SingleSubst(const GSUB_LookupTable *look, WORD *glyphs, IN TRACE(" Coverage index %i\n",index); if (index != -1) { + if (glyphs[glyph_index] == GET_BE_WORD(ssf2->Substitute[index])) + return GSUB_E_NOGLYPH; + TRACE(" Glyph is 0x%x ->",glyphs[glyph_index]); glyphs[glyph_index] = GET_BE_WORD(ssf2->Substitute[index]); TRACE("0x%x\n",glyphs[glyph_index]); @@ -548,6 +551,8 @@ static INT GSUB_apply_AlternateSubst(const GSUB_LookupTable *look, WORD *glyphs, offset = GET_BE_WORD(asf1->AlternateSet[index]); as = (const GSUB_AlternateSet*)((const BYTE*)asf1+offset); FIXME("%i alternates, picking index 0\n",GET_BE_WORD(as->GlyphCount)); + if (glyphs[glyph_index] == GET_BE_WORD(as->Alternate[0])) + return GSUB_E_NOGLYPH; TRACE(" Glyph 0x%x ->",glyphs[glyph_index]); glyphs[glyph_index] = GET_BE_WORD(as->Alternate[0]); @@ -759,6 +764,13 @@ static INT GSUB_apply_feature(const GSUB_Header * header, const GSUB_Feature* fe } if (out_index == GSUB_E_NOGLYPH) TRACE("lookups found no glyphs\n"); + else + { + int out2; + out2 = GSUB_apply_feature(header, feature, glyphs, glyph_index, write_dir, glyph_count); + if (out2!=GSUB_E_NOGLYPH) + out_index = out2; + } return out_index; }

1 0

Aric Stewart : usp10: Correct lookahead offset in Coverage-based Chaining Context Glyph Substitution .
by Alexandre Julliard 20 May '11

20 May '11

Module: wine Branch: master Commit: ecd43b4bd84de84d79b34bb44fabebbd4fa70731 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ecd43b4bd84de84d79b34bb44… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu May 19 10:47:36 2011 -0500 usp10: Correct lookahead offset in Coverage-based Chaining Context Glyph Substitution. --- dlls/usp10/shape.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/usp10/shape.c b/dlls/usp10/shape.c index fb6084e..9263e19 100644 --- a/dlls/usp10/shape.c +++ b/dlls/usp10/shape.c @@ -690,7 +690,7 @@ static INT GSUB_apply_ChainContextSubst(const GSUB_LookupList* lookup, const GSU for (k = 0; k < GET_BE_WORD(ccsf3_3->LookaheadGlyphCount); k++) { offset = GET_BE_WORD(ccsf3_3->Coverage[k]); - if (GSUB_is_glyph_covered((const BYTE*)ccsf3_1+offset, glyphs[glyph_index + (dirLookahead * (indexGlyphs + k+1))]) == -1) + if (GSUB_is_glyph_covered((const BYTE*)ccsf3_1+offset, glyphs[glyph_index + (dirLookahead * (indexGlyphs + k))]) == -1) break; } if (k != GET_BE_WORD(ccsf3_3->LookaheadGlyphCount))

1 0

Alexandre Julliard : quartz: Make sure the preferred allocator pointer is initialized.
by Alexandre Julliard 20 May '11

20 May '11

Module: wine Branch: master Commit: 7f6e677d5ad5048dd2f1defb50f64928c380606e URL: http://source.winehq.org/git/wine.git/?a=commit;h=7f6e677d5ad5048dd2f1defb5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 20 10:28:21 2011 +0200 quartz: Make sure the preferred allocator pointer is initialized. --- dlls/quartz/pin.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/quartz/pin.c b/dlls/quartz/pin.c index 2a3b2d1..af26a6b 100644 --- a/dlls/quartz/pin.c +++ b/dlls/quartz/pin.c @@ -199,6 +199,7 @@ static HRESULT PullPin_Init(const IPinVtbl *PullPin_Vtbl, const PIN_INFO * pPinI pPinImpl->fnDone = pDone; pPinImpl->fnPreConnect = NULL; pPinImpl->pAlloc = NULL; + pPinImpl->prefAlloc = NULL; pPinImpl->pReader = NULL; pPinImpl->hThread = NULL; pPinImpl->hEventStateChanged = CreateEventW(NULL, TRUE, TRUE, NULL);

1 0

Alistair Leslie-Hughes : gdi32: Add support in DeviceCaps CURVECAPS, LINECAPS, POLYGONALCAPS for EnhMetafiles.
by Alexandre Julliard 19 May '11

19 May '11

Module: wine Branch: master Commit: 5a7dba5a70c4c5cc29669358c0faddeb85af5b87 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5a7dba5a70c4c5cc29669358c… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed May 18 14:24:01 2011 +1000 gdi32: Add support in DeviceCaps CURVECAPS, LINECAPS, POLYGONALCAPS for EnhMetafiles. --- dlls/gdi32/enhmfdrv/dc.c | 9 +++++++++ dlls/gdi32/tests/metafile.c | 11 +++++++++++ 2 files changed, 20 insertions(+), 0 deletions(-) diff --git a/dlls/gdi32/enhmfdrv/dc.c b/dlls/gdi32/enhmfdrv/dc.c index 4b1e311..7a84265 100644 --- a/dlls/gdi32/enhmfdrv/dc.c +++ b/dlls/gdi32/enhmfdrv/dc.c @@ -541,6 +541,15 @@ INT CDECL EMFDRV_GetDeviceCaps(PHYSDEV dev, INT cap) return physDev->planes; case NUMCOLORS: return physDev->numcolors; + case CURVECAPS: + return (CC_CIRCLES | CC_PIE | CC_CHORD | CC_ELLIPSES | CC_WIDE | + CC_STYLED | CC_WIDESTYLED | CC_INTERIORS | CC_ROUNDRECT); + case LINECAPS: + return (LC_POLYLINE | LC_MARKER | LC_POLYMARKER | LC_WIDE | + LC_STYLED | LC_WIDESTYLED | LC_INTERIORS); + case POLYGONALCAPS: + return (PC_POLYGON | PC_RECTANGLE | PC_WINDPOLYGON | PC_SCANLINE | + PC_WIDE | PC_STYLED | PC_WIDESTYLED | PC_INTERIORS); default: FIXME("Unimplemented cap %d\n", cap); return 0; diff --git a/dlls/gdi32/tests/metafile.c b/dlls/gdi32/tests/metafile.c index 002e66a..206e15a 100644 --- a/dlls/gdi32/tests/metafile.c +++ b/dlls/gdi32/tests/metafile.c @@ -2806,6 +2806,7 @@ static void getwinmetafilebits(UINT mode, int scale, RECT *rc) METAHEADER *mh = NULL; METARECORD *rec; INT horz_res, vert_res, horz_size, vert_size; + INT curve_caps, line_caps, poly_caps; display_dc = GetDC(NULL); ok(display_dc != NULL, "display_dc is NULL\n"); @@ -2817,6 +2818,16 @@ static void getwinmetafilebits(UINT mode, int scale, RECT *rc) emf_dc = CreateEnhMetaFileA(display_dc, NULL, rc, NULL); ok(emf_dc != NULL, "emf_dc is NULL\n"); + + curve_caps = GetDeviceCaps(emf_dc, CURVECAPS); + ok(curve_caps == 511, "expect 511 got %d\n", curve_caps); + + line_caps = GetDeviceCaps(emf_dc, LINECAPS); + ok(line_caps == 254, "expect 254 got %d\n", line_caps); + + poly_caps = GetDeviceCaps(emf_dc, POLYGONALCAPS); + ok(poly_caps == 255, "expect 511 got %d\n", poly_caps); + for(i = 0; i < 3000; i++) /* This is enough to take emf_size > 0xffff */ Rectangle(emf_dc, 0, 0, 1000, 20); emf = CloseEnhMetaFile(emf_dc);

1 0

Piotr Caban : msvcrt: Make scanf functions thread safe.
by Alexandre Julliard 19 May '11

19 May '11

Module: wine Branch: master Commit: 73b9acf2735055df6ae285137f047b36da4c1b86 URL: http://source.winehq.org/git/wine.git/?a=commit;h=73b9acf2735055df6ae285137… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu May 19 15:32:00 2011 +0200 msvcrt: Make scanf functions thread safe. --- dlls/msvcrt/msvcrt.h | 2 ++ dlls/msvcrt/scanf.h | 26 +++++++++++++++++++++++++- 2 files changed, 27 insertions(+), 1 deletions(-) diff --git a/dlls/msvcrt/msvcrt.h b/dlls/msvcrt/msvcrt.h index bd41bac..e677022 100644 --- a/dlls/msvcrt/msvcrt.h +++ b/dlls/msvcrt/msvcrt.h @@ -784,6 +784,8 @@ int __cdecl MSVCRT_iswspace(MSVCRT_wint_t); int __cdecl MSVCRT_iswdigit(MSVCRT_wint_t); int __cdecl MSVCRT_isleadbyte(int); +void __cdecl MSVCRT__lock_file(MSVCRT_FILE*); +void __cdecl MSVCRT__unlock_file(MSVCRT_FILE*); int __cdecl MSVCRT_fgetc(MSVCRT_FILE*); int __cdecl MSVCRT_ungetc(int,MSVCRT_FILE*); MSVCRT_wint_t __cdecl MSVCRT_fgetwc(MSVCRT_FILE*); diff --git a/dlls/msvcrt/scanf.h b/dlls/msvcrt/scanf.h index 8b796b1..8324349 100644 --- a/dlls/msvcrt/scanf.h +++ b/dlls/msvcrt/scanf.h @@ -48,6 +48,8 @@ #ifdef CONSOLE #define _GETC_(file) (consumed++, _getch()) #define _UNGETC_(nch, file) do { _ungetch(nch); consumed--; } while(0) +#define _LOCK_FILE_(file) MSVCRT__lock_file(MSVCRT_stdin) +#define _UNLOCK_FILE_(file) MSVCRT__unlock_file(MSVCRT_stdin) #ifdef WIDE_SCANF #ifdef SECURE #define _FUNCTION_ static int MSVCRT_vcwscanf_s_l(const char *format, MSVCRT__locale_t locale, __ms_va_list ap) @@ -68,6 +70,8 @@ #ifdef STRING_LEN #define _GETC_(file) (consumed==length ? '\0' : (consumed++, *file++)) #define _UNGETC_(nch, file) do { file--; consumed--; } while(0) +#define _LOCK_FILE_(file) do {} while(0) +#define _UNLOCK_FILE_(file) do {} while(0) #ifdef WIDE_SCANF #ifdef SECURE #define _FUNCTION_ static int MSVCRT_vsnwscanf_s_l(const MSVCRT_wchar_t *file, MSVCRT_size_t length, const MSVCRT_wchar_t *format, MSVCRT__locale_t locale, __ms_va_list ap) @@ -84,6 +88,8 @@ #else /* STRING_LEN */ #define _GETC_(file) (consumed++, *file++) #define _UNGETC_(nch, file) do { file--; consumed--; } while(0) +#define _LOCK_FILE_(file) do {} while(0) +#define _UNLOCK_FILE_(file) do {} while(0) #ifdef WIDE_SCANF #ifdef SECURE #define _FUNCTION_ static int MSVCRT_vswscanf_s_l(const MSVCRT_wchar_t *file, const MSVCRT_wchar_t *format, MSVCRT__locale_t locale, __ms_va_list ap) @@ -102,6 +108,8 @@ #ifdef WIDE_SCANF #define _GETC_(file) (consumed++, MSVCRT_fgetwc(file)) #define _UNGETC_(nch, file) do { MSVCRT_ungetwc(nch, file); consumed--; } while(0) +#define _LOCK_FILE_(file) MSVCRT__lock_file(file) +#define _UNLOCK_FILE_(file) MSVCRT__unlock_file(file) #ifdef SECURE #define _FUNCTION_ static int MSVCRT_vfwscanf_s_l(MSVCRT_FILE* file, const MSVCRT_wchar_t *format, MSVCRT__locale_t locale, __ms_va_list ap) #else /* SECURE */ @@ -110,6 +118,8 @@ #else /* WIDE_SCANF */ #define _GETC_(file) (consumed++, MSVCRT_fgetc(file)) #define _UNGETC_(nch, file) do { MSVCRT_ungetc(nch, file); consumed--; } while(0) +#define _LOCK_FILE_(file) MSVCRT__lock_file(file) +#define _UNLOCK_FILE_(file) MSVCRT__unlock_file(file) #ifdef SECURE #define _FUNCTION_ static int MSVCRT_vfscanf_s_l(MSVCRT_FILE* file, const char *format, MSVCRT__locale_t locale, __ms_va_list ap) #else /* SECURE */ @@ -134,8 +144,13 @@ _FUNCTION_ { #endif /* STRING */ #endif /* CONSOLE */ #endif /* WIDE_SCANF */ + _LOCK_FILE_(file); + nch = _GETC_(file); - if (nch == _EOF_) return _EOF_RET; + if (nch == _EOF_) { + _UNLOCK_FILE_(file); + return _EOF_RET; + } if(!locale) locale = get_locale(); @@ -400,6 +415,7 @@ _FUNCTION_ { *sptr++ = _CHAR2SUPPORTED_(nch); if(size>1) size--; else { + _UNLOCK_FILE_(file); *sptr_beg = 0; return rd; } @@ -429,6 +445,7 @@ _FUNCTION_ { *sptr++ = _WIDE2SUPPORTED_(nch); if(size>1) size--; else { + _UNLOCK_FILE_(file); *sptr_beg = 0; return rd; } @@ -474,6 +491,7 @@ _FUNCTION_ { *str++ = _CHAR2SUPPORTED_(nch); if(size) size--; else { + _UNLOCK_FILE_(file); *pstr = 0; return rd; } @@ -499,6 +517,7 @@ _FUNCTION_ { *str++ = _WIDE2SUPPORTED_(nch); if(size) size--; else { + _UNLOCK_FILE_(file); *pstr = 0; return rd; } @@ -586,6 +605,7 @@ _FUNCTION_ { if (width>0) width--; if(size>1) size--; else { + _UNLOCK_FILE_(file); *str = 0; return rd; } @@ -628,7 +648,9 @@ _FUNCTION_ { if (nch!=_EOF_) { _UNGETC_(nch, file); } + TRACE("returning %d\n", rd); + _UNLOCK_FILE_(file); return rd; } @@ -642,5 +664,7 @@ _FUNCTION_ { #undef _CHAR2DIGIT_ #undef _GETC_ #undef _UNGETC_ +#undef _LOCK_FILE_ +#undef _UNLOCK_FILE_ #undef _FUNCTION_ #undef _BITMAPSIZE_

1 0

Piotr Caban : msvcrt: Make printf functions thread safe.
by Alexandre Julliard 19 May '11

19 May '11

Module: wine Branch: master Commit: c5894bacf5cc55c1e25bb9905535d5ef0dc2ab14 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c5894bacf5cc55c1e25bb9905… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu May 19 15:31:51 2011 +0200 msvcrt: Make printf functions thread safe. --- dlls/msvcrt/file.c | 32 ++++++++++++++++++++++++++++---- 1 files changed, 28 insertions(+), 4 deletions(-) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index ee95c28..b9c7e90 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -3563,7 +3563,13 @@ static int puts_clbk_file_w(void *file, int len, const MSVCRT_wchar_t *str) */ int CDECL MSVCRT_vfprintf(MSVCRT_FILE* file, const char *format, __ms_va_list valist) { - return pf_printf_a(puts_clbk_file_a, file, format, NULL, FALSE, FALSE, arg_clbk_valist, NULL, &valist); + int ret; + + MSVCRT__lock_file(file); + ret = pf_printf_a(puts_clbk_file_a, file, format, NULL, FALSE, FALSE, arg_clbk_valist, NULL, &valist); + MSVCRT__unlock_file(file); + + return ret; } /********************************************************************* @@ -3571,12 +3577,18 @@ int CDECL MSVCRT_vfprintf(MSVCRT_FILE* file, const char *format, __ms_va_list va */ int CDECL MSVCRT_vfprintf_s(MSVCRT_FILE* file, const char *format, __ms_va_list valist) { + int ret; + if(!MSVCRT_CHECK_PMT(file != NULL)) { *MSVCRT__errno() = MSVCRT_EINVAL; return -1; } - return pf_printf_a(puts_clbk_file_a, file, format, NULL, FALSE, TRUE, arg_clbk_valist, NULL, &valist); + MSVCRT__lock_file(file); + ret = pf_printf_a(puts_clbk_file_a, file, format, NULL, FALSE, TRUE, arg_clbk_valist, NULL, &valist); + MSVCRT__unlock_file(file); + + return ret; } /********************************************************************* @@ -3584,7 +3596,13 @@ int CDECL MSVCRT_vfprintf_s(MSVCRT_FILE* file, const char *format, __ms_va_list */ int CDECL MSVCRT_vfwprintf(MSVCRT_FILE* file, const MSVCRT_wchar_t *format, __ms_va_list valist) { - return pf_printf_w(puts_clbk_file_w, file, format, NULL, FALSE, FALSE, arg_clbk_valist, NULL, &valist); + int ret; + + MSVCRT__lock_file(file); + ret = pf_printf_w(puts_clbk_file_w, file, format, NULL, FALSE, FALSE, arg_clbk_valist, NULL, &valist); + MSVCRT__unlock_file(file); + + return ret; } /********************************************************************* @@ -3592,12 +3610,18 @@ int CDECL MSVCRT_vfwprintf(MSVCRT_FILE* file, const MSVCRT_wchar_t *format, __ms */ int CDECL MSVCRT_vfwprintf_s(MSVCRT_FILE* file, const MSVCRT_wchar_t *format, __ms_va_list valist) { + int ret; + if(!MSVCRT_CHECK_PMT( file != NULL)) { *MSVCRT__errno() = MSVCRT_EINVAL; return -1; } - return pf_printf_w(puts_clbk_file_w, file, format, NULL, FALSE, TRUE, arg_clbk_valist, NULL, &valist); + MSVCRT__lock_file(file); + ret = pf_printf_w(puts_clbk_file_w, file, format, NULL, FALSE, TRUE, arg_clbk_valist, NULL, &valist); + MSVCRT__unlock_file(file); + + return ret; } /*********************************************************************

1 0

Piotr Caban : msvcrt: Added _lock_file/_unlock_file implementation.
by Alexandre Julliard 19 May '11

19 May '11

Module: wine Branch: master Commit: 249917beac273a2959202722b89f34d53b694c91 URL: http://source.winehq.org/git/wine.git/?a=commit;h=249917beac273a2959202722b… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu May 19 15:31:42 2011 +0200 msvcrt: Added _lock_file/_unlock_file implementation. --- dlls/msvcrt/file.c | 15 +++++++++++++-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index 07c31ba..ee95c28 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -380,6 +380,9 @@ static int msvcrt_init_fp(MSVCRT_FILE* file, int fd, unsigned stream_flags) file->_file = fd; file->_flag = stream_flags; + if(file<MSVCRT__iob || file>=MSVCRT__iob+_IOB_ENTRIES) + InitializeCriticalSection(&((file_crit*)file)->crit); + TRACE(":got FILE* (%p)\n",file); return 0; } @@ -1034,7 +1037,10 @@ LONG CDECL MSVCRT__lseek(int fd, LONG offset, int whence) */ void CDECL MSVCRT__lock_file(MSVCRT_FILE *file) { - FIXME("(%p) stub\n",file); + if(file>=MSVCRT__iob && file<MSVCRT__iob+_IOB_ENTRIES) + _lock(_STREAM_LOCKS+(file-MSVCRT__iob)); + else + EnterCriticalSection(&((file_crit*)file)->crit); } /********************************************************************* @@ -1042,7 +1048,10 @@ void CDECL MSVCRT__lock_file(MSVCRT_FILE *file) */ void CDECL MSVCRT__unlock_file(MSVCRT_FILE *file) { - FIXME("(%p) stub\n",file); + if(file>=MSVCRT__iob && file<MSVCRT__iob+_IOB_ENTRIES) + _unlock(_STREAM_LOCKS+(file-MSVCRT__iob)); + else + LeaveCriticalSection(&((file_crit*)file)->crit); } /********************************************************************* @@ -2555,6 +2564,8 @@ int CDECL MSVCRT_fclose(MSVCRT_FILE* file) r=MSVCRT__close(file->_file); file->_flag = 0; + if(file<MSVCRT__iob || file>=MSVCRT__iob+_IOB_ENTRIES) + DeleteCriticalSection(&((file_crit*)file)->crit); if(file == msvcrt_get_file(MSVCRT_stream_idx-1)) { while(MSVCRT_stream_idx>3 && !file->_flag) {

1 0

← Newer
1
...
36
37
38
39
40
41
42
...
95
Older →

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 May 2011