-
Notifications
You must be signed in to change notification settings - Fork 190
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.
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
Change references to CoTaskMemFree #34
Comments
For the comments, I have largely left them exactly as they are in Microsoft's documentation. In fact, I have a tool that just scrapes it and pulls it into the comments. I could do a global search and replace on just 'CoTaskMemFree', but that could be wrong in places and then there are all the other replacements that I will have missed. Thoughts? |
Looking through the codebase I think all occurrences of CoTaskMemFree in comments could be replaced. Are you aware of any cases where it would be wrong? Perhaps the substitution could be added to the scaper? If it's done generically then it can be used to deal with other replacements in the future. |
Would it be possible/worthwhile to update the references to CoTaskMemFree in the comments to say Marshal.FreeCoTaskMem?
The text was updated successfully, but these errors were encountered: