← Back to team overview

yade-dev team mailing list archive

Re: [Bug 472987] Re: Suspecting a bug with CSG_shear

 

> But still, I'm just curious, is there a reason for not writing :
> 		
> 		/#ifdef SCG_SHEAR
> 			scm->prevNormal=scm->normal;
> 		#endif/
> 		
> instead of? :
> 		
> /		#ifdef SCG_SHEAR
> 			if(isNew) scm->prevNormal=normal; 
> 			else scm->prevNormal=scm->normal;
> 		#endif
> /
Hm, I fixed that in r1778, it looks like it was bug. (scm->normal should
not be the same as normal at that point, otherwise the condition would
indeed make no sense; I moved this block before the scm->normal=normal
statement.)

v





References