[cairo] clipping bug

[Jason Dorje Short]

Radek Doulík wrote:
> On Wed, 2005-08-31 at 09:08 -0700, Carl Worth wrote:
> 
>>On Thu, 25 Aug 2005 15:22:01 +0200, Radek Doulík wrote:
>>
>>>I prepared a fix which stops compositing even earlier. That way it will
>>>not hide other bugs, where compositing with zero width or height. It is
>>>included in the new patch.
>>
>>There's a style change I'd like to see for this bug fix. What I would
>>like to see is:
>>
>>	if (nothing_to_do)
>>	    return; /* or GOTO perhaps */
>>
>>rather than:
>>
>>	if (! nothing_to_do) {
>>	    /* Big existing block now indented more. */
>>	}
>>
>>I've got some notes on this general style point in cairo/CODING_STYLE.

goto is uglier than a long if block.  If indentation is too much or the 
block is too long a helper function can be used.

-jason