- Clean up method implementations to be consistent and pass tests.
- Factor out common code.
- Validate arguments.
- Correctly match szNameBuf user passed strings as case-insensitive as per documentation.
- Match method parameter identifiers to be as per documentation.
- Rewrite where required to use common helpers.
Signed-off-by: Edward O'Callaghan <edward(a)antitrust.cc>
--
v2: oleaut32: Factor out TLB_get_typeinfo_by_guid()
oleaut32: Rewrite ITypeInfo_fnGetIDsOfNames()
https://gitlab.winehq.org/wine/wine/-/merge_requests/7449
- Clean up method implementations to be consistent and pass tests.
- Factor out common code.
- Validate arguments.
- Correctly match szNameBuf user passed strings as case-insensitive as per documentation.
- Match method parameter identifiers to be as per documentation.
- Rewrite where required to use common helpers.
Signed-off-by: Edward O'Callaghan <edward(a)antitrust.cc>
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/7449
Partial, as per documentation, implement fixing as the default
locale.
Signed-off-by: Edward O'Callaghan <edward(a)antitrust.cc>
--
v2: oleaut32: Implement (S|G)etVarConversionLocaleSetting()
https://gitlab.winehq.org/wine/wine/-/merge_requests/7447
On Fri Feb 28 22:41:11 2025 +0000, Bernhard Übelacker wrote:
> I am not sure if I understand you right. Before this patch `res ==
> STATUS_INVALID_PARAMETER` was part of the ok and the if condition around
> the skip. After my patch the ok should never get reached with `res ==
> STATUS_INVALID_PARAMETER`, so isn't checking for it superfluous?
> I am happy with either way, should I add it again?
Maybe I have misread part of the patch, sorry for that.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/7445#note_96273