fixed shared_ptr constructor from unique_ptr and context formatting

- Fixed formatting issues in `context.h`
- Adjusted `shared_ptr` construction from `unique_ptr`
- Added test for move construction from `nostd::unique_ptr`

Author: Reneg973

api/include/opentelemetry/context/context.h

@@ -105,7 +105,7 @@ class Context
       if (iter == std::end(keys_and_vals))
         return;
       auto *node = this;
-      *node =      DataList(iter->first, iter->second);
+      *node      = DataList(iter->first, iter->second);
       for (++iter; iter != std::end(keys_and_vals); ++iter)
       {
         node->next_ = nostd::shared_ptr<DataList>(new DataList(iter->first, iter->second));
@@ -116,7 +116,7 @@ class Context
     // Builds a data list with just a key and value, so it will just be the head
     // and returns that head.
     DataList(nostd::string_view key, const ContextValue &value)
-      : key_(key.begin(), key.end()), value_(value)
+        : key_(key.begin(), key.end()), value_(value)
     {}
   };
 

api/include/opentelemetry/nostd/shared_ptr.h

@@ -102,7 +102,7 @@ class shared_ptr
 
   shared_ptr(unique_ptr<T> &&other) noexcept
   {
-    std::shared_ptr<T> ptr_(std::move(other));
+    std::shared_ptr<T> ptr_(other.release());
     new (buffer_.data) shared_ptr_wrapper{std::move(ptr_)};
   }
 

api/test/nostd/shared_ptr_test.cc

@@ -96,6 +96,15 @@ TEST(SharedPtrTest, MoveConstructionFromStdSharedPtr)
   EXPECT_EQ(ptr2.get(), value);
 }
 
+TEST(SharedPtrTest, MoveConstructionFromNoStdUniquePtr)
+{
+  opentelemetry::nostd::unique_ptr<int> value(new int{123});
+  auto p = value.get();
+  shared_ptr<int> ptr{std::move(value)};
+  EXPECT_EQ(value.get(), nullptr);  // NOLINT
+  EXPECT_EQ(ptr.get(), p);
+}
+
 TEST(SharedPtrTest, Destruction)
 {
   bool was_destructed;