From 3fe02c32901b1398dc49b51ad95aaf39f52e5484 Mon Sep 17 00:00:00 2001 From: Frank Dellaert Date: Sun, 4 Sep 2011 14:03:45 +0000 Subject: [PATCH] Re-formatted constructors for clarity --- gtsam/linear/GaussianSequentialSolver.cpp | 37 ++++++++++++++--------- 1 file changed, 22 insertions(+), 15 deletions(-) diff --git a/gtsam/linear/GaussianSequentialSolver.cpp b/gtsam/linear/GaussianSequentialSolver.cpp index 4ddb26da3..252d1d7aa 100644 --- a/gtsam/linear/GaussianSequentialSolver.cpp +++ b/gtsam/linear/GaussianSequentialSolver.cpp @@ -25,23 +25,30 @@ namespace gtsam { /* ************************************************************************* */ -GaussianSequentialSolver::GaussianSequentialSolver(const FactorGraph& factorGraph, bool useQR) : - Base(factorGraph), useQR_(useQR) {} - -/* ************************************************************************* */ -GaussianSequentialSolver::GaussianSequentialSolver(const FactorGraph::shared_ptr& factorGraph, - const VariableIndex::shared_ptr& variableIndex, bool useQR) : - Base(factorGraph, variableIndex), useQR_(useQR) {} - -/* ************************************************************************* */ -GaussianSequentialSolver::shared_ptr GaussianSequentialSolver::Create( - const FactorGraph::shared_ptr& factorGraph, const VariableIndex::shared_ptr& variableIndex, bool useQR) { - return shared_ptr(new GaussianSequentialSolver(factorGraph, variableIndex, useQR)); +GaussianSequentialSolver::GaussianSequentialSolver( + const FactorGraph& factorGraph, bool useQR) : + Base(factorGraph), useQR_(useQR) { } /* ************************************************************************* */ -void GaussianSequentialSolver::replaceFactors(const FactorGraph::shared_ptr& factorGraph) { - Base::replaceFactors(factorGraph); +GaussianSequentialSolver::GaussianSequentialSolver( + const FactorGraph::shared_ptr& factorGraph, + const VariableIndex::shared_ptr& variableIndex, bool useQR) : + Base(factorGraph, variableIndex), useQR_(useQR) { +} + +/* ************************************************************************* */ +GaussianSequentialSolver::shared_ptr GaussianSequentialSolver::Create( + const FactorGraph::shared_ptr& factorGraph, + const VariableIndex::shared_ptr& variableIndex, bool useQR) { + return shared_ptr( + new GaussianSequentialSolver(factorGraph, variableIndex, useQR)); +} + +/* ************************************************************************* */ +void GaussianSequentialSolver::replaceFactors( + const FactorGraph::shared_ptr& factorGraph) { + Base::replaceFactors(factorGraph); } /* ************************************************************************* */ @@ -113,4 +120,4 @@ GaussianSequentialSolver::jointFactorGraph(const std::vector& js) const { *Base::jointFactorGraph(js, &EliminatePreferLDL))); } -} +} /// namespace gtsam