<div dir="ltr">Hi,<br><br>thanks for review!<br><div class="gmail_extra"><br><div class="gmail_quote">On 27 October 2014 18:50, Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 10/15/2014 01:16 AM, Marek Chalupa wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 +                      shadow_height = height / 2;<br>
+<br>
+                       /* when the height is odd, we need some part of the<br>
+                        * shadow to be one pixel higher. Let's choose the<br>
+                        * upper one */<br>
+                       if (height % 2 != 0 && !fy)<br>
+                               ++shadow_height;<br>
</blockquote>
<br></span>
This does the same thing:<br>
                        shadow_height = (height + (fy ? 0 : 1)) / 2;<br></blockquote><div><br></div><div>Thinking about that, fy is always 0 or 1, so it will work even without the ternary operator. And if we want to be really sure that fy is 0 or 1 (even when<br>somebody will change its values range) we can do:<br><br></div><div>    shadow_height = (height + !fy) / 2;<br><br></div><div>It definitely looks better than the former code.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If there is any chance height is negative you want to use >>1 rather than /2. This is because it always rounds in the same direction (down rather than toward zero). I don't think negative height happens here, however.<br></blockquote><div><br></div><div>Good point. Hypothetically, the height can be negative, because its given from the client - but then the mistake is on the client's side and it would break much more things than one-pixel error in shadow. Maybe we should add a check for negative values, but that belongs in another patch. I gonna go for /2 because it does the same thing (assuming correct input) and it is more readable.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
______________________________<u></u>_________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.<u></u>freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/wayland-devel</a><br>
</blockquote></div><br></div><div class="gmail_extra">Thanks!<br></div><div class="gmail_extra">Marek<br></div></div>