New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move away from ATL smart pointers #237

Open

vanjac opened this issue Aug 17, 2023 · 0 comments

Labels

Owner

vanjac commented Aug 17, 2023 •

edited

Loading

CComPtr is useful, but anything else adds unnecessary overhead and is not up to date with C++ standards (eg. doesn't support move semantics).

Look into unique_ptr custom allocators for CoTaskMemAlloc, LocalAlloc... (see WIL?)

vanjac added the code quality label

vanjac added this to the v0.6.x milestone

vanjac added a commit that referenced this issue


          Use unique_ptr for local strings; #237

11369ce

vanjac removed this from the v0.6.x milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment