[1/2] windowscodecs: Add a test for CreateBitmapFromHICON.
bunglehead at gmail.com
Fri Jan 11 04:08:37 CST 2013
On 1/11/2013 14:58, Dmitry Timoshkov wrote:
> Nikolay Sivov <bunglehead at gmail.com> wrote:
>> Who is responsible for freeing icon handle? I guess it should be done
>> here cause your implementation doesn't free it either.
> You are right, DestroyIcon() is missing at the end of the test. Consider
> that as a feature, the tests are not supposed to free all the allocated
> resources, it's a waste of time :)
Fixing leaks like that is not optional. One reason to do this is to make
valgrind output clean
so it contains only real leaks in code. Another reason is that test
could serve as an example of API
More information about the wine-devel