setupapi: Fix the StringTableLookUpStringEx() prototype.
Francois Gouget
fgouget at free.fr
Thu Jan 8 07:16:27 CST 2009
---
Fixing the prototype as has been discussed on wine-devel.
dlls/setupapi/setupapi.spec | 2 +-
dlls/setupapi/stringtable.c | 14 +++++++-------
dlls/setupapi/tests/stringtable.c | 24 ++++++++++++------------
3 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/dlls/setupapi/setupapi.spec b/dlls/setupapi/setupapi.spec
index 03435bc..6c86043 100644
--- a/dlls/setupapi/setupapi.spec
+++ b/dlls/setupapi/setupapi.spec
@@ -533,7 +533,7 @@
@ stdcall StringTableInitialize()
@ stdcall StringTableInitializeEx(long long)
@ stdcall StringTableLookUpString(ptr wstr long)
-@ stdcall StringTableLookUpStringEx(ptr wstr long ptr ptr)
+@ stdcall StringTableLookUpStringEx(ptr wstr long ptr long)
@ stdcall StringTableSetExtraData(ptr long ptr long)
@ stdcall StringTableStringFromId(ptr long)
@ stdcall StringTableStringFromIdEx(ptr long ptr ptr)
diff --git a/dlls/setupapi/stringtable.c b/dlls/setupapi/stringtable.c
index fe319b8..0dbe6e8 100644
--- a/dlls/setupapi/stringtable.c
+++ b/dlls/setupapi/stringtable.c
@@ -457,7 +457,7 @@ StringTableGetExtraData(HSTRING_TABLE hStringTable,
* dwFlags [I] Flags
* 1: case sensitive compare
* lpExtraData [O] Pointer to the buffer that receives the extra data
- * lpReserved [I/O] Unused
+ * dwReserved [I/O] Unused
*
* RETURNS
* Success: String ID
@@ -468,13 +468,13 @@ StringTableLookUpStringEx(HSTRING_TABLE hStringTable,
LPWSTR lpString,
DWORD dwFlags,
LPVOID lpExtraData,
- LPDWORD lpReserved)
+ DWORD dwReserved)
{
PSTRING_TABLE pStringTable;
DWORD i;
- TRACE("%p %s %x %p, %p\n", hStringTable, debugstr_w(lpString), dwFlags,
- lpExtraData, lpReserved);
+ TRACE("%p %s %x %p, %x\n", hStringTable, debugstr_w(lpString), dwFlags,
+ lpExtraData, dwReserved);
pStringTable = (PSTRING_TABLE)hStringTable;
if (pStringTable == NULL)
@@ -493,7 +493,7 @@ StringTableLookUpStringEx(HSTRING_TABLE hStringTable,
if (!lstrcmpW(pStringTable->pSlots[i].pString, lpString))
{
if (lpExtraData)
- memcpy(lpExtraData, pStringTable->pSlots[i].pData, (DWORD)lpReserved);
+ memcpy(lpExtraData, pStringTable->pSlots[i].pData, dwReserved);
return i + 1;
}
}
@@ -502,7 +502,7 @@ StringTableLookUpStringEx(HSTRING_TABLE hStringTable,
if (!lstrcmpiW(pStringTable->pSlots[i].pString, lpString))
{
if (lpExtraData)
- memcpy(lpExtraData, pStringTable->pSlots[i].pData, (DWORD)lpReserved);
+ memcpy(lpExtraData, pStringTable->pSlots[i].pData, dwReserved);
return i + 1;
}
}
@@ -532,7 +532,7 @@ StringTableLookUpString(HSTRING_TABLE hStringTable,
LPWSTR lpString,
DWORD dwFlags)
{
- return StringTableLookUpStringEx(hStringTable, lpString, dwFlags, NULL, NULL);
+ return StringTableLookUpStringEx(hStringTable, lpString, dwFlags, NULL, 0);
}
diff --git a/dlls/setupapi/tests/stringtable.c b/dlls/setupapi/tests/stringtable.c
index dfceec8..40e4a54 100644
--- a/dlls/setupapi/tests/stringtable.c
+++ b/dlls/setupapi/tests/stringtable.c
@@ -46,7 +46,7 @@ static HSTRING_TABLE (WINAPI *pStringTableDuplicate)(HSTRING_TABLE hStringTable)
static HSTRING_TABLE (WINAPI *pStringTableInitialize)(VOID);
static HSTRING_TABLE (WINAPI *pStringTableInitializeEx)(DWORD, DWORD);
static DWORD (WINAPI *pStringTableLookUpString)(HSTRING_TABLE, LPWSTR, DWORD);
-static DWORD (WINAPI *pStringTableLookUpStringEx)(HSTRING_TABLE, LPWSTR, DWORD, LPVOID, LPDWORD);
+static DWORD (WINAPI *pStringTableLookUpStringEx)(HSTRING_TABLE, LPWSTR, DWORD, LPVOID, DWORD);
static LPWSTR (WINAPI *pStringTableStringFromId)(HSTRING_TABLE, DWORD);
#if 0
static BOOL (WINAPI *pStringTableStringFromIdEx)(HSTRING_TABLE, DWORD, LPWSTR, LPDWORD);
@@ -247,33 +247,33 @@ static void test_StringTableLookUpStringEx(void)
ok(table2 != NULL, "Failed to duplicate String Table\n");
/* case insensitive */
- retval = pStringTableLookUpStringEx(table, string, 0, NULL, NULL);
+ retval = pStringTableLookUpStringEx(table, string, 0, NULL, 0);
ok(retval != ~0u, "Failed find string in String Table 1\n");
ok(retval == hstring,
"Lookup for string (%x) does not match previous handle (%x) in String Table 1\n",
retval, hstring);
- retval = pStringTableLookUpStringEx(table2, string, 0, NULL, NULL);
+ retval = pStringTableLookUpStringEx(table2, string, 0, NULL, 0);
ok(retval != ~0u, "Failed find string in String Table 2\n");
- retval = pStringTableLookUpStringEx(table, String, 0, NULL, NULL);
+ retval = pStringTableLookUpStringEx(table, String, 0, NULL, 0);
ok(retval != ~0u, "Failed find String in String Table 1\n");
- retval = pStringTableLookUpStringEx(table2, String, 0, NULL, NULL);
+ retval = pStringTableLookUpStringEx(table2, String, 0, NULL, 0);
ok(retval != ~0u, "Failed find String in String Table 2\n");
- retval=pStringTableLookUpStringEx(table, foo, 0, NULL, NULL);
+ retval=pStringTableLookUpStringEx(table, foo, 0, NULL, 0);
ok(retval != ~0u, "Failed find foo in String Table 1\n");
ok(retval == hfoo,
"Lookup for foo (%x) does not match previous handle (%x) in String Table 1\n",
retval, hfoo);
- retval = pStringTableLookUpStringEx(table2, foo, 0, NULL, NULL);
+ retval = pStringTableLookUpStringEx(table2, foo, 0, NULL, 0);
ok(retval != ~0u, "Failed find foo in String Table 2\n");
/* case sensitive */
- retval = pStringTableLookUpStringEx(table, string,ST_CASE_SENSITIVE_COMPARE, NULL, NULL);
- retval2 = pStringTableLookUpStringEx(table, String, ST_CASE_SENSITIVE_COMPARE, NULL, NULL);
+ retval = pStringTableLookUpStringEx(table, string,ST_CASE_SENSITIVE_COMPARE, NULL, 0);
+ retval2 = pStringTableLookUpStringEx(table, String, ST_CASE_SENSITIVE_COMPARE, NULL, 0);
ok(retval != retval2, "Lookup of string equals String in Table 1\n");
ok(retval == hString,
"Lookup for String (%x) does not match previous handle (%x) in String Table 1\n",
@@ -289,17 +289,17 @@ static void test_StringTableLookUpStringEx(void)
ok(retval != ~0u, "failed to add 'UILEVEL' to string table\n");
memset(buffer, 0x55, sizeof(buffer));
- retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, NULL);
+ retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, 0);
ok(retval != ~0u, "failed find 'UILEVEL' in string table\n");
ok(memcmp(buffer, &data, 4), "unexpected data\n");
memset(buffer, 0x55, sizeof(buffer));
- retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, (LPDWORD)2);
+ retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, 2);
ok(retval != ~0u, "failed find 'UILEVEL' in string table\n");
ok(!memcmp(buffer, &data, 2), "unexpected data\n");
memset(buffer, 0x55, sizeof(buffer));
- retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, (LPDWORD)sizeof(buffer));
+ retval = pStringTableLookUpStringEx(table, uilevel, ST_CASE_SENSITIVE_COMPARE, buffer, sizeof(buffer));
ok(retval != ~0u, "failed find 'UILEVEL' in string table\n");
ok(!memcmp(buffer, &data, 4), "unexpected data\n");
--
1.5.6.5
More information about the wine-patches
mailing list