Re: [PATCH] The pot() function may be hard to understand. Add comment.

Maarten Bosmans <[email protected]>
Newsgroups gmane.comp.lib.cairo
Message-ID <CA+CvcKSRBumVU_KqE-mvMmhAuL-1--gLyb5BGLHQsBFvVa9q5A@mail.gmail.com>
2012/5/30 Homer Hsing <[email protected]>:
> From d6330f4af88a02d39e39eaa13de94e48b1767c8e Mon Sep 17 00:00:00 2001
> From: Homer Hsing <[email protected]>
> Date: Wed, 30 May 2012 14:06:02 +0800
> Subject: [PATCH] The pot() function may be hard to understand. Add comment.
>
> ---
>  src/cairo-clip-boxes.c    |    1 +
>  src/drm/cairo-drm-intel.c |    1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/src/cairo-clip-boxes.c b/src/cairo-clip-boxes.c
> index bbb4197..7b93de6 100644
> --- a/src/cairo-clip-boxes.c
> +++ b/src/cairo-clip-boxes.c
> @@ -52,6 +52,7 @@
>  #include "cairo-composite-rectangles-private.h"
>  #include "cairo-region-private.h"
>
> +/* return pow (2, ceil (log2 (v))) */

Why not simply do this in words? Something like:

/* Round up to the next power of two */

Maarten
--
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.