Any comments on latest patch?<br>
Br, Oleg<br><br><div class="gmail_quote">On Thu, Mar 18, 2010 at 1:51 PM, Oleg Romashin <span dir="ltr">&lt;<a href="mailto:romaxa@gmail.com">romaxa@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Added CAIRO_MUTEX_UNLOCK before return NULL if visual is not found<br>Fixed small indentation problem.<div><div></div><div class="h5"><br><br><br><div class="gmail_quote">On Wed, Mar 17, 2010 at 10:56 PM, Oleg Romashin <span dir="ltr">&lt;<a href="mailto:romaxa@gmail.com" target="_blank">romaxa@gmail.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><br><br><div class="gmail_quote"><div><div></div><div>On Wed, Mar 17, 2010 at 8:03 PM, Soeren Sandmann <span dir="ltr">&lt;<a href="mailto:sandmann@daimi.au.dk" target="_blank">sandmann@daimi.au.dk</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div><div></div><div>Oleg Romashin &lt;<a href="mailto:romaxa@gmail.com" target="_blank">romaxa@gmail.com</a>&gt; writes:<br>
<br>
&gt; Updated cairo 16bpp format support patch:<br>
&gt; <a href="https://bugs.freedesktop.org/attachment.cgi?id=34084&amp;action=edit" target="_blank">https://bugs.freedesktop.org/attachment.cgi?id=34084&amp;action=edit</a><br>
&gt;<br>
&gt; 1)  Removed CAIRO_CONTENT_COLOR16<br>
&gt; 2) Fixed rendering from 16bpp image surface -&gt; 16bpp Xlib surface<br>
&gt;<br>
&gt; Can we get this version into upstream?<br>
<br>
</div></div>In this:<br>
<br>
+           Visual *visual = NULL;<br>
+           Screen *screen = DefaultScreenOfDisplay(display-&gt;display);<br>
+           int j;<br>
+           for (j = 0; j &lt; screen-&gt;ndepths; j++) {<br>
+               Depth *d = &amp;screen-&gt;depths[j];<br>
+               if (d-&gt;depth == 16 &amp;&amp; d-&gt;nvisuals &amp;&amp; &amp;d-&gt;visuals[0]) {<br>
+                   visual = &amp;d-&gt;visuals[0];<br>
+                   break;<br>
+               }<br>
+           }<br>
<br>
don&#39;t you need to check that the visual is (a) the right class and (b)<br>
the masks are actually 565 and not, say, 655?<br></blockquote></div></div><div><br>Is this version better: ?<br><a href="https://bugs.freedesktop.org/attachment.cgi?id=34161&amp;action=edit" target="_blank">https://bugs.freedesktop.org/attachment.cgi?id=34161&amp;action=edit</a><br>


<br>Br, Oleg<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br>
<br>
Soren<br>
</blockquote></div><br>
</blockquote></div><br>
</div></div></blockquote></div><br>