Hi,<br><br>Compiz++ uses CompRegion instead of XRectangle, right? This patch tries to improve CompRect usability<br><br><b>First</b><br><br>I was using CompRect, but I've found that the only way to modify a CompRect is using setGeometry function:<br>
<br style="background-color: rgb(255, 255, 0); color: rgb(0, 0, 0);">
<span style="background-color: rgb(255, 255, 0); color: rgb(0, 0, 0);">void</span><br style="background-color: rgb(255, 255, 0); color: rgb(0, 0, 0);"><span style="background-color: rgb(255, 255, 0); color: rgb(0, 0, 0);">CompOutput::setGeometry (int x1, int x2, int y1, int y2);</span><br>
<br>So I created setX1, setY1, setX2, setY2 functions<br><br><b>Second</b><br><br>As you can see above, setGeometry uses: (x1,x2,y1,y2). I'm proposing to change to <span style="color: rgb(0, 0, 0); background-color: rgb(255, 255, 0);">setGeometry (x1,y1,x2,y2)</span> <br>
<br>A Rectangle is defined by two points (x1,y1) and (x2,y2). So, setGeometry will be more intuitive if we use this order.<br clear="all"><br>This patch change all core/plugins and src/ <br><br>Thanks for attention<br><br>
P.S.: Sorry for bad english.<br>
-- <br>Eduardo Gurgel Pinho<br>(GELSoL-UFC)<br><br>(Gentoo) Linux User #415930<br><a href="http://edgurgel.wordpress.com" target="_blank">http://edgurgel.wordpress.com</a><br><a href="http://alu.dc.ufc.br/%7Eeduardo" target="_blank">http://alu.dc.ufc.br/~eduardo</a><br>