update spelling/indent
							parent
							
								
									8749969977
								
							
						
					
					
						commit
						9564c4873b
					
				|  | @ -91,7 +91,7 @@ namespace gtsam { | |||
|   template<class DERIVED, class FACTORGRAPH> | ||||
|   size_t BayesTreeCliqueBase<DERIVED, FACTORGRAPH>::numCachedSeparatorMarginals() const | ||||
|   { | ||||
|       std::lock_guard<std::mutex> marginalLock(cachedSeparatorMarginalMutex_); | ||||
|     std::lock_guard<std::mutex> marginalLock(cachedSeparatorMarginalMutex_); | ||||
|     if (!cachedSeparatorMarginal_) | ||||
|       return 0; | ||||
| 
 | ||||
|  |  | |||
|  | @ -93,7 +93,7 @@ namespace gtsam { | |||
| 
 | ||||
|     /// This stores the Cached separator marginal P(S)
 | ||||
|     mutable boost::optional<FactorGraphType> cachedSeparatorMarginal_; | ||||
|     /// This protects Cached seperator marginal P(S) from concurent read/writes
 | ||||
|     /// This protects Cached seperator marginal P(S) from concurrent read/writes
 | ||||
|     /// as many the functions which access it are const (hence the mutable)
 | ||||
|     /// leading to the false impression that these const functions are thread-safe
 | ||||
|     /// which is not true due to these mutable values. This is fixed by applying this mutex.
 | ||||
|  |  | |||
		Loading…
	
		Reference in New Issue