From ed6505fbf207de846ff38ba538cb4eaadc217c7c Mon Sep 17 00:00:00 2001 From: David Coeurjolly Date: Tue, 2 Feb 2021 17:27:14 +0100 Subject: [PATCH 1/2] Fixing bad buffer init in Regularization --- ChangeLog.md | 4 +++- src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/ChangeLog.md b/ChangeLog.md index 41fa7d39e0..e02663d1a9 100644 --- a/ChangeLog.md +++ b/ChangeLog.md @@ -51,7 +51,9 @@ (Jacques-Olivier Lachaud,[#1538](https://github.com/DGtal-team/DGtal/pull/1538)) - Fix BoundedLatticePolytope::init when using half-spaces initialization (Jacques-Olivier Lachaud,[#1531](https://github.com/DGtal-team/DGtal/pull/1531)) - + - Fix an issue in DigitalSurfaceRegularization about bad buffer init + (David Coeurjolly, [#XXX](https://github.com/DGtal-team/DGtal/pull/XXX)) + - *Shapes package* - Fix the use of uninitialized variable in NGon2D. (Daniel Antunes,[#1540](https://github.com/DGtal-team/DGtal/issues/1540)) diff --git a/src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih b/src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih index 457571849e..6122228e87 100644 --- a/src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih +++ b/src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih @@ -107,10 +107,10 @@ void DGtal::DigitalSurfaceRegularization::cacheInit() std::copy(myOriginalPositions.begin(), myOriginalPositions.end(), myRegularizedPositions.begin()); //Allocating Gradient vector - myGradient.resize(myOriginalPositions.size()); myGradient.clear(); + myGradientAlign.clear(); + myGradient.resize(myOriginalPositions.size()); myGradientAlign.resize(myOriginalPositions.size()); - myGradient.clear(); ///// ///Cacheing some topological information From cfbe496ef1608650f244db7735306e756492ad0c Mon Sep 17 00:00:00 2001 From: David Coeurjolly Date: Tue, 2 Feb 2021 17:33:40 +0100 Subject: [PATCH 2/2] changelog --- ChangeLog.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ChangeLog.md b/ChangeLog.md index e02663d1a9..4232a64674 100644 --- a/ChangeLog.md +++ b/ChangeLog.md @@ -52,7 +52,7 @@ - Fix BoundedLatticePolytope::init when using half-spaces initialization (Jacques-Olivier Lachaud,[#1531](https://github.com/DGtal-team/DGtal/pull/1531)) - Fix an issue in DigitalSurfaceRegularization about bad buffer init - (David Coeurjolly, [#XXX](https://github.com/DGtal-team/DGtal/pull/XXX)) + (David Coeurjolly, [#1548](https://github.com/DGtal-team/DGtal/pull/1548)) - *Shapes package* - Fix the use of uninitialized variable in NGon2D.