[PATCH 5/5] msado15: Implement Field_put_Type, Field_put_DefinedSize and Field_put_Attributes.

[Hans Leidekker]

Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
---
 dlls/msado15/recordset.c | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/dlls/msado15/recordset.c b/dlls/msado15/recordset.c
index 99d22e4db0..78d13d1577 100644
--- a/dlls/msado15/recordset.c
+++ b/dlls/msado15/recordset.c
@@ -54,6 +54,9 @@ struct field
     Field             Field_iface;
     LONG              refs;
     WCHAR            *name;
+    DataTypeEnum      type;
+    LONG              defined_size;
+    LONG              attrs;
     LONG              index;
     struct recordset *recordset;
 };
@@ -240,20 +243,32 @@ static HRESULT WINAPI field_put_NumericScale( Field *iface, unsigned char scale
 
 static HRESULT WINAPI field_put_Type( Field *iface, DataTypeEnum type )
 {
-    FIXME( "%p, %u\n", iface, type );
-    return E_NOTIMPL;
+    struct field *field = impl_from_Field( iface );
+
+    TRACE( "%p, %u\n", field, type );
+
+    field->type = type;
+    return S_OK;
 }
 
 static HRESULT WINAPI field_put_DefinedSize( Field *iface, LONG size )
 {
-    FIXME( "%p, %d\n", iface, size );
-    return E_NOTIMPL;
+    struct field *field = impl_from_Field( iface );
+
+    TRACE( "%p, %d\n", field, size );
+
+    field->defined_size = size;
+    return S_OK;
 }
 
 static HRESULT WINAPI field_put_Attributes( Field *iface, LONG attrs )
 {
-    FIXME( "%p, %d\n", iface, attrs );
-    return E_NOTIMPL;
+    struct field *field = impl_from_Field( iface );
+
+    TRACE( "%p, %d\n", field, attrs );
+
+    field->attrs = attrs;
+    return S_OK;
 }
 
 static HRESULT WINAPI field_get_Status( Field *iface, LONG *status )
-- 
2.20.1

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=61911

Your paranoid android.


=== debian10 (build log) ===

error: patch failed: dlls/msado15/tests/msado15.c:342
error: patch failed: dlls/msado15/tests/msado15.c:77
Task: Patch failed to apply

=== debian10 (build log) ===

error: patch failed: dlls/msado15/tests/msado15.c:342
error: patch failed: dlls/msado15/tests/msado15.c:77
Task: Patch failed to apply