On Fri, May 22, 2009 at 1:29 AM, Andre Noll [off-list ref] wrote:

If P is disabled, we could further collapse this loop by also skipping
sources where the coefficient is zero. Not sure if this would be a win
though.

It's ok to leave this as an optimization opportunity for the caller.

Otherwise, this patch looks also very nice.

Thanks
Andre

Thanks, and the other cleanups were addressed as well.