On Sun Jul 2 20:51:00 2023 +0000, Alex Henrie wrote:
> atlbase.h, atlcom.h, and atlwin.h are also C++-only in Windows, but Wine
> allows them to be used in C and uses them in its own C code. I assumed
> that the same logic would apply to atlconv.h, but if that doesn't work
> then we may want to introduce a new helper in include/wine.
We have the other headers but we use them only for the ATL implementation. They are not something we'd want to use globally.
As for a new include/wine helper, there doesn't seem to be enough usage of that function to justify it.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3191#note_37561
On Sun Jul 2 20:29:53 2023 +0000, Alexandre Julliard wrote:
> A2BSTR is a C++ only macro, it's not something we'd want to use as a
> standard replacement.
atlbase.h, atlcom.h, and atlwin.h are also C++-only in Windows, but Wine allows them to be used in C and uses them in its own C code. I assumed that the same logic would apply to atlconv.h, but if that doesn't work then we may want to introduce a new helper in include/wine.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3191#note_37559
On Sun Jul 2 20:14:43 2023 +0000, Alex Henrie wrote:
> @mstefani Thanks for the feedback. Would you be willing to remove your
> downvote now that the scope of this MR has been reduced?
Sorry, forgot about that
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3191#note_37557
On Sun Jul 2 17:27:26 2023 +0000, Michael Stefaniuc wrote:
> My comment was referring to the tests where we convert ascii string literals.
> The two left commits seem fine though as those don't just convert ascii
> string literals in the code.
> But I had just a quick look at them.
@mstefani Thanks for the feedback. Would you be willing to remove your downvote now that the scope of this MR has been reduced?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3191#note_37554