OLEPICTURE: Initialize [orig|himetric][Width|Height] for PICTYPE_ICON case

[Alex Villaci­s Lasso]

Alexandre Julliard wrote:

>Alex Villaci­s Lasso <a_villacis at palosanto.com> writes:
>
>  
>
>>You are right. I did a copy&paste from the SerializeIcon code. What do
>>you think about this patch?
>>    
>>
>
>Better, but note that there is a twist with GetIconInfo, if the icon
>is monochrome the color bitmap is 0 and you then need to get the info
>from hbmMask and divide the height by 2 (you can always count on
>Microsoft to design clean and intuitive APIs...)
>
>You also need to delete the bitmaps when done. Note also that GetDC(0)
>is preferable to CreateCompatibleDC, it avoids creating a new DC just
>to delete it again.
>
>  
>
Here is the third try, with all noted changes.

Changelog:
* Initialize [orig|himetric][Width|Height] for PICTYPE_ICON case


-------------- next part --------------
A non-text attachment was scrubbed...
Name: wine-oleaut32-olepicture-OLEPictureImpl_SetIcon_try3.patch
Type: text/x-patch
Size: 1696 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20050329/173ddf95/wine-oleaut32-olepicture-OLEPictureImpl_SetIcon_try3.bin