Pixyco-XYZ/openmesh
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

took out an OpenMP for loop that caused non-deterministic behaviour

Browse Source 
refs #1005

git-svn-id: http://www.openmesh.org/svnrepo/OpenMesh/trunk@677 fdac6126-5c0c-442c-9429-916003d36597
This commit is contained in:
Isaak Lim
2012-09-17 13:37:23 +00:00
parent 1835921e40
commit 99c8dca30c
1 changed files with 0 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/OpenMesh/Tools/Decimater/ModHausdorffT.cc
View File

@@ -335,22 +335,6 @@ postprocess_collapse(const CollapseInfo& _ci)
for (p_it=points.begin(); p_it!=p_end; ++p_it) { for (p_it=points.begin(); p_it!=p_end; ++p_it) {
emin = FLT_MAX; emin = FLT_MAX;
#ifdef USE_OPENMP
int facesCount = faces.size();
#pragma omp parallel for private(e) shared(emin)
for (int i = 0; i < facesCount; ++i) {
const Point& p0 = mesh_.point(fv_it=mesh_.cfv_iter(faces[i]));
const Point& p1 = mesh_.point(++fv_it);
const Point& p2 = mesh_.point(++fv_it);
e = distPointTriangleSquared(*p_it, p0, p1, p2, dummy);
if (e < emin) {
emin = e;
fh = faces[i];
}
}
#else
for (fh_it=faces.begin(); fh_it!=fh_end; ++fh_it) { for (fh_it=faces.begin(); fh_it!=fh_end; ++fh_it) {
const Point& p0 = mesh_.point(fv_it=mesh_.cfv_iter(*fh_it)); const Point& p0 = mesh_.point(fv_it=mesh_.cfv_iter(*fh_it));
const Point& p1 = mesh_.point(++fv_it); const Point& p1 = mesh_.point(++fv_it);
@@ -363,7 +347,6 @@ postprocess_collapse(const CollapseInfo& _ci)
} }
} }
#endif
mesh_.property(points_, fh).push_back(*p_it); mesh_.property(points_, fh).push_back(*p_it);
} }