From 081b8053407dc9d671efec1dc0b0affd01d1a98d Mon Sep 17 00:00:00 2001 From: Richard Roberts Date: Sat, 15 Oct 2011 19:59:22 +0000 Subject: [PATCH] Removed noncopyable restriction on Conditional - this is not used on any other factors, and makes it harder to make unit tests. Should not make a difference for users since we already use exclusively pointer containers and using the linear side directly is fairly low-level anyway. --- gtsam/inference/Conditional.h | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/gtsam/inference/Conditional.h b/gtsam/inference/Conditional.h index 72fdbc44c..d75070635 100644 --- a/gtsam/inference/Conditional.h +++ b/gtsam/inference/Conditional.h @@ -36,13 +36,9 @@ namespace gtsam { * Derived classes *must* redefine the Factor and shared_ptr typedefs to refer * to the associated factor type and shared_ptr type of the derived class. See * IndexConditional and GaussianConditional for examples. - * - * We make it noncopyable so we enforce the fact that factors are - * kept in pointer containers. To be safe, you should make them - * immutable, i.e., practicing functional programming. */ template -class Conditional: public gtsam::Factor, boost::noncopyable { +class Conditional: public gtsam::Factor { private: