Merge pull request #317 from DNKpp/main

Fix streamer for input-range like types

include/trompeloeil.hpp

@@ -1381,8 +1381,8 @@ template <typename T>
   using is_collection = is_detected<iterable, T>;
 
   template <typename T,
-            bool = is_output_streamable<T>::value,
-            bool = is_collection<detail::remove_reference_t<T>>::value>
+            bool = is_output_streamable<const T>::value,
+            bool = is_collection<const detail::remove_reference_t<T>>::value>
   struct streamer
   {
     static

test/compiling_tests.hpp

@@ -531,6 +531,23 @@ struct my_printable
     int x;
 };
 
+struct my_input_range
+{
+  using iterator = std::vector<int>::iterator;
+
+  std::vector<int> data{};
+
+  iterator begin()
+  {
+    return data.begin();
+  }
+
+  iterator end()
+  {
+    return data.end();
+  }
+};
+
 class TestOutputMock
 {
 public:

test/compiling_tests_11.cpp

@@ -4523,6 +4523,28 @@ TEST_CASE_METHOD(
   REQUIRE(os.str() == "pseudo_null_comparable");
 }
 
+TEST_CASE_METHOD(
+  Fixture,
+  "C++11: An object with non-const begin/end members, is printed using its memory representation.",
+  "[C++11][C++14][streaming]")
+{
+  std::ostringstream os;
+  trompeloeil::print(
+    os, 
+    []
+    {
+      my_input_range range{};
+      range.data.emplace_back(42);
+      range.data.emplace_back(44);
+      range.data.emplace_back(44);
+  	  return range;
+    }());
+
+  const std::string printResult{os.str()};
+  const std::string expectedPrefix{std::to_string(sizeof(my_input_range)) + "-byte object={"};
+  REQUIRE(expectedPrefix == printResult.substr(0, expectedPrefix.size()));
+}
+
 #if !(defined(_MSC_VER) && _MSC_VER < 1910)
 // Disable this test case for Microsoft Visual Studio 2015
 // until a working implementation of is_null_comparable is found