On 05/16/2017 07:24 AM, clipka wrote:
> William F Pokorny <ano### [at] anonymousorg> wrote:
>
> But yes, it might be prettier to rename that variable to `n` instead.
>
>

Perhaps remove the extra scope braces in SphereSweep::Compute_BBox when 
you do this?

At or about line 1336:

     else
     {
         for (i = 0; i < Num_Modeling_Spheres; i++)
         {
             { // Extra
                 mins = min(mins, Modeling_Sphere[i].Center - 
Vector3d(fabs(Modeling_Sphere[i].Radius)));
                 maxs = max(maxs, Modeling_Sphere[i].Center + 
Vector3d(fabs(Modeling_Sphere[i].Radius)));
             } // Extra
         }
     }

Bill P.

Post a reply to this message