November 2009 - Wine-commits

Rob Shearman : rpcrt4: Fix the pointer saved during full pointer unmarshalling.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 37793cd5507df9a42e93f79ba6894fc0be0be431 URL: http://source.winehq.org/git/wine.git/?a=commit;h=37793cd5507df9a42e93f79ba… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Wed Nov 18 00:25:26 2009 +0000 rpcrt4: Fix the pointer saved during full pointer unmarshalling. The pointer address may set by the type unmarshalling function and therefore will not be base_ptr_val. This value is the value at *pPointer and is what needs to be retrieved during subsequent unmarshalling of the same function. --- dlls/rpcrt4/ndr_marshall.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/rpcrt4/ndr_marshall.c b/dlls/rpcrt4/ndr_marshall.c index d1b57fd..8db45b8 100644 --- a/dlls/rpcrt4/ndr_marshall.c +++ b/dlls/rpcrt4/ndr_marshall.c @@ -935,7 +935,7 @@ static void PointerUnmarshall(PMIDL_STUB_MESSAGE pStubMsg, if (type == RPC_FC_FP) NdrFullPointerInsertRefId(pStubMsg->FullPtrXlatTables, pointer_id, - base_ptr_val); + *pPointer); } TRACE("pointer=%p\n", *pPointer);

1 0

Rob Shearman : rpcrt4: Add tests for full pointers.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: af271255f5ef44565d40d619d2ddd18b2e5d25c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=af271255f5ef44565d40d619d… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Wed Nov 18 00:25:53 2009 +0000 rpcrt4: Add tests for full pointers. --- dlls/rpcrt4/tests/server.c | 19 +++++++++++++++++++ dlls/rpcrt4/tests/server.idl | 4 ++++ 2 files changed, 23 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/tests/server.c b/dlls/rpcrt4/tests/server.c index 1205ea4..6453f53 100644 --- a/dlls/rpcrt4/tests/server.c +++ b/dlls/rpcrt4/tests/server.c @@ -717,6 +717,21 @@ s_get_numbers_struct(numbers_struct_t **ns) } void +s_full_pointer_test(int *a, int *b) +{ + ok(*a == 42, "Expected *a to be 42 instead of %d\n", *a); + ok(*b == 42, "Expected *b to be 42 instead of %d\n", *a); + ok(a == b, "Expected a (%p) to point to the same memory as b (%p)\n", a, b); +} + +void +s_full_pointer_null_test(int *a, int *b) +{ + ok(*a == 42, "Expected *a to be 42 instead of %d\n", *a); + ok(b == NULL, "Expected b to be NULL instead of %p\n", b); +} + +void s_stop(void) { ok(RPC_S_OK == RpcMgmtStopServerListening(NULL), "RpcMgmtStopServerListening\n"); @@ -1077,6 +1092,7 @@ pointer_tests(void) void *buffer; int *pa2; s123_t *s123; + int val = 42; ok(test_list_length(list) == 3, "RPC test_list_length\n"); ok(square_puint(p1) == 121, "RPC square_puint\n"); @@ -1137,6 +1153,9 @@ pointer_tests(void) s123 = get_s123(); ok(s123->f1 == 1 && s123->f2 == 2 && s123->f3 == 3, "RPC get_s123\n"); MIDL_user_free(s123); + + full_pointer_test(&val, &val); + full_pointer_null_test(&val, NULL); } static int diff --git a/dlls/rpcrt4/tests/server.idl b/dlls/rpcrt4/tests/server.idl index d8e79bd..ee03c0b 100644 --- a/dlls/rpcrt4/tests/server.idl +++ b/dlls/rpcrt4/tests/server.idl @@ -357,5 +357,9 @@ cpp_quote("#endif") void get_ranged_enum([out] renum_t *re); void context_handle_test(void); + + void full_pointer_test([in, ptr] int *a, [in, ptr] int *b); + void full_pointer_null_test([in, ptr] int *a, [in, ptr] int *b); + void stop(void); }

1 0

Alexandre Julliard : include: Add some more definitions for FILE_INFORMATION_CLASS.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 44f520a5b6f2bd6ec6da8793177711e57bda9c72 URL: http://source.winehq.org/git/wine.git/?a=commit;h=44f520a5b6f2bd6ec6da87931… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 18 15:15:32 2009 +0100 include: Add some more definitions for FILE_INFORMATION_CLASS. --- dlls/ntdll/file.c | 20 +++++++++++++++++++- include/winternl.h | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 67 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index b62c7c2..984f466 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -1578,7 +1578,25 @@ NTSTATUS WINAPI NtQueryInformationFile( HANDLE hFile, PIO_STATUS_BLOCK io, 0, /* FileReparsePointInformation */ 0, /* FileNetworkOpenInformation */ 0, /* FileAttributeTagInformation */ - 0 /* FileTrackingInformation */ + 0, /* FileTrackingInformation */ + 0, /* FileIdBothDirectoryInformation */ + 0, /* FileIdFullDirectoryInformation */ + 0, /* FileValidDataLengthInformation */ + 0, /* FileShortNameInformation */ + 0, + 0, + 0, + 0, /* FileSfioReserveInformation */ + 0, /* FileSfioVolumeInformation */ + 0, /* FileHardLinkInformation */ + 0, + 0, /* FileNormalizedNameInformation */ + 0, + 0, /* FileIdGlobalTxDirectoryInformation */ + 0, + 0, + 0, + 0 /* FileStandardLinkInformation */ }; struct stat st; diff --git a/include/winternl.h b/include/winternl.h index 633239f..fe77359 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -407,6 +407,20 @@ typedef enum _FILE_INFORMATION_CLASS { FileNetworkOpenInformation, FileAttributeTagInformation, FileTrackingInformation, + FileIdBothDirectoryInformation, + FileIdFullDirectoryInformation, + FileValidDataLengthInformation, + FileShortNameInformation = 40, + /* 41, 42, 43 undocumented */ + FileSfioReserveInformation = 44, + FileSfioVolumeInformation = 45, + FileHardLinkInformation = 46, + /* 47 undocumented */ + FileNormalizedNameInformation = 48, + /* 49 undocumented */ + FileIdGlobalTxDirectoryInformation = 50, + /* 51, 52, 53 undocumented */ + FileStandardLinkInformation = 54, FileMaximumInformation } FILE_INFORMATION_CLASS, *PFILE_INFORMATION_CLASS; @@ -440,6 +454,22 @@ typedef struct _FILE_FULL_DIRECTORY_INFORMATION { } FILE_FULL_DIRECTORY_INFORMATION, *PFILE_FULL_DIRECTORY_INFORMATION, FILE_FULL_DIR_INFORMATION, *PFILE_FULL_DIR_INFORMATION; +typedef struct _FILE_ID_FULL_DIRECTORY_INFORMATION { + ULONG NextEntryOffset; + ULONG FileIndex; + LARGE_INTEGER CreationTime; + LARGE_INTEGER LastAccessTime; + LARGE_INTEGER LastWriteTime; + LARGE_INTEGER ChangeTime; + LARGE_INTEGER EndOfFile; + LARGE_INTEGER AllocationSize; + ULONG FileAttributes; + ULONG FileNameLength; + ULONG EaSize; + LARGE_INTEGER FileId; + WCHAR FileName[ANYSIZE_ARRAY]; +} FILE_ID_FULL_DIRECTORY_INFORMATION, *PFILE_ID_FULL_DIRECTORY_INFORMATION; + typedef struct _FILE_BOTH_DIRECTORY_INFORMATION { ULONG NextEntryOffset; ULONG FileIndex; @@ -458,6 +488,24 @@ typedef struct _FILE_BOTH_DIRECTORY_INFORMATION { } FILE_BOTH_DIRECTORY_INFORMATION, *PFILE_BOTH_DIRECTORY_INFORMATION, FILE_BOTH_DIR_INFORMATION, *PFILE_BOTH_DIR_INFORMATION; +typedef struct _FILE_ID_BOTH_DIRECTORY_INFORMATION { + ULONG NextEntryOffset; + ULONG FileIndex; + LARGE_INTEGER CreationTime; + LARGE_INTEGER LastAccessTime; + LARGE_INTEGER LastWriteTime; + LARGE_INTEGER ChangeTime; + LARGE_INTEGER EndOfFile; + LARGE_INTEGER AllocationSize; + ULONG FileAttributes; + ULONG FileNameLength; + ULONG EaSize; + CHAR ShortNameLength; + WCHAR ShortName[12]; + LARGE_INTEGER FileId; + WCHAR FileName[ANYSIZE_ARRAY]; +} FILE_ID_BOTH_DIRECTORY_INFORMATION, *PFILE_ID_BOTH_DIRECTORY_INFORMATION; + typedef struct _FILE_BASIC_INFORMATION { LARGE_INTEGER CreationTime; LARGE_INTEGER LastAccessTime;

1 0

Francois Gouget : mshtml/tests: Add a trailing '\n' to ok() calls.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 1e4412d7319e6673c2251015d761fe8fca83eb1c URL: http://source.winehq.org/git/wine.git/?a=commit;h=1e4412d7319e6673c2251015d… Author: Francois Gouget <fgouget(a)free.fr> Date: Wed Nov 18 11:11:47 2009 +0100 mshtml/tests: Add a trailing '\n' to ok() calls. --- dlls/mshtml/tests/dom.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 00c67a8..5484134 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -5707,7 +5707,7 @@ static void test_frameset(IHTMLDocument2 *doc) hres = IHTMLFramesCollection2_item(frames, &index_var, &result_var); ok(hres == S_OK, "IHTMLFramesCollection2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { - ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x", V_VT(&result_var)); + ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); test_frame((IDispatch*)V_DISPATCH(&result_var), "fr1"); } VariantClear(&result_var); @@ -5717,7 +5717,7 @@ static void test_frameset(IHTMLDocument2 *doc) hres = IHTMLFramesCollection2_item(frames, &index_var, &result_var); ok(hres == S_OK, "IHTMLFramesCollection2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { - ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x", V_VT(&result_var)); + ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); test_frame((IDispatch*)V_DISPATCH(&result_var), "fr2"); } VariantClear(&result_var); @@ -5735,7 +5735,7 @@ static void test_frameset(IHTMLDocument2 *doc) hres = IHTMLFramesCollection2_item(frames, &index_var, &result_var); ok(hres == S_OK, "IHTMLFramesCollection2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { - ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x", V_VT(&result_var)); + ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); test_frame((IDispatch*)V_DISPATCH(&result_var), "fr1"); } VariantClear(&result_var); @@ -5764,7 +5764,7 @@ static void test_frameset(IHTMLDocument2 *doc) hres = IHTMLWindow2_item(window, &index_var, &result_var); ok(hres == S_OK, "IHTMLWindow2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { - ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x", V_VT(&result_var)); + ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); test_frame((IDispatch*)V_DISPATCH(&result_var), "fr1"); } VariantClear(&result_var); @@ -5774,7 +5774,7 @@ static void test_frameset(IHTMLDocument2 *doc) hres = IHTMLWindow2_item(window, &index_var, &result_var); ok(hres == S_OK, "IHTMLWindow2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { - ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x", V_VT(&result_var)); + ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); test_frame((IDispatch*)V_DISPATCH(&result_var), "fr2"); } VariantClear(&result_var); @@ -5792,7 +5792,7 @@ static void test_frameset(IHTMLDocument2 *doc) hres = IHTMLWindow2_item(window, &index_var, &result_var); ok(hres == S_OK, "IHTMLWindow2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { - ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x", V_VT(&result_var)); + ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); test_frame((IDispatch*)V_DISPATCH(&result_var), "fr2"); } VariantClear(&result_var);

1 0

Francois Gouget : cmd: Remove spaces before '\n's.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 085c39342b83ba342180c425ce69edf735e839ba URL: http://source.winehq.org/git/wine.git/?a=commit;h=085c39342b83ba342180c425c… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Nov 18 11:14:25 2009 +0100 cmd: Remove spaces before '\n's. --- programs/cmd/It.rc | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/cmd/It.rc b/programs/cmd/It.rc index d334ed1..8eb2a33 100644 --- a/programs/cmd/It.rc +++ b/programs/cmd/It.rc @@ -75,7 +75,7 @@ esiste nel cmd di wine.\n" file batch.\n\ \n\ L'etichetta che è l'obiettivo di un GOTO può essere lunga fino a 255 caratteri\n\ -ma non può includere spazi (questo è differente da altri sistemi \n\ +ma non può includere spazi (questo è differente da altri sistemi\n\ operativi). Se esistono due o più etichette identiche esistono in un file batch, la\n\ prima sarà sempre eseguita. Tentare un GOTO verso un'etichetta\n\ inesistente termina l'esecuzione del file batch.\n\

1 0

Austin Lund : advapi32/tests: Skip tests for ACL file info if ACL info not stored.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 5c45c35e4e21b577a22de5ae2ae4c4f1c25d25ea URL: http://source.winehq.org/git/wine.git/?a=commit;h=5c45c35e4e21b577a22de5ae2… Author: Austin Lund <austin.lund(a)gmail.com> Date: Wed Nov 18 20:52:59 2009 +1000 advapi32/tests: Skip tests for ACL file info if ACL info not stored. --- dlls/advapi32/tests/security.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c index a327c53..470164a 100644 --- a/dlls/advapi32/tests/security.c +++ b/dlls/advapi32/tests/security.c @@ -3156,8 +3156,10 @@ static void test_GetSecurityInfo(void) ok(sd != NULL, "GetSecurityInfo\n"); ok(owner != NULL, "GetSecurityInfo\n"); ok(group != NULL, "GetSecurityInfo\n"); - ok(dacl != NULL, "GetSecurityInfo\n"); - ok(IsValidAcl(dacl), "GetSecurityInfo\n"); + if (dacl != NULL) + ok(IsValidAcl(dacl), "GetSecurityInfo\n"); + else + win_skip("No ACL information returned\n"); LocalFree(sd); @@ -3176,8 +3178,10 @@ static void test_GetSecurityInfo(void) ok(ret == ERROR_SUCCESS, "GetSecurityInfo returned %d\n", ret); ok(owner != NULL, "GetSecurityInfo\n"); ok(group != NULL, "GetSecurityInfo\n"); - ok(dacl != NULL, "GetSecurityInfo\n"); - ok(IsValidAcl(dacl), "GetSecurityInfo\n"); + if (dacl != NULL) + ok(IsValidAcl(dacl), "GetSecurityInfo\n"); + else + win_skip("No ACL information returned\n"); CloseHandle(obj); }

1 0

Piotr Caban : msxml3: Implemented xmlnode_get_nodeTypedValue 'number' data type handling.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 0a6d3a70ca1e8811efbc0d41a23ff1ba90a62858 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0a6d3a70ca1e8811efbc0d41a… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Nov 18 00:51:11 2009 +0100 msxml3: Implemented xmlnode_get_nodeTypedValue 'number' data type handling. --- dlls/msxml3/node.c | 3 ++- dlls/msxml3/tests/domdoc.c | 28 ++++++++++++++++++++++++++++ 2 files changed, 30 insertions(+), 1 deletions(-) diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index 1f4dc32..56a8578 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -948,7 +948,8 @@ static HRESULT WINAPI xmlnode_get_dataType(IXMLDOMNode*, VARIANT*); inline HRESULT VARIANT_from_xmlChar(xmlChar *str, VARIANT *v, BSTR type) { - if(!type || !lstrcmpiW(type, szString)) + if(!type || !lstrcmpiW(type, szString) || + !lstrcmpiW(type, szNumber)) { V_VT(v) = VT_BSTR; V_BSTR(v) = bstr_from_xmlChar(str); diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index b946970..2f34a24 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -151,6 +151,8 @@ static const CHAR szTypeValueXML[] = "<root xmlns:dt=\"urn:schemas-microsoft-com:datatypes\">\n" " <string>Wine</string>\n" " <string2 dt:dt=\"string\">String</string2>\n" +" <number dt:dt=\"number\">12.44</number>\n" +" <number2 dt:dt=\"NUMbEr\">-3.71e3</number2>\n" "</root>"; static const CHAR szBasicTransformSSXMLPart1[] = @@ -4009,6 +4011,32 @@ static void test_NodeTypeValue(void) IXMLDOMNode_Release(pNode); } + + hr = IXMLDOMDocument2_selectSingleNode(doc, _bstr_("root/number"), &pNode); + ok(hr == S_OK, "ret %08x\n", hr ); + if(hr == S_OK) + { + hr = IXMLDOMNode_get_nodeTypedValue(pNode, &v); + ok(hr == S_OK, "ret %08x\n", hr ); + ok(V_VT(&v) == VT_BSTR, "incorrect type\n"); + ok(!lstrcmpW( V_BSTR(&v), _bstr_("12.44") ), "incorrect value\n"); + VariantClear( &v ); + + IXMLDOMNode_Release(pNode); + } + + hr = IXMLDOMDocument2_selectSingleNode(doc, _bstr_("root/number2"), &pNode); + ok(hr == S_OK, "ret %08x\n", hr ); + if(hr == S_OK) + { + hr = IXMLDOMNode_get_nodeTypedValue(pNode, &v); + ok(hr == S_OK, "ret %08x\n", hr ); + ok(V_VT(&v) == VT_BSTR, "incorrect type\n"); + ok(!lstrcmpW( V_BSTR(&v), _bstr_("-3.71e3") ), "incorrect value\n"); + VariantClear( &v ); + + IXMLDOMNode_Release(pNode); + } } IXMLDOMDocument2_Release(doc);

1 0

Piotr Caban : msxml3: Implemented xmlnode_get_nodeTypedValue 'int' data type handling.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 44980403da3e86b01343defb18e30743bdd4ca58 URL: http://source.winehq.org/git/wine.git/?a=commit;h=44980403da3e86b01343defb1… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Nov 18 00:51:17 2009 +0100 msxml3: Implemented xmlnode_get_nodeTypedValue 'int' data type handling. --- dlls/msxml3/node.c | 22 ++++++++++++++++++---- dlls/msxml3/tests/domdoc.c | 14 ++++++++++++++ 2 files changed, 32 insertions(+), 4 deletions(-) diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index 56a8578..b1f8a9d 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -959,12 +959,26 @@ inline HRESULT VARIANT_from_xmlChar(xmlChar *str, VARIANT *v, BSTR type) } else { - FIXME("Type handling not yet implemented\n"); - V_VT(v) = VT_BSTR; - V_BSTR(v) = bstr_from_xmlChar(str); + VARIANT src; + HRESULT hres; - if(!V_BSTR(v)) + if(!lstrcmpiW(type, szInt)) + V_VT(v) = VT_I4; + else + { + FIXME("Type handling not yet implemented\n"); + V_VT(v) = VT_BSTR; + } + + V_VT(&src) = VT_BSTR; + V_BSTR(&src) = bstr_from_xmlChar(str); + + if(!V_BSTR(&src)) return E_OUTOFMEMORY; + + hres = VariantChangeType(v, &src, 0, V_VT(v)); + VariantClear(&src); + return hres; } return S_OK; diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index 2f34a24..a0dbe02 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -153,6 +153,7 @@ static const CHAR szTypeValueXML[] = " <string2 dt:dt=\"string\">String</string2>\n" " <number dt:dt=\"number\">12.44</number>\n" " <number2 dt:dt=\"NUMbEr\">-3.71e3</number2>\n" +" <int dt:dt=\"int\">-13</int>\n" "</root>"; static const CHAR szBasicTransformSSXMLPart1[] = @@ -4037,6 +4038,19 @@ static void test_NodeTypeValue(void) IXMLDOMNode_Release(pNode); } + + hr = IXMLDOMDocument2_selectSingleNode(doc, _bstr_("root/int"), &pNode); + ok(hr == S_OK, "ret %08x\n", hr ); + if(hr == S_OK) + { + hr = IXMLDOMNode_get_nodeTypedValue(pNode, &v); + ok(hr == S_OK, "ret %08x\n", hr ); + ok(V_VT(&v) == VT_I4, "incorrect type\n"); + ok(V_I4(&v) == -13, "incorrect value\n"); + VariantClear( &v ); + + IXMLDOMNode_Release(pNode); + } } IXMLDOMDocument2_Release(doc);

1 0

Piotr Caban : msxml3: Implemented xmlnode_get_nodeTypedValue 'fixed.14.4' data type handling.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 47e037d4bb945c6d7ddcea80b1f6fd0dd0bfcf1c URL: http://source.winehq.org/git/wine.git/?a=commit;h=47e037d4bb945c6d7ddcea80b… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Nov 18 00:51:24 2009 +0100 msxml3: Implemented xmlnode_get_nodeTypedValue 'fixed.14.4' data type handling. --- dlls/msxml3/node.c | 2 ++ dlls/msxml3/tests/domdoc.c | 22 ++++++++++++++++++++++ 2 files changed, 24 insertions(+), 0 deletions(-) diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index b1f8a9d..0f515c4 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -964,6 +964,8 @@ inline HRESULT VARIANT_from_xmlChar(xmlChar *str, VARIANT *v, BSTR type) if(!lstrcmpiW(type, szInt)) V_VT(v) = VT_I4; + else if(!lstrcmpiW(type, szFixed)) + V_VT(v) = VT_CY; else { FIXME("Type handling not yet implemented\n"); diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index a0dbe02..cb8aa4e 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -154,6 +154,7 @@ static const CHAR szTypeValueXML[] = " <number dt:dt=\"number\">12.44</number>\n" " <number2 dt:dt=\"NUMbEr\">-3.71e3</number2>\n" " <int dt:dt=\"int\">-13</int>\n" +" <fixed dt:dt=\"fixed.14.4\">7322.9371</fixed>\n" "</root>"; static const CHAR szBasicTransformSSXMLPart1[] = @@ -4051,6 +4052,27 @@ static void test_NodeTypeValue(void) IXMLDOMNode_Release(pNode); } + + hr = IXMLDOMDocument2_selectSingleNode(doc, _bstr_("root/fixed"), &pNode); + ok(hr == S_OK, "ret %08x\n", hr ); + if(hr == S_OK) + { + VARIANT hlp_in, hlp_out; + V_VT(&hlp_in) = VT_BSTR; + V_BSTR(&hlp_in) = _bstr_("7322.9371"); + V_VT(&hlp_out) = VT_CY; + ok(VariantChangeType(&hlp_out, &hlp_in, 0, VT_CY) == S_OK, "VariantChangeType failed\n"); + VariantClear(&hlp_in); + + hr = IXMLDOMNode_get_nodeTypedValue(pNode, &v); + ok(hr == S_OK, "ret %08x\n", hr ); + ok(V_VT(&v) == VT_CY, "incorrect type\n"); + ok(VarCyCmp(V_CY(&v), V_CY(&hlp_out)) == VARCMP_EQ, "incorrect value\n"); + VariantClear(&hlp_out); + VariantClear(&v); + + IXMLDOMNode_Release(pNode); + } } IXMLDOMDocument2_Release(doc);

1 0

Piotr Caban : msxml3: Implemented xmlnode_get_nodeTypedValue 'boolean' data type handling.
by Alexandre Julliard 18 Nov '09

18 Nov '09

Module: wine Branch: master Commit: 39f0504c86b203b24e7ee15eed4815bd90f09eab URL: http://source.winehq.org/git/wine.git/?a=commit;h=39f0504c86b203b24e7ee15ee… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Nov 18 00:51:30 2009 +0100 msxml3: Implemented xmlnode_get_nodeTypedValue 'boolean' data type handling. --- dlls/msxml3/node.c | 2 ++ dlls/msxml3/tests/domdoc.c | 14 ++++++++++++++ 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index 0f515c4..f293154 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -966,6 +966,8 @@ inline HRESULT VARIANT_from_xmlChar(xmlChar *str, VARIANT *v, BSTR type) V_VT(v) = VT_I4; else if(!lstrcmpiW(type, szFixed)) V_VT(v) = VT_CY; + else if(!lstrcmpiW(type, szBoolean)) + V_VT(v) = VT_BOOL; else { FIXME("Type handling not yet implemented\n"); diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index cb8aa4e..8efbddd 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -155,6 +155,7 @@ static const CHAR szTypeValueXML[] = " <number2 dt:dt=\"NUMbEr\">-3.71e3</number2>\n" " <int dt:dt=\"int\">-13</int>\n" " <fixed dt:dt=\"fixed.14.4\">7322.9371</fixed>\n" +" <bool dt:dt=\"boolean\">1</bool>\n" "</root>"; static const CHAR szBasicTransformSSXMLPart1[] = @@ -4073,6 +4074,19 @@ static void test_NodeTypeValue(void) IXMLDOMNode_Release(pNode); } + + hr = IXMLDOMDocument2_selectSingleNode(doc, _bstr_("root/bool"), &pNode); + ok(hr == S_OK, "ret %08x\n", hr ); + if(hr == S_OK) + { + hr = IXMLDOMNode_get_nodeTypedValue(pNode, &v); + ok(hr == S_OK, "ret %08x\n", hr ); + ok(V_VT(&v) == VT_BOOL, "incorrect type\n"); + ok(V_BOOL(&v) == VARIANT_TRUE, "incorrect value\n"); + VariantClear( &v ); + + IXMLDOMNode_Release(pNode); + } } IXMLDOMDocument2_Release(doc);

1 0