Move MakeCopyable helper class into detail namespace.

Jojo-1000 · Moritz Wirger
1 parent bc52722a
Showing 5 changed files with 36 additions and 18 deletions
include/hueplusplus/BridgeConfig.h
include/hueplusplus/Hue.h
include/hueplusplus/Utils.h
src/CMakeLists.txt
test/CMakeLists.txt
@@ -19,6 +19,9 @@
     along with hueplusplus.  If not, see <http://www.gnu.org/licenses/>.
 **/
+#ifndef INCLUDE_HUEPLUSPLUS_BRIDGE_CONFIG_H
+#define INCLUDE_HUEPLUSPLUS_BRIDGE_CONFIG_H
+
 #include <string>
 #include <vector>
@@ -55,7 +58,6 @@ public:
     //! \brief Construct BridgeConfig
     BridgeConfig(std::shared_ptr<APICache> baseCache, std::chrono::steady_clock::duration refreshDuration);
-
     //! \brief Refreshes internal cached state.
     //! \throws std::system_error when system or socket operations fail
     //! \throws HueException when response contained no body
@@ -102,4 +104,6 @@ protected:
 private:
     APICache cache;
 };
-} // namespace hueplusplus
 \ No newline at end of file
+} // namespace hueplusplus
+
+#endif
@@ -42,6 +42,7 @@
 #include "ResourceList.h"
 #include "Scene.h"
 #include "Schedule.h"
+#include "Utils.h"
 #include "json/json.hpp"
@@ -118,14 +119,6 @@ private:
     std::shared_ptr<const IHttpHandler> http_handler;
 };
-template <typename T>
-class MakeCopyable : public T
-{
-public:
-    using T::T;
-    using T::operator=;
-};
-
 //! \brief Hue class for a bridge.
 //!
 //! This is the main class used to interact with the Hue bridge.
@@ -204,7 +197,7 @@ public:
     //! \brief Provides access to the configuration of the bridge.
     //! \note Does not refresh state.
     const BridgeConfig& config() const;
-    
+
     //! \brief Provides access to the HueLight%s on the bridge.
     LightList& lights();
     //! \brief Provides access to the HueLight%s on the bridge.
@@ -238,6 +231,7 @@ private:
     //! before Hue is used.
     //! Resets all caches and resource lists.
     void setHttpHandler(std::shared_ptr<const IHttpHandler> handler);
+
 private:
     std::string ip; //!< IP-Address of the hue bridge in dotted decimal notation
                     //!< like "192.168.2.1"
@@ -248,11 +242,11 @@ private:
                                                       //!< bridge
     std::chrono::steady_clock::duration refreshDuration;
     std::shared_ptr<APICache> stateCache;
-    MakeCopyable<LightList> lightList;
-    MakeCopyable<GroupList> groupList;
-    MakeCopyable<ScheduleList> scheduleList;
-    MakeCopyable<SceneList> sceneList;
-    MakeCopyable<BridgeConfig> bridgeConfig;
+    detail::MakeCopyable<LightList> lightList;
+    detail::MakeCopyable<GroupList> groupList;
+    detail::MakeCopyable<ScheduleList> scheduleList;
+    detail::MakeCopyable<SceneList> sceneList;
+    detail::MakeCopyable<BridgeConfig> bridgeConfig;
 };
 } // namespace hueplusplus
@@ -95,6 +95,23 @@ nlohmann::json safeGetMember(const nlohmann::json&amp; json, Paths&amp;&amp;... paths)
 }
 } // namespace utils
+
+namespace detail
+{
+//! \brief Makes a class with protected copy constructor copyable.
+//!
+//! Used in private members to expose mutable references to \c T
+//! while not allowing them to be assigned to.
+//! Make sure \c T is actually designed to be used this way!
+template <typename T>
+class MakeCopyable : public T
+{
+public:
+    // Make copy constructor and assignment operator public
+    using T::T;
+    using T::operator=;
+};
+} // namespace detail
 } // namespace hueplusplus
 #endif
 \ No newline at end of file
 set(hueplusplus_SOURCES
 	APICache.cpp
     BaseHttpHandler.cpp
+    BridgeConfig.cpp
     ColorUnits.cpp
     ExtendedColorHueStrategy.cpp
     ExtendedColorTemperatureStrategy.cpp
@@ -19,7 +20,7 @@ set(hueplusplus_SOURCES
     StateTransaction.cpp
     TimePattern.cpp
     UPnP.cpp
-    Utils.cpp "BridgeConfig.cpp")
+    Utils.cpp)
 # on windows we want to compile the WinHttpHandler
 if(WIN32)
@@ -32,6 +32,8 @@ target_compile_features(gtest PUBLIC cxx_std_14)
 set(TEST_SOURCES
     test_APICache.cpp
     test_BaseHttpHandler.cpp
+    test_BridgeConfig.cpp
+    test_ColorUnits.cpp
     test_ExtendedColorHueStrategy.cpp
     test_ExtendedColorTemperatureStrategy.cpp
 	test_Group.cpp
@@ -48,7 +50,7 @@ set(TEST_SOURCES
     test_SimpleColorHueStrategy.cpp
     test_SimpleColorTemperatureStrategy.cpp
     test_StateTransaction.cpp
-    test_TimePattern.cpp "test_ColorUnits.cpp" "test_BridgeConfig.cpp")
+    test_TimePattern.cpp)
 set(HuePlusPlus_INCLUDE_DIR "${PROJECT_SOURCE_DIR}/include")