Bug fix for BRepExtrema_DistanceSS

I have corrected a bug fix in BRepExtrema_DistanceSS.cxx, there was missing sqrt at line 1190 and 1340. This cause incorrect computation of extrema between a vertex and a planar face (trimmed Geom_Plane) through the class BRepExtrema_DistShapeShape (which use at some point BRepExtrema_DistanceSS). Here is the corrected line: 1190: Dstmin=sqrt(Ext.SquareDistance(1)); 1340: {Dst=sqrt(ExtPF.SquareDistance(ii));

Francois Lauzon's picture

I forgot to mention it apply to release 6.3.1.

Cauchy Ding's picture

Hi Lauzon,

I searched my 6.3.1 version, both of them use the function Value instead of SquardDistance. So I think maybe your source version is not the 6.3.1.

-Ding

Francois Lauzon's picture

Hello Cauchy,
are you sure about that, because in release 6.3.1 they remove a bunch of Value() method from some of the extrema class, they don't exist anymore, I had to change my source code to reflect that.

NOTE: I just extracted a fresh release of OCC 6.3.1 and it still without the Value method, maybe you could check you own installation...?

Cauchy Ding's picture

Hi Lauzon,

I am sorry, that's my mistake. My version is 6.3.0.

-Ding

Roman Lygin's picture

Hi Francois,

Thanks for noticing this. This must have been my initial mistake when preparing a fix for Extrema (fix210 on http://sourceforge.net/projects/opencascade). Apparently Open CASCADE regression tests did not catch this case.

Sorry for any inconvenience this have caused.
Roman

(OCCPATCH)