If this patch isn&#39;t accepted then the winemenubuilder ones shouldn&#39;t be either. PNG is typically used to compress the largest icon in the ICO file, and the largest icon is the one winemenubuilder will extracts to be used during the +/- 1 minute delay in Gnome before a menu item starts having the best-sized icon, so with the winemenubuilder patches and without this one some icons will regress to showing nothing initially.<br>
<br><div>IWICStream::InitializeFromIStreamRegion can always come later. The read loop is thus temporary, and a 256x256 PNG (the largest size ICO supports) cannot take up more than 256*256*4 bytes = 256 kB, so the memory usage with this patch is not a concern.</div>
<div><br><div class="gmail_quote">On Fri, Aug 27, 2010 at 7:42 AM, Vincent Povirk <span dir="ltr">&lt;<a href="mailto:madewokherd@gmail.com">madewokherd@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
You mean you&#39;re not going to implement<br>
IWICStream::InitializeFromIStreamRegion? That&#39;s no fun.<br>
<br>
Also, that read loop is a bit silly. You might as well just lock the<br>
HGLOBAL and read into it directly.<br>
<div><div></div><div class="h5"><br>
On Fri, Aug 27, 2010 at 12:30 AM, Damjan Jovanovic &lt;<a href="mailto:damjan.jov@gmail.com">damjan.jov@gmail.com</a>&gt; wrote:<br>
&gt; Changelog:<br>
&gt; *�windowscodecs: support ICO files with embedded PNGs<br>
&gt; Closes #23708.<br>
&gt; Damjan Jovanovic<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
<br>
<br>
</div></div></blockquote></div><br></div>