<div dir="ltr"><div>Ok to squash them together. Do you want me to do that?<br><br></div>It actually does not increase the runtime, because the two loops are only adding every *other* sample. Thus the same number of samples are computed.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 1:23 PM, Oded Gabbay <span dir="ltr"><<a href="mailto:oded.gabbay@gmail.com" target="_blank">oded.gabbay@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sat, Dec 12, 2015 at 8:06 PM,  <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>> wrote:<br>
> From: Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>><br>
><br>
> With the cubic fix this is plenty accurate enough, far in excess of the pixman<br>
> fixed-point error limit. Likely even 16 samples is too many.<br>
> ---<br>
>  pixman/pixman-filter.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/pixman/pixman-filter.c b/pixman/pixman-filter.c<br>
> index 7c1da0d..4aafa51 100644<br>
> --- a/pixman/pixman-filter.c<br>
> +++ b/pixman/pixman-filter.c<br>
> @@ -190,7 +190,7 @@ integral (pixman_kernel_t reconstruct, double x1,<br>
>      else<br>
>      {<br>
>         /* Integration via Simpson's rule */<br>
> -#define N_SEGMENTS 128<br>
> +#define N_SEGMENTS 16<br>
>  #define SAMPLE(a1, a2)                                                 \<br>
>         (filters[reconstruct].func ((a1)) * filters[sample].func ((a2) / scale))<br>
><br>
> --<br>
> 1.9.1<br>
><br>
</div></div>> _______________________________________________<br>
> Pixman mailing list<br>
> <a href="mailto:Pixman@lists.freedesktop.org">Pixman@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/pixman" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/pixman</a><br>
<br>
I think it is better to just squash this patch into the previous one,<br>
as it closely related and actually makes more sense to put them<br>
together so we can see the run time hasn't increased but actually<br>
decreased.<br>
<span class="HOEnZb"><font color="#888888"><br>
      Oded<br>
</font></span></blockquote></div><br></div>