[PATCH 3/6] cairo-util: Kill a duplicate test

2014-04-28 Thread Jasper St. Pierre
If !(x margin), then clearly margin = x. No need to test for it again. --- shared/cairo-util.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/shared/cairo-util.c b/shared/cairo-util.c index 3948572..a1568ff 100644 --- a/shared/cairo-util.c +++ b/shared/cairo-util.c @@

Re: [PATCH 3/6] cairo-util: Kill a duplicate test

2014-04-28 Thread Bryce W. Harrington
On Mon, Apr 28, 2014 at 11:19:29AM -0400, Jasper St. Pierre wrote: If !(x margin), then clearly margin = x. No need to test for it again. Yep, this looks correct, and clearer logic. Reviewed-by: Bryce Harrington b.harring...@samsung.com --- shared/cairo-util.c | 4 ++-- 1 file changed, 2