From 45ac14fac74ebb4bcc1faea744c0f5c4276b08a9 Mon Sep 17 00:00:00 2001 From: Sameer Agarwal Date: Mon, 20 May 2013 09:15:57 -0700 Subject: [PATCH] Add destructor to Covariance. This allows CovarianceImpl to be forward declared without scoped_ptr freaking out. Thanks to Nima Keivan for reporting this. Change-Id: Icd5aa766b3aab70246055225231a4b971c6b7b90 --- include/ceres/covariance.h | 1 + internal/ceres/covariance.cc | 3 +++ 2 files changed, 4 insertions(+) diff --git a/include/ceres/covariance.h b/include/ceres/covariance.h index 1e9bb9fdb..6533487a2 100644 --- a/include/ceres/covariance.h +++ b/include/ceres/covariance.h @@ -234,6 +234,7 @@ class Covariance { }; explicit Covariance(const Options& options); + ~Covariance(); // Compute a part of the covariance matrix. // diff --git a/internal/ceres/covariance.cc b/internal/ceres/covariance.cc index 6cb52eed9..35146c582 100644 --- a/internal/ceres/covariance.cc +++ b/internal/ceres/covariance.cc @@ -42,6 +42,9 @@ Covariance::Covariance(const Covariance::Options& options) { impl_.reset(new internal::CovarianceImpl(options)); } +Covariance::~Covariance() { +} + bool Covariance::Compute( const vector >& covariance_blocks, Problem* problem) {