June 2012 - Wine-Commits - list.winehq.org

Henri Verbeet : wined3d: Set undefined vertex attributes to 0.0.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 611b541f292761e459eac4eb1cf88d765b939ff8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=611b541f292761e459eac4eb1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jun 12 17:03:49 2012 +0200 wined3d: Set undefined vertex attributes to 0.0. --- dlls/wined3d/state.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c index f2da77c..35e1736 100644 --- a/dlls/wined3d/state.c +++ b/dlls/wined3d/state.c @@ -4082,7 +4082,10 @@ static void load_numbered_arrays(struct wined3d_context *context, if (!(stream_info->use_map & (1 << i))) { - if (context->numbered_array_mask & (1 << i)) unload_numbered_array(context, i); + if (context->numbered_array_mask & (1 << i)) + unload_numbered_array(context, i); + if (state->vertex_shader->reg_maps.input_registers & (1 << i)) + GL_EXTCALL(glVertexAttrib4fARB(i, 0.0f, 0.0f, 0.0f, 0.0f)); continue; }

1 0

Henri Verbeet : wined3d: Invalidate the entire buffer on maps with WINED3DLOCK_DISCARD.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 0610d1eec9a287e22831a40427e6a9a38f3f745f URL: http://source.winehq.org/git/wine.git/?a=commit;h=0610d1eec9a287e22831a4042… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jun 12 17:03:50 2012 +0200 wined3d: Invalidate the entire buffer on maps with WINED3DLOCK_DISCARD. --- dlls/wined3d/buffer.c | 15 ++++++++++++++- 1 files changed, 14 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index f11edf4..4735595 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -1011,7 +1011,20 @@ HRESULT CDECL wined3d_buffer_map(struct wined3d_buffer *buffer, UINT offset, UIN flags = buffer_sanitize_flags(buffer, flags); if (!(flags & WINED3DLOCK_READONLY)) { - if (!buffer_add_dirty_area(buffer, offset, size)) return E_OUTOFMEMORY; + if (flags & WINED3DLOCK_DISCARD) + { + /* DISCARD invalidates the entire buffer, regardless of the + * specified offset and size. Some applications also depend on the + * entire buffer being uploaded in that case. Two such + * applications are Port Royale and Darkstar One. */ + if (!buffer_add_dirty_area(buffer, 0, 0)) + return E_OUTOFMEMORY; + } + else + { + if (!buffer_add_dirty_area(buffer, offset, size)) + return E_OUTOFMEMORY; + } } count = ++buffer->resource.map_count;

1 0

Piotr Caban : msvcp90: Fixed default locale constructor implementation.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 1d0b072d0f1d5adf3229eca282ce42de11a231d9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1d0b072d0f1d5adf3229eca28… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Jun 12 18:12:12 2012 +0200 msvcp90: Fixed default locale constructor implementation. --- dlls/msvcp90/locale.c | 78 +++++++++++++++++++++++++++++++------------ dlls/msvcp90/msvcp90.h | 1 + dlls/msvcp90/msvcp90_main.c | 1 + 3 files changed, 58 insertions(+), 22 deletions(-) diff --git a/dlls/msvcp90/locale.c b/dlls/msvcp90/locale.c index 54c2eaf..49d864a 100644 --- a/dlls/msvcp90/locale.c +++ b/dlls/msvcp90/locale.c @@ -163,6 +163,9 @@ typedef struct _istreambuf_iterator_wchar /* ?_Id_cnt(a)id@locale(a)std@@0HA */ int locale_id__Id_cnt = 0; +static locale__Locimp *global_locale; +static locale classic_locale; + /* ?_Clocptr(a)_Locimp@locale(a)std@@0PAV123(a)A */ /* ?_Clocptr(a)_Locimp@locale(a)std@@0PEAV123(a)EA */ locale__Locimp *locale__Locimp__Clocptr = NULL; @@ -2606,7 +2609,6 @@ codecvt_char* __thiscall codecvt_char_ctor(codecvt_char *this) return codecvt_char_ctor_locinfo(this, NULL, 0); } -/* ?_Getcat@?$codecvt(a)DDH@std@@SAIPAPBVfacet(a)locale@2(a)PBV42@@Z */ /* ??1?$codecvt(a)DDH@std@@MAE(a)XZ */ /* ??1?$codecvt(a)DDH@std@@MEAA(a)XZ */ DEFINE_THISCALL_WRAPPER(codecvt_char_dtor, 4) @@ -2636,6 +2638,7 @@ codecvt_char* __thiscall MSVCP_codecvt_char_vector_dtor(codecvt_char *this, unsi return this; } +/* ?_Getcat@?$codecvt(a)DDH@std@@SAIPAPBVfacet(a)locale@2(a)PBV42@@Z */ /* ?_Getcat@?$codecvt(a)DDH@std@@SA_KPEAPEBVfacet(a)locale@2(a)PEBV42@@Z */ MSVCP_size_t __cdecl codecvt_char__Getcat(const locale_facet **facet, const locale *loc) { @@ -4338,6 +4341,41 @@ locale* __thiscall locale_ctor_locimp(locale *this, locale__Locimp *locimp) return this; } +/* ?_Init(a)locale@std@@CAPAV_Locimp(a)12@XZ */ +/* ?_Init(a)locale@std@@CAPEAV_Locimp(a)12@XZ */ +locale__Locimp* __cdecl locale__Init(void) +{ + _Lockit lock; + + TRACE("\n"); + + _Lockit_ctor_locktype(&lock, _LOCK_LOCALE); + if(global_locale) { + _Lockit_dtor(&lock); + return global_locale; + } + + global_locale = MSVCRT_operator_new(sizeof(locale__Locimp)); + if(!global_locale) { + _Lockit_dtor(&lock); + ERR("Out of memory\n"); + throw_exception(EXCEPTION_BAD_ALLOC, NULL); + return NULL; + } + + locale__Locimp_ctor(global_locale); + global_locale->catmask = (1<<(LC_MAX+1))-1; + MSVCP_basic_string_char_dtor(&global_locale->name); + MSVCP_basic_string_char_ctor_cstr(&global_locale->name, "C"); + + locale__Locimp__Clocptr = global_locale; + global_locale->facet.refs++; + locale_ctor_locimp(&classic_locale, locale__Locimp__Clocptr); + _Lockit_dtor(&lock); + + return global_locale; +} + /* ??0locale(a)std@@QAE(a)ABV01@0H(a)Z */ /* ??0locale(a)std@@QEAA(a)AEBV01@0H(a)Z */ DEFINE_THISCALL_WRAPPER(locale_ctor_locale_locale, 16) @@ -4392,14 +4430,8 @@ DEFINE_THISCALL_WRAPPER(locale_ctor, 4) locale* __thiscall locale_ctor(locale *this) { TRACE("(%p)\n", this); - this->ptr = MSVCRT_operator_new(sizeof(locale__Locimp)); - if(!this->ptr) { - ERR("Out of memory\n"); - throw_exception(EXCEPTION_BAD_ALLOC, NULL); - return NULL; - } - - locale__Locimp_ctor(this->ptr); + this->ptr = locale__Init(); + locale_facet__Incref(&this->ptr->facet); return this; } @@ -4497,35 +4529,29 @@ const locale_facet* __thiscall locale__Getfacet(const locale *this, MSVCP_size_t return NULL; } -/* ?_Init(a)locale@std@@CAPAV_Locimp(a)12@XZ */ -/* ?_Init(a)locale@std@@CAPEAV_Locimp(a)12@XZ */ -locale__Locimp* __cdecl locale__Init(void) -{ - FIXME("stub\n"); - return NULL; -} - /* ?_Getgloballocale(a)locale@std@@CAPAV_Locimp(a)12@XZ */ /* ?_Getgloballocale(a)locale@std@@CAPEAV_Locimp(a)12@XZ */ locale__Locimp* __cdecl locale__Getgloballocale(void) { - FIXME("stub\n"); - return NULL; + TRACE("\n"); + return global_locale; } /* ?_Setgloballocale(a)locale@std@@CAXPAX(a)Z */ /* ?_Setgloballocale(a)locale@std@@CAXPEAX(a)Z */ void __cdecl locale__Setgloballocale(void *locimp) { - FIXME("(%p) stub\n", locimp); + TRACE("(%p)\n", locimp); + global_locale = locimp; } /* ?classic(a)locale@std@@SAABV12(a)XZ */ /* ?classic(a)locale@std@@SAAEBV12(a)XZ */ const locale* __cdecl locale_classic(void) { - FIXME("stub\n"); - return NULL; + TRACE("\n"); + locale__Init(); + return &classic_locale; } /* ?name(a)locale@std@@QBE?AV?$basic_string(a)DU?$char_traits(a)D@std@@V?$allocator(a)D@2@@2(a)XZ */ @@ -4682,3 +4708,11 @@ void __asm_dummy_vtables(void) { #ifndef __GNUC__ } #endif + +void free_locale(void) +{ + if(global_locale) { + locale__Locimp_dtor(global_locale); + locale_dtor(&classic_locale); + } +} diff --git a/dlls/msvcp90/msvcp90.h b/dlls/msvcp90/msvcp90.h index f15e832..47bcc26 100644 --- a/dlls/msvcp90/msvcp90.h +++ b/dlls/msvcp90/msvcp90.h @@ -267,6 +267,7 @@ locale* __thiscall locale_ctor(locale*); locale* __thiscall locale_copy_ctor(locale*, const locale*); locale* __thiscall locale_operator_assign(locale*, const locale*); void __thiscall locale_dtor(locale*); +void free_locale(void); /* class _Lockit */ typedef struct { diff --git a/dlls/msvcp90/msvcp90_main.c b/dlls/msvcp90/msvcp90_main.c index 6a1d4d2..de036e5 100644 --- a/dlls/msvcp90/msvcp90_main.c +++ b/dlls/msvcp90/msvcp90_main.c @@ -92,6 +92,7 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) init_lockit(); break; case DLL_PROCESS_DETACH: + free_locale(); free_lockit(); break; }

1 0

Piotr Caban : msvcp90: Added use_facet< codecvt<char> > implementation.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 9c83b6ee42574857d035f1a379cf766003686761 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9c83b6ee42574857d035f1a37… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Jun 12 18:12:32 2012 +0200 msvcp90: Added use_facet< codecvt<char> > implementation. --- dlls/msvcp90/locale.c | 73 +++++++++++++++++++++++++++++++++++++++++++---- dlls/msvcp90/msvcp90.h | 6 ++++ 2 files changed, 72 insertions(+), 7 deletions(-) diff --git a/dlls/msvcp90/locale.c b/dlls/msvcp90/locale.c index 49d864a..f5fec45 100644 --- a/dlls/msvcp90/locale.c +++ b/dlls/msvcp90/locale.c @@ -25,6 +25,8 @@ #include "errno.h" #include "limits.h" +#include "wine/list.h" + #include "windef.h" #include "winbase.h" #include "winnls.h" @@ -37,6 +39,7 @@ char* __cdecl _Getmonths(void); void* __cdecl _Gettnames(void); unsigned int __cdecl ___lc_codepage_func(void); LCID* __cdecl ___lc_handle_func(void); +const locale_facet* __thiscall locale__Getfacet(const locale*, MSVCP_size_t); typedef int category; @@ -44,11 +47,6 @@ typedef struct { MSVCP_size_t id; } locale_id; -typedef struct { - const vtable_ptr *vtable; - MSVCP_size_t refs; -} locale_facet; - typedef struct _locale__Locimp { locale_facet facet; locale_facet **facetvec; @@ -200,6 +198,25 @@ locale_facet* __thiscall MSVCP_locale_facet_vector_dtor(locale_facet *this, unsi return this; } +typedef struct +{ + locale_facet *fac; + struct list entry; +} facets_elem; +static struct list lazy_facets = LIST_INIT(lazy_facets); + +static void locale_facet_register(locale_facet *add) +{ + facets_elem *head = MSVCRT_operator_new(sizeof(*head)); + if(!head) { + ERR("Out of memory\n"); + throw_exception(EXCEPTION_BAD_ALLOC, NULL); + } + + head->fac = add; + list_add_head(&lazy_facets, &head->entry); +} + extern const vtable_ptr MSVCP_locale_facet_vtable; /* ??0id(a)locale@std@@QAE(a)I@Z */ @@ -2657,6 +2674,32 @@ MSVCP_size_t __cdecl codecvt_char__Getcat(const locale_facet **facet, const loca return LC_CTYPE; } +codecvt_char* codecvt_char_use_facet(locale *loc) +{ + static codecvt_char *obj = NULL; + + _Lockit lock; + const locale_facet *fac; + + _Lockit_ctor_locktype(&lock, _LOCK_LOCALE); + fac = locale__Getfacet(loc, codecvt_char_id.id); + if(fac) { + _Lockit_dtor(&lock); + return (codecvt_char*)fac; + } + + if(obj) + return obj; + + codecvt_char__Getcat(&fac, loc); + obj = (codecvt_char*)fac; + locale_facet__Incref(&obj->base.facet); + locale_facet_register(&obj->base.facet); + _Lockit_dtor(&lock); + + return obj; +} + /* ?do_in@?$codecvt(a)DDH@std@@MBEHAAHPBD1AAPBDPAD3AAPAD(a)Z */ /* ?do_in@?$codecvt(a)DDH@std@@MEBAHAEAHPEBD1AEAPEBDPEAD3AEAPEAD(a)Z */ #define call_codecvt_char_do_in(this, state, from, from_end, from_next, to, to_end, to_next) \ @@ -4525,8 +4568,15 @@ locale* __thiscall locale__Addfac(locale *this, locale_facet *facet, MSVCP_size_ DEFINE_THISCALL_WRAPPER(locale__Getfacet, 8) const locale_facet* __thiscall locale__Getfacet(const locale *this, MSVCP_size_t id) { - FIXME("(%p %lu) stub\n", this, id); - return NULL; + locale_facet *fac; + + TRACE("(%p %lu)\n", this, id); + + fac = id < this->ptr->facet_cnt ? this->ptr->facetvec[id] : NULL; + if(fac || !this->ptr->transparent) + return fac; + + return id < global_locale->facet_cnt ? global_locale->facetvec[id] : NULL; } /* ?_Getgloballocale(a)locale@std@@CAPAV_Locimp(a)12@XZ */ @@ -4711,8 +4761,17 @@ void __asm_dummy_vtables(void) { void free_locale(void) { + facets_elem *iter, *safe; + if(global_locale) { locale__Locimp_dtor(global_locale); locale_dtor(&classic_locale); } + + LIST_FOR_EACH_ENTRY_SAFE(iter, safe, &lazy_facets, facets_elem, entry) { + list_remove(&iter->entry); + if(locale_facet__Decref(iter->fac)) + call_locale_facet_vector_dtor(iter->fac, 1); + MSVCRT_operator_delete(iter); + } } diff --git a/dlls/msvcp90/msvcp90.h b/dlls/msvcp90/msvcp90.h index 47bcc26..aebd034 100644 --- a/dlls/msvcp90/msvcp90.h +++ b/dlls/msvcp90/msvcp90.h @@ -257,6 +257,12 @@ wchar_t* __stdcall MSVCP_allocator_wchar_allocate(void*, MSVCP_size_t); void __stdcall MSVCP_allocator_wchar_deallocate(void*, wchar_t*, MSVCP_size_t); MSVCP_size_t __stdcall MSVCP_allocator_wchar_max_size(void*); +/* class locale::facet */ +typedef struct { + const vtable_ptr *vtable; + MSVCP_size_t refs; +} locale_facet; + /* class locale */ typedef struct {

1 0

Piotr Caban : msvcp90: Added locale::global implementation.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 65db238da0bd62bd830d4867abef6c921f3e04f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=65db238da0bd62bd830d4867a… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Jun 12 18:12:51 2012 +0200 msvcp90: Added locale::global implementation. --- dlls/msvcp90/locale.c | 27 +++++++++++++++++++++++++++ dlls/msvcp90/msvcp90.spec | 4 ++-- 2 files changed, 29 insertions(+), 2 deletions(-) diff --git a/dlls/msvcp90/locale.c b/dlls/msvcp90/locale.c index f5fec45..ca8001b 100644 --- a/dlls/msvcp90/locale.c +++ b/dlls/msvcp90/locale.c @@ -4614,6 +4614,33 @@ basic_string_char* __thiscall locale_name(const locale *this, basic_string_char return ret; } +/* ?global(a)locale@std@@SA?AV12(a)ABV12@@Z */ +/* ?global(a)locale@std@@SA?AV12(a)AEBV12@@Z */ +locale* __cdecl locale_global(locale *ret, const locale *loc) +{ + _Lockit lock; + int i; + + TRACE("(%p %p)\n", loc, ret); + + _Lockit_ctor_locktype(&lock, _LOCK_LOCALE); + locale_ctor(ret); + + if(loc->ptr != global_locale) { + locale_facet__Decref(&global_locale->facet); + global_locale = loc->ptr; + locale_facet__Incref(&global_locale->facet); + + for(i=LC_ALL+1; i<=LC_MAX; i++) { + if((global_locale->catmask & (1<<(i-1))) == 0) + continue; + setlocale(i, MSVCP_basic_string_char_c_str(&global_locale->name)); + } + } + _Lockit_dtor(&lock); + return ret; +} + DEFINE_RTTI_DATA(locale_facet, 0, 0, NULL, NULL, NULL, ".?AVfacet(a)locale@std@@"); DEFINE_RTTI_DATA(collate_char, 0, 1, &locale_facet_rtti_base_descriptor, NULL, NULL, ".?AV?$collate(a)D@std@@"); DEFINE_RTTI_DATA(collate_wchar, 0, 1, &locale_facet_rtti_base_descriptor, NULL, NULL, ".?AV?$collate(a)_W@std@@"); diff --git a/dlls/msvcp90/msvcp90.spec b/dlls/msvcp90/msvcp90.spec index 880e84d..a823f3f 100644 --- a/dlls/msvcp90/msvcp90.spec +++ b/dlls/msvcp90/msvcp90.spec @@ -4155,8 +4155,8 @@ @ cdecl -arch=win64 ?getloc@?$basic_streambuf(a)_WU?$char_traits(a)_W@std@@@std@@QEBA?AVlocale(a)2@XZ(ptr ptr) basic_streambuf_wchar_getloc @ thiscall -arch=win32 ?getloc(a)ios_base@std@@QBE?AVlocale(a)2@XZ(ptr ptr) ios_base_getloc @ cdecl -arch=win64 ?getloc(a)ios_base@std@@QEBA?AVlocale(a)2@XZ(ptr ptr) ios_base_getloc -@ stub -arch=win32 ?global(a)locale@std@@SA?AV12(a)ABV12@@Z -@ stub -arch=win64 ?global(a)locale@std@@SA?AV12(a)AEBV12@@Z +@ cdecl -arch=win32 ?global(a)locale@std@@SA?AV12(a)ABV12@@Z(ptr ptr) locale_global +@ cdecl -arch=win64 ?global(a)locale@std@@SA?AV12(a)AEBV12@@Z(ptr ptr) locale_global @ thiscall -arch=win32 ?good(a)ios_base@std@@QBE_NXZ(ptr) ios_base_good @ cdecl -arch=win64 ?good(a)ios_base@std@@QEBA_NXZ(ptr) ios_base_good @ thiscall -arch=win32 ?gptr@?$basic_streambuf(a)DU?$char_traits(a)D@std@@@std@@IBEPADXZ(ptr) basic_streambuf_char_gptr

1 0

Huw Davies : winspool: Update the ppd file of an already installed printer in the first call to OpenPrinter .
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 28ef1af6d0435211e243bc4b22856fa8fc579d52 URL: http://source.winehq.org/git/wine.git/?a=commit;h=28ef1af6d0435211e243bc4b2… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Jun 12 09:42:49 2012 +0100 winspool: Update the ppd file of an already installed printer in the first call to OpenPrinter. --- dlls/winspool.drv/info.c | 201 +++++++++++++++++++++++++++++++--------------- 1 files changed, 137 insertions(+), 64 deletions(-) diff --git a/dlls/winspool.drv/info.c b/dlls/winspool.drv/info.c index a5ccf49..ed470bb 100644 --- a/dlls/winspool.drv/info.c +++ b/dlls/winspool.drv/info.c @@ -450,6 +450,62 @@ static inline DWORD set_reg_DWORD( HKEY hkey, const WCHAR *keyname, const DWORD return RegSetValueExW( hkey, keyname, 0, REG_DWORD, (const BYTE*)&value, sizeof(value) ); } +/****************************************************************** + * get_opened_printer + * Get the pointer to the opened printer referred by the handle + */ +static opened_printer_t *get_opened_printer(HANDLE hprn) +{ + UINT_PTR idx = (UINT_PTR)hprn; + opened_printer_t *ret = NULL; + + EnterCriticalSection(&printer_handles_cs); + + if ((idx > 0) && (idx <= nb_printer_handles)) { + ret = printer_handles[idx - 1]; + } + LeaveCriticalSection(&printer_handles_cs); + return ret; +} + +/****************************************************************** + * get_opened_printer_name + * Get the pointer to the opened printer name referred by the handle + */ +static LPCWSTR get_opened_printer_name(HANDLE hprn) +{ + opened_printer_t *printer = get_opened_printer(hprn); + if(!printer) return NULL; + return printer->name; +} + +static DWORD open_printer_reg_key( const WCHAR *name, HKEY *key ) +{ + HKEY printers; + DWORD err; + + *key = NULL; + err = RegCreateKeyW( HKEY_LOCAL_MACHINE, PrintersW, &printers ); + if (err) return err; + + err = RegOpenKeyW( printers, name, key ); + if (err) err = ERROR_INVALID_PRINTER_NAME; + RegCloseKey( printers ); + return err; +} + +/****************************************************************** + * WINSPOOL_GetOpenedPrinterRegKey + * + */ +static DWORD WINSPOOL_GetOpenedPrinterRegKey(HANDLE hPrinter, HKEY *phkey) +{ + LPCWSTR name = get_opened_printer_name(hPrinter); + + if(!name) return ERROR_INVALID_HANDLE; + return open_printer_reg_key( name, phkey ); +} + static void WINSPOOL_SetDefaultPrinter(const char *devname, const char *name, BOOL force) { char qbuf[200]; @@ -476,13 +532,13 @@ WINSPOOL_SetDefaultPrinter(const char *devname, const char *name, BOOL force) { } } -static BOOL add_printer_driver(WCHAR *name, WCHAR *ppd) +static BOOL add_printer_driver(const WCHAR *name, WCHAR *ppd) { DRIVER_INFO_3W di3; ZeroMemory(&di3, sizeof(DRIVER_INFO_3W)); di3.cVersion = 3; - di3.pName = name; + di3.pName = (WCHAR*)name; di3.pEnvironment = envname_x86W; di3.pDriverPath = driver_nt; di3.pDataFile = ppd; @@ -831,7 +887,61 @@ static BOOL CUPS_LoadPrinters(void) RegCloseKey(hkeyPrinters); return TRUE; } + +static char *get_cups_name( HANDLE printer ) +{ + WCHAR *port; + DWORD err, needed, type; + char *ret = NULL; + HKEY key; + + err = WINSPOOL_GetOpenedPrinterRegKey( printer, &key ); + if (err) return NULL; + err = RegQueryValueExW( key, PortW, 0, &type, NULL, &needed ); + if (err || needed <= sizeof( CUPS_Port )) goto end; + port = HeapAlloc( GetProcessHeap(), 0, needed ); + if (!port) goto end; + RegQueryValueExW( key, PortW, 0, &type, (BYTE*)port, &needed ); + if (!memcmp( port, CUPS_Port, sizeof(CUPS_Port) - sizeof(WCHAR) )) + { + WCHAR *name = port + sizeof(CUPS_Port) / sizeof(WCHAR) - 1; + needed = WideCharToMultiByte( CP_UNIXCP, 0, name, -1, NULL, 0, NULL, NULL ); + ret = HeapAlloc( GetProcessHeap(), 0, needed ); + if(ret) WideCharToMultiByte( CP_UNIXCP, 0, name, -1, ret, needed, NULL, NULL ); + } + HeapFree( GetProcessHeap(), 0, port ); +end: + RegCloseKey( key ); + return ret; +} +#endif + +static BOOL update_driver( HANDLE printer ) +{ + BOOL ret = FALSE; +#ifdef SONAME_LIBCUPS + const WCHAR *name = get_opened_printer_name( printer ); + WCHAR *ppd_dir, *ppd; + char *cups_name; + + if (!name) return FALSE; + cups_name = get_cups_name( printer ); + if (!cups_name) return FALSE; + + ppd_dir = get_ppd_dir(); + ppd = get_ppd_filename( ppd_dir, name ); + if (get_cups_ppd( cups_name, ppd )) + { + TRACE( "updating driver %s\n", debugstr_w( name ) ); + ret = add_printer_driver( name, ppd ); + unlink_ppd( ppd ); + } + HeapFree( GetProcessHeap(), 0, ppd_dir ); + HeapFree( GetProcessHeap(), 0, ppd ); + HeapFree( GetProcessHeap(), 0, cups_name ); #endif + return ret; +} static BOOL PRINTCAP_ParseEntry( const char *pent, BOOL isfirst ) { @@ -1243,62 +1353,6 @@ end: return (HANDLE)handle; } -/****************************************************************** - * get_opened_printer - * Get the pointer to the opened printer referred by the handle - */ -static opened_printer_t *get_opened_printer(HANDLE hprn) -{ - UINT_PTR idx = (UINT_PTR)hprn; - opened_printer_t *ret = NULL; - - EnterCriticalSection(&printer_handles_cs); - - if ((idx > 0) && (idx <= nb_printer_handles)) { - ret = printer_handles[idx - 1]; - } - LeaveCriticalSection(&printer_handles_cs); - return ret; -} - -/****************************************************************** - * get_opened_printer_name - * Get the pointer to the opened printer name referred by the handle - */ -static LPCWSTR get_opened_printer_name(HANDLE hprn) -{ - opened_printer_t *printer = get_opened_printer(hprn); - if(!printer) return NULL; - return printer->name; -} - -static DWORD open_printer_reg_key( const WCHAR *name, HKEY *key ) -{ - HKEY printers; - DWORD err; - - *key = NULL; - err = RegCreateKeyW( HKEY_LOCAL_MACHINE, PrintersW, &printers ); - if (err) return err; - - err = RegOpenKeyW( printers, name, key ); - if (err) err = ERROR_INVALID_PRINTER_NAME; - RegCloseKey( printers ); - return err; -} - -/****************************************************************** - * WINSPOOL_GetOpenedPrinterRegKey - * - */ -static DWORD WINSPOOL_GetOpenedPrinterRegKey(HANDLE hPrinter, HKEY *phkey) -{ - LPCWSTR name = get_opened_printer_name(hPrinter); - - if(!name) return ERROR_INVALID_HANDLE; - return open_printer_reg_key( name, phkey ); -} - static void old_printer_check( BOOL delete_phase ) { PRINTER_INFO_5W* pi; @@ -1348,6 +1402,7 @@ static void old_printer_check( BOOL delete_phase ) static const WCHAR winspool_mutex_name[] = {'_','_','W','I','N','E','_','W','I','N','S','P','O','O','L','_', 'M','U','T','E','X','_','_','\0'}; +static HANDLE init_mutex; void WINSPOOL_LoadSystemPrinters(void) { @@ -1355,19 +1410,18 @@ void WINSPOOL_LoadSystemPrinters(void) DWORD needed, num, i; WCHAR PrinterName[256]; BOOL done = FALSE; - HANDLE mutex; /* FIXME: The init code should be moved to spoolsv.exe */ - mutex = CreateMutexW( NULL, TRUE, winspool_mutex_name ); - if (!mutex) + init_mutex = CreateMutexW( NULL, TRUE, winspool_mutex_name ); + if (!init_mutex) { ERR( "Failed to create mutex\n" ); return; } if (GetLastError() == ERROR_ALREADY_EXISTS) { - WaitForSingleObject( mutex, INFINITE ); - ReleaseMutex( mutex ); + WaitForSingleObject( init_mutex, INFINITE ); + ReleaseMutex( init_mutex ); TRACE( "Init already done\n" ); return; } @@ -1403,7 +1457,7 @@ void WINSPOOL_LoadSystemPrinters(void) old_printer_check( TRUE ); - ReleaseMutex( mutex ); + ReleaseMutex( init_mutex ); return; } @@ -2277,6 +2331,25 @@ BOOL WINAPI OpenPrinterW(LPWSTR lpPrinterName,HANDLE *phPrinter, LPPRINTER_DEFAU /* Get the unique handle of the printer or Printserver */ *phPrinter = get_opened_printer_entry(lpPrinterName, pDefault); + + WaitForSingleObject( init_mutex, INFINITE ); + if (*phPrinter) + { + HKEY key; + DWORD deleting = 0, size = sizeof( deleting ), type; + DWORD status; + WINSPOOL_GetOpenedPrinterRegKey( *phPrinter, &key ); + RegQueryValueExW( key, May_Delete_Value, NULL, &type, (LPBYTE)&deleting, &size ); + status = get_dword_from_reg( key, StatusW ); + if (!deleting && (status & PRINTER_STATUS_DRIVER_UPDATE_NEEDED)) + { + update_driver( *phPrinter ); + set_reg_DWORD( key, StatusW, status & ~PRINTER_STATUS_DRIVER_UPDATE_NEEDED ); + } + RegCloseKey( key ); + } + ReleaseMutex( init_mutex ); + TRACE("returning %d with %u and %p\n", *phPrinter != NULL, GetLastError(), *phPrinter); return (*phPrinter != 0); }

1 0

Aric Stewart : usp10: In ShapeCharGlyphProp_BaseIndic set justification to SCRIPT_JUSTIFY_NONE for glyphs with no characters .
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 9ea4543d12adecb749ceba9e150413844fbe0011 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9ea4543d12adecb749ceba9e1… Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Jun 12 08:22:50 2012 -0500 usp10: In ShapeCharGlyphProp_BaseIndic set justification to SCRIPT_JUSTIFY_NONE for glyphs with no characters. --- dlls/usp10/shape.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/usp10/shape.c b/dlls/usp10/shape.c index cb5adcc..c654b07 100644 --- a/dlls/usp10/shape.c +++ b/dlls/usp10/shape.c @@ -2991,7 +2991,10 @@ static void ShapeCharGlyphProp_BaseIndic( HDC hdc, ScriptCache *psc, SCRIPT_ANAL } if (char_count == 0) + { + pGlyphProp[i].sva.uJustification = SCRIPT_JUSTIFY_NONE; continue; + } if (char_count ==1 && pwcChars[char_index[0]] == 0x0020) /* space */ {

1 0

Hans Leidekker : msi: Don't fail if no .NET runtime is available.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 458f6b7ccd0c25753ffbc071673d034dc02fe9c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=458f6b7ccd0c25753ffbc0716… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Jun 12 09:24:45 2012 +0200 msi: Don't fail if no .NET runtime is available. --- dlls/msi/assembly.c | 98 ++++++++++++++++---------------------------------- 1 files changed, 32 insertions(+), 66 deletions(-) diff --git a/dlls/msi/assembly.c b/dlls/msi/assembly.c index d384003..bfa5142 100644 --- a/dlls/msi/assembly.c +++ b/dlls/msi/assembly.c @@ -49,13 +49,21 @@ static BOOL init_function_pointers( void ) static const WCHAR szVersion20[] = {'v','2','.','0','.','5','0','7','2','7',0}; static const WCHAR szVersion40[] = {'v','4','.','0','.','3','0','3','1','9',0}; - if (pCreateAssemblyCacheNet10 || pCreateAssemblyCacheNet11 || pCreateAssemblyCacheNet20 || - pCreateAssemblyCacheNet40 ) return TRUE; - - if (!(hmscoree = LoadLibraryA( "mscoree.dll" ))) return FALSE; + if (!hsxs && !(hsxs = LoadLibraryA( "sxs.dll" ))) return FALSE; + if (!(pCreateAssemblyCacheSxs = (void *)GetProcAddress( hsxs, "CreateAssemblyCache" ))) + { + FreeLibrary( hsxs ); + hsxs = NULL; + return FALSE; + } + if (hmscoree || !(hmscoree = LoadLibraryA( "mscoree.dll" ))) return TRUE; pGetFileVersion = (void *)GetProcAddress( hmscoree, "GetFileVersion" ); /* missing from v1.0.3705 */ - if (!(pLoadLibraryShim = (void *)GetProcAddress( hmscoree, "LoadLibraryShim" ))) goto error; - + if (!(pLoadLibraryShim = (void *)GetProcAddress( hmscoree, "LoadLibraryShim" ))) + { + FreeLibrary( hmscoree ); + hmscoree = NULL; + return TRUE; + } if (!pLoadLibraryShim( szFusion, szVersion10, NULL, &hfusion10 )) pCreateAssemblyCacheNet10 = (void *)GetProcAddress( hfusion10, "CreateAssemblyCache" ); @@ -68,76 +76,27 @@ static BOOL init_function_pointers( void ) if (!pLoadLibraryShim( szFusion, szVersion40, NULL, &hfusion40 )) pCreateAssemblyCacheNet40 = (void *)GetProcAddress( hfusion40, "CreateAssemblyCache" ); - if (!pCreateAssemblyCacheNet10 && !pCreateAssemblyCacheNet11 && !pCreateAssemblyCacheNet20 - && !pCreateAssemblyCacheNet40) goto error; - - if (!(hsxs = LoadLibraryA( "sxs.dll" ))) goto error; - if (!(pCreateAssemblyCacheSxs = (void *)GetProcAddress( hsxs, "CreateAssemblyCache" ))) goto error; return TRUE; - -error: - pCreateAssemblyCacheNet10 = NULL; - pCreateAssemblyCacheNet11 = NULL; - pCreateAssemblyCacheNet20 = NULL; - pCreateAssemblyCacheNet40 = NULL; - FreeLibrary( hfusion10 ); - FreeLibrary( hfusion11 ); - FreeLibrary( hfusion20 ); - FreeLibrary( hfusion40 ); - FreeLibrary( hmscoree ); - return FALSE; } BOOL msi_init_assembly_caches( MSIPACKAGE *package ) { if (!init_function_pointers()) return FALSE; - if (package->cache_net[CLR_VERSION_V10] || - package->cache_net[CLR_VERSION_V11] || - package->cache_net[CLR_VERSION_V20] || - package->cache_net[CLR_VERSION_V40]) return TRUE; if (pCreateAssemblyCacheSxs( &package->cache_sxs, 0 ) != S_OK) return FALSE; - if (pCreateAssemblyCacheNet10) pCreateAssemblyCacheNet10( &package->cache_net[CLR_VERSION_V10], 0 ); if (pCreateAssemblyCacheNet11) pCreateAssemblyCacheNet11( &package->cache_net[CLR_VERSION_V11], 0 ); if (pCreateAssemblyCacheNet20) pCreateAssemblyCacheNet20( &package->cache_net[CLR_VERSION_V20], 0 ); if (pCreateAssemblyCacheNet40) pCreateAssemblyCacheNet40( &package->cache_net[CLR_VERSION_V40], 0 ); - - if (package->cache_net[CLR_VERSION_V10] || - package->cache_net[CLR_VERSION_V11] || - package->cache_net[CLR_VERSION_V20] || - package->cache_net[CLR_VERSION_V40]) - { - return TRUE; - } - if (package->cache_net[CLR_VERSION_V10]) - { - IAssemblyCache_Release( package->cache_net[CLR_VERSION_V10] ); - package->cache_net[CLR_VERSION_V10] = NULL; - } - if (package->cache_net[CLR_VERSION_V11]) - { - IAssemblyCache_Release( package->cache_net[CLR_VERSION_V11] ); - package->cache_net[CLR_VERSION_V11] = NULL; - } - if (package->cache_net[CLR_VERSION_V20]) - { - IAssemblyCache_Release( package->cache_net[CLR_VERSION_V20] ); - package->cache_net[CLR_VERSION_V20] = NULL; - } - if (package->cache_net[CLR_VERSION_V40]) - { - IAssemblyCache_Release( package->cache_net[CLR_VERSION_V40] ); - package->cache_net[CLR_VERSION_V40] = NULL; - } - IAssemblyCache_Release( package->cache_sxs ); - package->cache_sxs = NULL; - return FALSE; + return TRUE; } void msi_destroy_assembly_caches( MSIPACKAGE *package ) { UINT i; + IAssemblyCache_Release( package->cache_sxs ); + package->cache_sxs = NULL; + for (i = 0; i < CLR_VERSION_MAX; i++) { if (package->cache_net[i]) @@ -146,11 +105,6 @@ void msi_destroy_assembly_caches( MSIPACKAGE *package ) package->cache_net[i] = NULL; } } - if (package->cache_sxs) - { - IAssemblyCache_Release( package->cache_sxs ); - package->cache_sxs = NULL; - } pCreateAssemblyCacheNet10 = NULL; pCreateAssemblyCacheNet11 = NULL; pCreateAssemblyCacheNet20 = NULL; @@ -161,6 +115,12 @@ void msi_destroy_assembly_caches( MSIPACKAGE *package ) FreeLibrary( hfusion40 ); FreeLibrary( hmscoree ); FreeLibrary( hsxs ); + hfusion10 = NULL; + hfusion11 = NULL; + hfusion20 = NULL; + hfusion40 = NULL; + hmscoree = NULL; + hsxs = NULL; } static MSIRECORD *get_assembly_record( MSIPACKAGE *package, const WCHAR *comp ) @@ -279,6 +239,8 @@ static BOOL is_assembly_installed( IAssemblyCache *cache, const WCHAR *display_n HRESULT hr; ASSEMBLY_INFO info; + if (!cache) return FALSE; + memset( &info, 0, sizeof(info) ); info.cbAssemblyInfo = sizeof(info); hr = IAssemblyCache_QueryAssemblyInfo( cache, 0, display_name, &info ); @@ -433,6 +395,7 @@ UINT msi_install_assembly( MSIPACKAGE *package, MSICOMPONENT *comp ) { manifest = msi_get_loaded_file( package, comp->KeyPath )->TargetPath; cache = package->cache_net[get_clr_version( manifest )]; + if (!cache) return ERROR_SUCCESS; } TRACE("installing assembly %s\n", debugstr_w(manifest)); @@ -477,8 +440,11 @@ UINT msi_uninstall_assembly( MSIPACKAGE *package, MSICOMPONENT *comp ) { if (!assembly->clr_version[i]) continue; cache = package->cache_net[i]; - hr = IAssemblyCache_UninstallAssembly( cache, 0, assembly->display_name, NULL, NULL ); - if (FAILED( hr )) WARN("failed to uninstall assembly 0x%08x\n", hr); + if (cache) + { + hr = IAssemblyCache_UninstallAssembly( cache, 0, assembly->display_name, NULL, NULL ); + if (FAILED( hr )) WARN("failed to uninstall assembly 0x%08x\n", hr); + } } } if (feature) feature->Action = INSTALLSTATE_ABSENT;

1 0

Hans Leidekker : msi: Always load mscoree.dll from the system directory.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: 3480ccbbb4d7ac9ec365595b57a9e938834d1efd URL: http://source.winehq.org/git/wine.git/?a=commit;h=3480ccbbb4d7ac9ec365595b5… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Jun 12 09:24:58 2012 +0200 msi: Always load mscoree.dll from the system directory. --- dlls/msi/assembly.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/dlls/msi/assembly.c b/dlls/msi/assembly.c index bfa5142..e7660bd 100644 --- a/dlls/msi/assembly.c +++ b/dlls/msi/assembly.c @@ -43,20 +43,25 @@ static HMODULE hfusion10, hfusion11, hfusion20, hfusion40, hmscoree, hsxs; static BOOL init_function_pointers( void ) { - static const WCHAR szFusion[] = {'f','u','s','i','o','n','.','d','l','l',0}; + static const WCHAR szFusion[] = {'f','u','s','i','o','n','.','d','l','l',0}; + static const WCHAR szMscoree[] = {'\\','m','s','c','o','r','e','e','.','d','l','l',0}; + static const WCHAR szSxs[] = {'s','x','s','.','d','l','l',0}; static const WCHAR szVersion10[] = {'v','1','.','0','.','3','7','0','5',0}; static const WCHAR szVersion11[] = {'v','1','.','1','.','4','3','2','2',0}; static const WCHAR szVersion20[] = {'v','2','.','0','.','5','0','7','2','7',0}; static const WCHAR szVersion40[] = {'v','4','.','0','.','3','0','3','1','9',0}; + WCHAR path[MAX_PATH]; + DWORD len = GetSystemDirectoryW( path, MAX_PATH ); - if (!hsxs && !(hsxs = LoadLibraryA( "sxs.dll" ))) return FALSE; + if (!hsxs && !(hsxs = LoadLibraryW( szSxs ))) return FALSE; if (!(pCreateAssemblyCacheSxs = (void *)GetProcAddress( hsxs, "CreateAssemblyCache" ))) { FreeLibrary( hsxs ); hsxs = NULL; return FALSE; } - if (hmscoree || !(hmscoree = LoadLibraryA( "mscoree.dll" ))) return TRUE; + strcpyW( path + len, szMscoree ); + if (hmscoree || !(hmscoree = LoadLibraryW( path ))) return TRUE; pGetFileVersion = (void *)GetProcAddress( hmscoree, "GetFileVersion" ); /* missing from v1.0.3705 */ if (!(pLoadLibraryShim = (void *)GetProcAddress( hmscoree, "LoadLibraryShim" ))) {

1 0

Hans Leidekker : msi: Keep assembly caches loaded until the package is destroyed.
by Alexandre Julliard June 12, 2012

June 12, 2012

Module: wine Branch: master Commit: a17ca164952526815e98a345a6f7513842baa535 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a17ca164952526815e98a345a… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Jun 12 09:25:09 2012 +0200 msi: Keep assembly caches loaded until the package is destroyed. --- dlls/msi/action.c | 1 - dlls/msi/assembly.c | 8 +++++--- dlls/msi/files.c | 4 ---- dlls/msi/package.c | 7 ++----- 4 files changed, 7 insertions(+), 13 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index b256439..d452c92 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -1017,7 +1017,6 @@ UINT msi_load_all_components( MSIPACKAGE *package ) r = MSI_IterateRecords(view, NULL, load_component, package); msiobj_release(&view->hdr); - msi_destroy_assembly_caches( package ); return r; } diff --git a/dlls/msi/assembly.c b/dlls/msi/assembly.c index e7660bd..2f321db 100644 --- a/dlls/msi/assembly.c +++ b/dlls/msi/assembly.c @@ -99,9 +99,11 @@ void msi_destroy_assembly_caches( MSIPACKAGE *package ) { UINT i; - IAssemblyCache_Release( package->cache_sxs ); - package->cache_sxs = NULL; - + if (package->cache_sxs) + { + IAssemblyCache_Release( package->cache_sxs ); + package->cache_sxs = NULL; + } for (i = 0; i < CLR_VERSION_MAX; i++) { if (package->cache_net[i]) diff --git a/dlls/msi/files.c b/dlls/msi/files.c index fe72c24..11913ef 100644 --- a/dlls/msi/files.c +++ b/dlls/msi/files.c @@ -399,7 +399,6 @@ UINT ACTION_InstallFiles(MSIPACKAGE *package) goto done; } } - msi_init_assembly_caches( package ); LIST_FOR_EACH_ENTRY( comp, &package->components, MSICOMPONENT, entry ) { comp->Action = msi_get_component_action( package, comp ); @@ -414,7 +413,6 @@ UINT ACTION_InstallFiles(MSIPACKAGE *package) } } } - msi_destroy_assembly_caches( package ); done: msi_free_media_info(mi); @@ -1309,7 +1307,6 @@ UINT ACTION_RemoveFiles( MSIPACKAGE *package ) msiobj_release( &uirow->hdr ); } - msi_init_assembly_caches( package ); LIST_FOR_EACH_ENTRY( comp, &package->components, MSICOMPONENT, entry ) { comp->Action = msi_get_component_action( package, comp ); @@ -1328,6 +1325,5 @@ UINT ACTION_RemoveFiles( MSIPACKAGE *package ) remove_folder( folder ); } } - msi_destroy_assembly_caches( package ); return ERROR_SUCCESS; } diff --git a/dlls/msi/package.c b/dlls/msi/package.c index 7c8b1b4..4944065 100644 --- a/dlls/msi/package.c +++ b/dlls/msi/package.c @@ -344,9 +344,10 @@ static void free_package_structures( MSIPACKAGE *package ) static void MSI_FreePackage( MSIOBJECTHDR *arg) { - UINT i; MSIPACKAGE *package = (MSIPACKAGE *)arg; + msi_destroy_assembly_caches( package ); + if( package->dialog ) msi_dialog_destroy( package->dialog ); @@ -354,10 +355,6 @@ static void MSI_FreePackage( MSIOBJECTHDR *arg) free_package_structures(package); CloseHandle( package->log_file ); - for (i = 0; i < CLR_VERSION_MAX; i++) - if (package->cache_net[i]) IAssemblyCache_Release( package->cache_net[i] ); - if (package->cache_sxs) IAssemblyCache_Release( package->cache_sxs ); - if (package->delete_on_close) DeleteFileW( package->localfile ); msi_free( package->localfile ); }

1 0