wine-commits August 2010

wine-commits@winehq.org

2 participants
884 discussions

Matteo Bruni : d3dx9_43: Add d3dx9_43.dll, forwarding to d3dx9_36.dll.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: 90c615aa6117a1014633b6a747aef2822eb3d0bc URL: http://source.winehq.org/git/wine.git/?a=commit;h=90c615aa6117a1014633b6a74… Author: Matteo Bruni <matteo.mystral(a)gmail.com> Date: Wed Aug 25 17:12:51 2010 +0200 d3dx9_43: Add d3dx9_43.dll, forwarding to d3dx9_36.dll. --- configure | 1 + configure.ac | 1 + dlls/d3dx9_43/Makefile.in | 13 ++ dlls/d3dx9_43/d3dx9_43.spec | 329 +++++++++++++++++++++++++++++++++++++++++ dlls/d3dx9_43/d3dx9_43_main.c | 63 ++++++++ dlls/d3dx9_43/version.rc | 27 ++++ 6 files changed, 434 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=90c615aa6117a1014633b…

1 0

Hans Leidekker : wininet: Handle HTTP_STATUS_REDIRECT_METHOD.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: 94deb85d12eba2a7a0162523e82b55320316ea6a URL: http://source.winehq.org/git/wine.git/?a=commit;h=94deb85d12eba2a7a0162523e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Aug 27 10:50:59 2010 +0200 wininet: Handle HTTP_STATUS_REDIRECT_METHOD. --- dlls/wininet/http.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index e74c974..f5ae426 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -3787,7 +3787,9 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *lpwhr, LPCWSTR lpszHeaders, { WCHAR *new_url, szNewLocation[INTERNET_MAX_URL_LENGTH]; dwBufferSize=sizeof(szNewLocation); - if ((dwStatusCode==HTTP_STATUS_REDIRECT || dwStatusCode==HTTP_STATUS_MOVED) && + if ((dwStatusCode == HTTP_STATUS_REDIRECT || + dwStatusCode == HTTP_STATUS_MOVED || + dwStatusCode == HTTP_STATUS_REDIRECT_METHOD) && HTTP_HttpQueryInfoW(lpwhr,HTTP_QUERY_LOCATION,szNewLocation,&dwBufferSize,NULL) == ERROR_SUCCESS) { if (strcmpW(lpwhr->lpszVerb, szGET) && strcmpW(lpwhr->lpszVerb, szHEAD))

1 0

Hans Leidekker : msi: Move database initialization to a separate function.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: dd7514a38d0be4bdd7f0635d1abad2fca832df0f URL: http://source.winehq.org/git/wine.git/?a=commit;h=dd7514a38d0be4bdd7f0635d1… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Aug 27 10:50:35 2010 +0200 msi: Move database initialization to a separate function. --- dlls/msi/database.c | 79 +++++++++++++++++++++++++++++++++------------------ 1 files changed, 51 insertions(+), 28 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index 986dcbb..748cbfd 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -255,6 +255,43 @@ static VOID MSI_CloseDatabase( MSIOBJECTHDR *arg ) } } +static HRESULT db_initialize( IStorage *stg, const GUID *clsid ) +{ + static const WCHAR szTables[] = { '_','T','a','b','l','e','s',0 }; + HRESULT hr; + + hr = IStorage_SetClass( stg, clsid ); + if (FAILED( hr )) + { + WARN("failed to set class id 0x%08x\n", hr); + return hr; + } + + /* create the _Tables stream */ + hr = write_stream_data( stg, szTables, NULL, 0, TRUE ); + if (FAILED( hr )) + { + WARN("failed to create _Tables stream 0x%08x\n", hr); + return hr; + } + + hr = msi_init_string_table( stg ); + if (FAILED( hr )) + { + WARN("failed to initialize string table 0x%08x\n", hr); + return hr; + } + + hr = IStorage_Commit( stg, 0 ); + if (FAILED( hr )) + { + WARN("failed to commit changes 0x%08x\n", hr); + return hr; + } + + return S_OK; +} + UINT MSI_OpenDatabaseW(LPCWSTR szDBPath, LPCWSTR szPersist, MSIDATABASE **pdb) { IStorage *stg = NULL; @@ -266,8 +303,6 @@ UINT MSI_OpenDatabaseW(LPCWSTR szDBPath, LPCWSTR szPersist, MSIDATABASE **pdb) BOOL created = FALSE, patch = FALSE; WCHAR path[MAX_PATH]; - static const WCHAR szTables[] = { '_','T','a','b','l','e','s',0 }; - TRACE("%s %s\n",debugstr_w(szDBPath),debugstr_w(szPersist) ); if( !pdb ) @@ -298,34 +333,22 @@ UINT MSI_OpenDatabaseW(LPCWSTR szDBPath, LPCWSTR szPersist, MSIDATABASE **pdb) r = StgOpenStorage( szDBPath, NULL, STGM_DIRECT|STGM_READ|STGM_SHARE_DENY_WRITE, NULL, 0, &stg); } - else if( szPersist == MSIDBOPEN_CREATE || szPersist == MSIDBOPEN_CREATEDIRECT ) + else if( szPersist == MSIDBOPEN_CREATE ) { - if ( szPersist == MSIDBOPEN_CREATE ) - { - r = StgCreateDocfile( szDBPath, - STGM_CREATE|STGM_TRANSACTED|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg); - } - else - { - r = StgCreateDocfile( szDBPath, - STGM_CREATE|STGM_DIRECT|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg); - } + r = StgCreateDocfile( szDBPath, + STGM_CREATE|STGM_TRANSACTED|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg ); + if( SUCCEEDED(r) ) - { - IStorage_SetClass( stg, patch ? &CLSID_MsiPatch : &CLSID_MsiDatabase ); - /* create the _Tables stream */ - r = write_stream_data(stg, szTables, NULL, 0, TRUE); - if (SUCCEEDED(r)) - { - r = msi_init_string_table( stg ); - if (SUCCEEDED(r)) - { - r = IStorage_Commit( stg, 0 ); - if (FAILED(r)) - WARN("failed to commit changes 0x%08x\n", r); - } - } - } + r = db_initialize( stg, patch ? &CLSID_MsiPatch : &CLSID_MsiDatabase ); + created = TRUE; + } + else if( szPersist == MSIDBOPEN_CREATEDIRECT ) + { + r = StgCreateDocfile( szDBPath, + STGM_CREATE|STGM_DIRECT|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg ); + + if( SUCCEEDED(r) ) + r = db_initialize( stg, patch ? &CLSID_MsiPatch : &CLSID_MsiDatabase ); created = TRUE; } else if( szPersist == MSIDBOPEN_TRANSACT )

1 0

Hans Leidekker : msi: Open the database storage in transacted mode when MSIDBOPEN_CREATE or MSIDBOPEN_TRANSACT is specified .
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: eef54ffe6bcda7a17a62af432f6386e6967e795a URL: http://source.winehq.org/git/wine.git/?a=commit;h=eef54ffe6bcda7a17a62af432… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Aug 27 10:50:20 2010 +0200 msi: Open the database storage in transacted mode when MSIDBOPEN_CREATE or MSIDBOPEN_TRANSACT is specified. --- dlls/msi/database.c | 28 ++++++++++++++++++++-------- dlls/msi/table.c | 9 ++++++++- 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/dlls/msi/database.c b/dlls/msi/database.c index 01c3249..986dcbb 100644 --- a/dlls/msi/database.c +++ b/dlls/msi/database.c @@ -300,26 +300,38 @@ UINT MSI_OpenDatabaseW(LPCWSTR szDBPath, LPCWSTR szPersist, MSIDATABASE **pdb) } else if( szPersist == MSIDBOPEN_CREATE || szPersist == MSIDBOPEN_CREATEDIRECT ) { - /* FIXME: MSIDBOPEN_CREATE should case STGM_TRANSACTED flag to be - * used here: */ - r = StgCreateDocfile( szDBPath, - STGM_CREATE|STGM_DIRECT|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg); - if( r == ERROR_SUCCESS ) + if ( szPersist == MSIDBOPEN_CREATE ) + { + r = StgCreateDocfile( szDBPath, + STGM_CREATE|STGM_TRANSACTED|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg); + } + else + { + r = StgCreateDocfile( szDBPath, + STGM_CREATE|STGM_DIRECT|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, 0, &stg); + } + if( SUCCEEDED(r) ) { IStorage_SetClass( stg, patch ? &CLSID_MsiPatch : &CLSID_MsiDatabase ); /* create the _Tables stream */ r = write_stream_data(stg, szTables, NULL, 0, TRUE); if (SUCCEEDED(r)) + { r = msi_init_string_table( stg ); + if (SUCCEEDED(r)) + { + r = IStorage_Commit( stg, 0 ); + if (FAILED(r)) + WARN("failed to commit changes 0x%08x\n", r); + } + } } created = TRUE; } else if( szPersist == MSIDBOPEN_TRANSACT ) { - /* FIXME: MSIDBOPEN_TRANSACT should case STGM_TRANSACTED flag to be - * used here: */ r = StgOpenStorage( szDBPath, NULL, - STGM_DIRECT|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, NULL, 0, &stg); + STGM_TRANSACTED|STGM_READWRITE|STGM_SHARE_EXCLUSIVE, NULL, 0, &stg); } else if( szPersist == MSIDBOPEN_DIRECT ) { diff --git a/dlls/msi/table.c b/dlls/msi/table.c index 72d1f21..6755496 100644 --- a/dlls/msi/table.c +++ b/dlls/msi/table.c @@ -2319,6 +2319,7 @@ UINT TABLE_CreateView( MSIDATABASE *db, LPCWSTR name, MSIVIEW **view ) UINT MSI_CommitTables( MSIDATABASE *db ) { UINT r; + HRESULT hr; MSITABLE *table = NULL; TRACE("%p\n",db); @@ -2344,7 +2345,13 @@ UINT MSI_CommitTables( MSIDATABASE *db ) /* force everything to reload next time */ free_cached_tables( db ); - return ERROR_SUCCESS; + hr = IStorage_Commit( db->storage, 0 ); + if (FAILED( hr )) + { + WARN("failed to commit changes 0x%08x\n", hr); + r = ERROR_FUNCTION_FAILED; + } + return r; } MSICONDITION MSI_DatabaseIsTablePersistent( MSIDATABASE *db, LPCWSTR table )

1 0

Hans Leidekker : msi/tests: Fix a WoW64 test failure.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: d411c49dd148270d8abb55f1d9f4c6b9f334929a URL: http://source.winehq.org/git/wine.git/?a=commit;h=d411c49dd148270d8abb55f1d… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Aug 27 10:50:07 2010 +0200 msi/tests: Fix a WoW64 test failure. --- dlls/msi/tests/automation.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msi/tests/automation.c b/dlls/msi/tests/automation.c index 2b1449b..f883eac 100644 --- a/dlls/msi/tests/automation.c +++ b/dlls/msi/tests/automation.c @@ -2546,8 +2546,8 @@ static void test_Installer_InstallProduct(void) ok(res == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", res); /* Remove registry keys written by RegisterProduct standard action */ - res = delete_key_portable(HKEY_LOCAL_MACHINE, - "SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\{F1C3AF50-8B56-4A69-A00C-00773FE42F30}", access); + res = RegDeleteKeyA(HKEY_LOCAL_MACHINE, + "SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\{F1C3AF50-8B56-4A69-A00C-00773FE42F30}"); ok(res == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", res); res = delete_key_portable(HKEY_LOCAL_MACHINE,

1 0

Hans Leidekker : msi: Create the product key if necessary in msi_publish_patches.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: 44b15b061b22d0929019b2831ea83ba3762c32b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=44b15b061b22d0929019b2831… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Aug 27 10:49:55 2010 +0200 msi: Create the product key if necessary in msi_publish_patches. Fixes a regression caused by c9fb24927db773905e144db63d9e34741e97f703. --- dlls/msi/action.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index bb559c0..5eb4ae8 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -3891,7 +3891,7 @@ static UINT msi_publish_patches( MSIPACKAGE *package ) WCHAR *p, *all_patches = NULL; DWORD len = 0; - r = MSIREG_OpenProductKey( package->ProductCode, NULL, package->Context, &product_key, FALSE ); + r = MSIREG_OpenProductKey( package->ProductCode, NULL, package->Context, &product_key, TRUE ); if (r != ERROR_SUCCESS) return ERROR_FUNCTION_FAILED;

1 0

Hans Leidekker : msi/tests: Use a different product code in the package tests to avoid cascading test failures .
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: 137dfbe80a5cc42b2c6214124144cbc431b79bb3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=137dfbe80a5cc42b2c6214124… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Aug 27 10:49:31 2010 +0200 msi/tests: Use a different product code in the package tests to avoid cascading test failures. --- dlls/msi/tests/package.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c index 9e7f104..f9d6a63 100644 --- a/dlls/msi/tests/package.c +++ b/dlls/msi/tests/package.c @@ -2478,7 +2478,7 @@ static void test_states(void) r = create_property_table( hdb ); ok( r == ERROR_SUCCESS, "cannot create Property table: %d\n", r ); - r = add_property_entry( hdb, "'ProductCode', '{7DF88A48-996F-4EC8-A022-BF956F9B2CBB}'" ); + r = add_property_entry( hdb, "'ProductCode', '{7262AC98-EEBD-4364-8CE3-D654F6A425B9}'" ); ok( r == ERROR_SUCCESS, "cannot add property entry: %d\n", r ); r = add_property_entry( hdb, "'ProductLanguage', '1033'" );

1 0

Peter Rosin : winepath: Adhere to the requested separator on failure.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: ed62749cc85b3bdc02a4e9d8aec98355dcfbfd07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ed62749cc85b3bdc02a4e9d8a… Author: Peter Rosin <peda(a)lysator.liu.se> Date: Fri Aug 27 07:54:52 2010 +0200 winepath: Adhere to the requested separator on failure. --- programs/winepath/winepath.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/programs/winepath/winepath.c b/programs/winepath/winepath.c index c4a50fb..6e5f58d 100644 --- a/programs/winepath/winepath.c +++ b/programs/winepath/winepath.c @@ -237,7 +237,7 @@ int wmain(int argc, WCHAR *argv[]) /* This is a complete path conversion failure. * It would typically happen if ntpath == "". */ - printf("\n"); + printf("%c", separator); break; } c=slash+1; @@ -249,7 +249,7 @@ int wmain(int argc, WCHAR *argv[]) /* If this is not a valid NT path to start with, * then obviously we cannot convert it. */ - printf("\n"); + printf("%c", separator); break; } if (tail) @@ -274,7 +274,7 @@ int wmain(int argc, WCHAR *argv[]) printf("%s%c", path, separator); HeapFree( GetProcessHeap(), 0, windows_name ); } - else printf( "\n" ); + else printf("%c", separator); HeapFree( GetProcessHeap(), 0, unix_name ); } }

1 0

Piotr Caban : msvcr90: Added _wstat64i32 implementation.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: 25efb008579679e8236eba7e2dc18a8208cf9047 URL: http://source.winehq.org/git/wine.git/?a=commit;h=25efb008579679e8236eba7e2… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Aug 27 01:46:45 2010 +0200 msvcr90: Added _wstat64i32 implementation. --- dlls/msvcr100/msvcr100.spec | 2 +- dlls/msvcr80/msvcr80.spec | 2 +- dlls/msvcr90/msvcr90.c | 14 ++++++++++++++ dlls/msvcr90/msvcr90.spec | 2 +- 4 files changed, 17 insertions(+), 3 deletions(-) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index a8cbbab..5c68805 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -1396,7 +1396,7 @@ @ stub _wstat32 @ stub _wstat32i64 @ cdecl _wstat64(wstr ptr) msvcrt._wstat64 -@ stub _wstat64i32 +@ cdecl _wstat64i32(wstr ptr) msvcr90._wstat64i32 @ cdecl _wstrdate(ptr) msvcrt._wstrdate @ cdecl _wstrdate_s(ptr long) msvcrt._wstrdate_s @ cdecl _wstrtime(ptr) msvcrt._wstrtime diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index 0a54876..701e28a 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -1252,7 +1252,7 @@ @ stub _wstat32 @ stub _wstat32i64 @ cdecl _wstat64(wstr ptr) msvcrt._wstat64 -@ stub _wstat64i32 +@ cdecl _wstat64i32(wstr ptr) msvcr90._wstat64i32 @ cdecl _wstrdate(ptr) msvcrt._wstrdate @ cdecl _wstrdate_s(ptr long) msvcrt._wstrdate_s @ cdecl _wstrtime(ptr) msvcrt._wstrtime diff --git a/dlls/msvcr90/msvcr90.c b/dlls/msvcr90/msvcr90.c index 0d7d101..2fff15a 100644 --- a/dlls/msvcr90/msvcr90.c +++ b/dlls/msvcr90/msvcr90.c @@ -204,3 +204,17 @@ int CDECL _stat64i32(const char* path, struct _stat64i32 * buf) msvcrt_stat64_to_stat64i32(&buf64, buf); return ret; } + +/********************************************************************* + * _wstat64i32 (MSVCRT.@) + */ +int CDECL _wstat64i32(const wchar_t *path, struct _stat64i32 *buf) +{ + int ret; + struct _stat64 buf64; + + ret = _wstat64(path, &buf64); + if (!ret) + msvcrt_stat64_to_stat64i32(&buf64, buf); + return ret; +} diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index ffdd84e..965b9c3 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -1236,7 +1236,7 @@ @ stub _wstat32 @ stub _wstat32i64 @ cdecl _wstat64(wstr ptr) msvcrt._wstat64 -@ stub _wstat64i32 +@ cdecl _wstat64i32(wstr ptr) @ cdecl _wstrdate(ptr) msvcrt._wstrdate @ cdecl _wstrdate_s(ptr long) msvcrt._wstrdate_s @ cdecl _wstrtime(ptr) msvcrt._wstrtime

1 0

Piotr Caban : msvcrt: Added _gcvt_s implementation.
by Alexandre Julliard 27 Aug '10

27 Aug '10

Module: wine Branch: master Commit: 3a25d7d9aad35a28373cad6740741cdd777f51ef URL: http://source.winehq.org/git/wine.git/?a=commit;h=3a25d7d9aad35a28373cad674… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Aug 27 01:46:35 2010 +0200 msvcrt: Added _gcvt_s implementation. --- dlls/msvcr100/msvcr100.spec | 2 +- dlls/msvcr80/msvcr80.spec | 2 +- dlls/msvcr90/msvcr90.spec | 2 +- dlls/msvcrt/math.c | 45 +++++++++++++++++++++++++++++++++++++++-- dlls/msvcrt/msvcrt.h | 2 + dlls/msvcrt/msvcrt.spec | 2 +- dlls/msvcrt/tests/string.c | 46 +++++++++++++++++++++++++++++++++++++++--- 7 files changed, 90 insertions(+), 11 deletions(-) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index a1c5a37..a8cbbab 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -674,7 +674,7 @@ @ varargs _fwscanf_l(ptr wstr ptr) msvcrt._fwscanf_l @ varargs _fwscanf_s_l(ptr wstr ptr) msvcrt._fwscanf_s_l @ cdecl _gcvt(double long str) msvcrt._gcvt -@ stub _gcvt_s +@ cdecl _gcvt_s(ptr long double long) msvcrt._gcvt_s @ stub _get_current_locale @ stub _get_daylight @ stub _get_doserrno diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index a161b64..0a54876 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -515,7 +515,7 @@ @ varargs _fwscanf_l(ptr wstr ptr) msvcrt._fwscanf_l @ varargs _fwscanf_s_l(ptr wstr ptr) msvcrt._fwscanf_s_l @ cdecl _gcvt(double long str) msvcrt._gcvt -@ stub _gcvt_s +@ cdecl _gcvt_s(ptr long double long) msvcrt._gcvt_s @ stub _get_amblksiz @ stub _get_current_locale @ stub _get_daylight diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index d369331..ffdd84e 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -507,7 +507,7 @@ @ varargs _fwscanf_l(ptr wstr ptr) msvcrt._fwscanf_l @ varargs _fwscanf_s_l(ptr wstr ptr) msvcrt._fwscanf_s_l @ cdecl _gcvt(double long str) msvcrt._gcvt -@ stub _gcvt_s +@ cdecl _gcvt_s(ptr long double long) msvcrt._gcvt_s @ stub _get_amblksiz @ stub _get_current_locale @ stub _get_daylight diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index 5a31ae1..4a3f65e 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -1196,15 +1196,54 @@ char * CDECL _fcvt( double number, int ndigits, int *decpt, int *sign ) /*********************************************************************** * _gcvt (MSVCRT.@) - * - * FIXME: uses both E and F. */ char * CDECL _gcvt( double number, int ndigit, char *buff ) { - sprintf(buff, "%.*E", ndigit, number); + if(!buff) { + *MSVCRT__errno() = MSVCRT_EINVAL; + return NULL; + } + + if(ndigit < 0) { + *MSVCRT__errno() = MSVCRT_ERANGE; + return NULL; + } + + MSVCRT_sprintf(buff, "%.*g", ndigit, number); return buff; } +/*********************************************************************** + * _gcvt_s (MSVCRT.@) + */ +int CDECL _gcvt_s(char *buff, MSVCRT_size_t size, double number, int digits) +{ + int len; + + if(!buff) { + *MSVCRT__errno() = MSVCRT_EINVAL; + return MSVCRT_EINVAL; + } + + if( digits<0 || digits>=size) { + if(size) + buff[0] = '\0'; + + *MSVCRT__errno() = MSVCRT_ERANGE; + return MSVCRT_ERANGE; + } + + len = MSVCRT__scprintf("%.*g", digits, number); + if(len > size) { + buff[0] = '\0'; + *MSVCRT__errno() = MSVCRT_ERANGE; + return MSVCRT_ERANGE; + } + + MSVCRT_sprintf(buff, "%.*g", digits, number); + return 0; +} + #include <stdlib.h> /* div_t, ldiv_t */ /********************************************************************* diff --git a/dlls/msvcrt/msvcrt.h b/dlls/msvcrt/msvcrt.h index 363223d..86df133 100644 --- a/dlls/msvcrt/msvcrt.h +++ b/dlls/msvcrt/msvcrt.h @@ -743,6 +743,8 @@ MSVCRT_FILE* __cdecl MSVCRT__wfdopen(int, const MSVCRT_wchar_t *); int __cdecl MSVCRT_vsnprintf(char *str, MSVCRT_size_t len, const char *format, __ms_va_list valist); int __cdecl MSVCRT_vsnwprintf(MSVCRT_wchar_t *str, MSVCRT_size_t len, const MSVCRT_wchar_t *format, __ms_va_list valist ); +int __cdecl MSVCRT_sprintf(char*,const char*,...); +int __cdecl MSVCRT__scprintf(const char*,...); int __cdecl MSVCRT_raise(int sig); typedef struct MSVCRT_tagLC_ID { diff --git a/dlls/msvcrt/msvcrt.spec b/dlls/msvcrt/msvcrt.spec index 6d3e1dd..a7371a8 100644 --- a/dlls/msvcrt/msvcrt.spec +++ b/dlls/msvcrt/msvcrt.spec @@ -461,7 +461,7 @@ @ varargs _fwscanf_l(ptr wstr ptr) MSVCRT__fwscanf_l @ varargs _fwscanf_s_l(ptr wstr ptr) MSVCRT__fwscanf_s_l @ cdecl _gcvt(double long str) -# stub _gcvt_s +@ cdecl _gcvt_s(ptr long double long) # stub _get_doserrno # stub _get_environ # stub _get_errno diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c index e2324b0..94c2fa3 100644 --- a/dlls/msvcrt/tests/string.c +++ b/dlls/msvcrt/tests/string.c @@ -58,6 +58,7 @@ static __int64 (__cdecl *p_strtoi64)(const char *, char **, int); static unsigned __int64 (__cdecl *p_strtoui64)(const char *, char **, int); static int (__cdecl *pwcstombs_s)(size_t*,char*,size_t,const wchar_t*,size_t); static int (__cdecl *pmbstowcs_s)(size_t*,wchar_t*,size_t,const char*,size_t); +static errno_t (__cdecl *p_gcvt_s)(char*,size_t,double,int); static int *p__mb_cur_max; static unsigned char *p_mbctype; @@ -1241,6 +1242,41 @@ static void test_mbstowcs(void) ok(!memcmp(mOut, mHiragana, sizeof(mHiragana)), "mOut = %s\n", mOut); } +static void test_gcvt(void) +{ + char buf[1024], *res; + errno_t err; + + if(!p_gcvt_s) { + win_skip("Skipping _gcvt tests\n"); + return; + } + + errno = 0; + res = _gcvt(1.2, -1, buf); + ok(res == NULL, "res != NULL\n"); + ok(errno == ERANGE, "errno = %d\n", errno); + + errno = 0; + res = _gcvt(1.2, 5, NULL); + ok(res == NULL, "res != NULL\n"); + ok(errno == EINVAL, "errno = %d\n", errno); + + res = gcvt(1.2, 5, buf); + ok(res == buf, "res != buf\n"); + ok(!strcmp(buf, "1.2"), "buf = %s\n", buf); + + buf[0] = 'x'; + err = p_gcvt_s(buf, 5, 1.2, 10); + ok(err == ERANGE, "err = %d\n", err); + ok(buf[0] == '\0', "buf[0] = %c\n", buf[0]); + + buf[0] = 'x'; + err = p_gcvt_s(buf, 4, 123456, 2); + ok(err == ERANGE, "err = %d\n", err); + ok(buf[0] == '\0', "buf[0] = %c\n", buf[0]); +} + START_TEST(string) { char mem[100]; @@ -1261,10 +1297,11 @@ START_TEST(string) p_wcscpy_s = (void *)GetProcAddress( hMsvcrt,"wcscpy_s" ); p_wcsupr_s = (void *)GetProcAddress( hMsvcrt,"_wcsupr_s" ); p_strnlen = (void *)GetProcAddress( hMsvcrt,"strnlen" ); - p_strtoi64 = (void *) GetProcAddress(hMsvcrt, "_strtoi64"); - p_strtoui64 = (void *) GetProcAddress(hMsvcrt, "_strtoui64"); - pmbstowcs_s = (void *) GetProcAddress(hMsvcrt, "mbstowcs_s"); - pwcstombs_s = (void *) GetProcAddress(hMsvcrt, "wcstombs_s"); + p_strtoi64 = (void *)GetProcAddress(hMsvcrt, "_strtoi64"); + p_strtoui64 = (void *)GetProcAddress(hMsvcrt, "_strtoui64"); + pmbstowcs_s = (void *)GetProcAddress(hMsvcrt, "mbstowcs_s"); + pwcstombs_s = (void *)GetProcAddress(hMsvcrt, "wcstombs_s"); + p_gcvt_s = (void *)GetProcAddress(hMsvcrt, "_gcvt_s"); /* MSVCRT memcpy behaves like memmove for overlapping moves, MFC42 CString::Insert seems to rely on that behaviour */ @@ -1298,4 +1335,5 @@ START_TEST(string) test__strtoi64(); test__strtod(); test_mbstowcs(); + test_gcvt(); }

1 0

← Newer
1
...
10
11
12
13
14
15
16
...
89
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 August 2010