From eab53f69de01ac06b9d2d20821539b4be4fade21 Mon Sep 17 00:00:00 2001
From: Fan Jiang <i@fanjiang.me>
Date: Mon, 22 Jun 2020 13:37:50 -0400
Subject: [PATCH] Address Frank's comments

---
 gtsam/base/make_shared.h | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/gtsam/base/make_shared.h b/gtsam/base/make_shared.h
index c5ac98f9c..03de30497 100644
--- a/gtsam/base/make_shared.h
+++ b/gtsam/base/make_shared.h
@@ -17,28 +17,28 @@
 
 #pragma once
 
-#include <boost/make_shared.hpp>
 #include <gtsam/base/types.h>
 
-#include <boost/type_traits.hpp>
-
 #include <Eigen/Core>
 
+#include <boost/make_shared.hpp>
+#include <boost/utility/enable_if.hpp>
+
+
 namespace gtsam {
 
-  /**
-   * And our own `make_shared` as a layer of wrapping on `boost::make_shared`
-   */
+  /// Add our own `make_shared` as a layer of wrapping on `boost::make_shared`
   template<typename T, typename ... Args>
   boost::enable_if_t<has_custom_allocator<T>::value, boost::shared_ptr<T>> make_shared(Args&&... args)
   {
     return boost::allocate_shared<T>(Eigen::aligned_allocator<T>(), std::forward<Args> (args)...);
   }
 
+  /// Fall back to the boost version if no need for alignment
   template<typename T, typename ... Args>
   boost::enable_if_t<!has_custom_allocator<T>::value, boost::shared_ptr<T>> make_shared(Args&&... args)
   {
     return boost::make_shared<T>(std::forward<Args> (args)...);
   }
 
-}
\ No newline at end of file
+}