react-native-windows 0.68.1 → 0.69.0-preview.1

package/Mso/future/details/ifuture.h

@@ -70,7 +70,7 @@ struct IFuture : IUnknown {
   virtual void AddContinuation(Mso::CntPtr<IFuture> &&continuation) noexcept = 0;
 
   _Success_(
-      return ) virtual bool TryStartSetValue(_Out_ ByteArrayView &valueBuffer, bool crashIfFailed = false) noexcept = 0;
+      return) virtual bool TryStartSetValue(_Out_ ByteArrayView &valueBuffer, bool crashIfFailed = false) noexcept = 0;
   virtual void Post() noexcept = 0;
   virtual void StartAwaiting() noexcept = 0;
   virtual bool TrySetSuccess(bool crashIfFailed = false) noexcept = 0;
@@ -81,7 +81,7 @@ struct IFuture : IUnknown {
   virtual bool IsFailed() const noexcept = 0;
 
   template <class T, class... TArgs>
-  void SetValue(TArgs &&... args) noexcept {
+  void SetValue(TArgs &&...args) noexcept {
     ByteArrayView valueBuffer;
     (void)TryStartSetValue(/*ref*/ valueBuffer, /*crashIfFailed:*/ true);
     ::new (valueBuffer.VoidDataChecked(sizeof(T))) T(std::forward<TArgs>(args)...);
@@ -89,7 +89,7 @@ struct IFuture : IUnknown {
   }
 
   template <class T, class... TArgs>
-  bool TrySetValue(TArgs &&... args) noexcept {
+  bool TrySetValue(TArgs &&...args) noexcept {
     ByteArrayView valueBuffer;
     if (TryStartSetValue(/*ref*/ valueBuffer)) {
       ::new (valueBuffer.VoidDataChecked(sizeof(T))) T(std::forward<TArgs>(args)...);

package/Mso/future/details/maybeInvoker.h

@@ -15,7 +15,7 @@ namespace Mso::Futures {
 template <class TResult, bool isNoExcept>
 struct MaybeInvoker {
   template <class TCallback, class... TArgs>
-  static auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  static auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     OACR_WARNING_SUPPRESS(UNNECESSARY_TRY_CATCH, "Here we only execute lambdas that have no noexcept");
     try {
       return Mso::Maybe<TResult>(callback(std::forward<TArgs>(args)...));
@@ -29,7 +29,7 @@ struct MaybeInvoker {
 template <class TResult>
 struct MaybeInvoker<TResult, /*isNoExcept:*/ true> {
   template <class TCallback, class... TArgs>
-  static auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  static auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     return Mso::Maybe<TResult>(callback(std::forward<TArgs>(args)...));
   }
 };
@@ -38,7 +38,7 @@ struct MaybeInvoker<TResult, /*isNoExcept:*/ true> {
 template <>
 struct MaybeInvoker<void, /*isNoExcept:*/ false> {
   template <class TCallback, class... TArgs>
-  static auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  static auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     OACR_WARNING_SUPPRESS(UNNECESSARY_TRY_CATCH, "Here we only execute lambdas that have no noexcept");
     try {
       callback(std::forward<TArgs>(args)...);
@@ -53,7 +53,7 @@ struct MaybeInvoker<void, /*isNoExcept:*/ false> {
 template <>
 struct MaybeInvoker<void, /*isNoExcept:*/ true> {
   template <class TCallback, class... TArgs>
-  static auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  static auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     callback(std::forward<TArgs>(args)...);
     return Mso::Maybe<void>();
   }
@@ -63,7 +63,7 @@ struct MaybeInvoker<void, /*isNoExcept:*/ true> {
 template <class T>
 struct MaybeInvoker<Mso::Maybe<T>, /*isNoExcept:*/ false> {
   template <class TCallback, class... TArgs>
-  static auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  static auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     OACR_WARNING_SUPPRESS(UNNECESSARY_TRY_CATCH, "Here we only execute lambdas that have no noexcept");
     try {
       return callback(std::forward<TArgs>(args)...);
@@ -77,7 +77,7 @@ struct MaybeInvoker<Mso::Maybe<T>, /*isNoExcept:*/ false> {
 template <class T>
 struct MaybeInvoker<Mso::Maybe<T>, /*isNoExcept:*/ true> {
   template <class TCallback, class... TArgs>
-  static auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  static auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     return callback(std::forward<TArgs>(args)...);
   }
 };

package/Mso/future/details/promiseGroupInl.h

@@ -103,28 +103,28 @@ inline bool PromiseGroup<T>::TrySetValue(T &&value) const noexcept {
 
 template <class T>
 template <class... TArgs>
-inline void PromiseGroup<T>::EmplaceValue(TArgs &&... args) const noexcept {
+inline void PromiseGroup<T>::EmplaceValue(TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x0150d40b /* tag_bunql */);
   m_state->SetValue<T>(std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class TArg, class... TArgs>
-inline void PromiseGroup<T>::EmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept {
+inline void PromiseGroup<T>::EmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x0150d40c /* tag_bunqm */);
   m_state->SetValue<T>(init, std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class... TArgs>
-inline bool PromiseGroup<T>::TryEmplaceValue(TArgs &&... args) const noexcept {
+inline bool PromiseGroup<T>::TryEmplaceValue(TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x0150d40d /* tag_bunqn */);
   return m_state->TrySetValue<T>(std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class TArg, class... TArgs>
-inline bool PromiseGroup<T>::TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept {
+inline bool PromiseGroup<T>::TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x0150d40e /* tag_bunqo */);
   return m_state->TrySetValue<T>(init, std::forward<TArgs>(args)...);
 }

package/Mso/future/details/promiseInl.h

@@ -132,28 +132,28 @@ inline bool Promise<T>::TrySetValue(Mso::Maybe<T> &&value) const noexcept {
 
 template <class T>
 template <class... TArgs>
-inline void Promise<T>::EmplaceValue(TArgs &&... args) const noexcept {
+inline void Promise<T>::EmplaceValue(TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x012ca405 /* tag_blkqf */);
   this->m_state->template SetValue<T>(std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class TArg, class... TArgs>
-inline void Promise<T>::EmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept {
+inline void Promise<T>::EmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x016056ce /* tag_byf1o */);
   m_state->SetValue<T>(init, std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class... TArgs>
-inline bool Promise<T>::TryEmplaceValue(TArgs &&... args) const noexcept {
+inline bool Promise<T>::TryEmplaceValue(TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x012ca406 /* tag_blkqg */);
   return m_state->TrySetValue<T>(std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class TArg, class... TArgs>
-inline bool Promise<T>::TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept {
+inline bool Promise<T>::TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept {
   VerifyElseCrashSzTag(!m_state.IsEmpty(), "State is empty.", 0x016056cf /* tag_byf1p */);
   return m_state->TrySetValue<T>(init, std::forward<TArgs>(args)...);
 }

package/Mso/future/details/resultTraits.h

@@ -75,8 +75,8 @@ template <class TFutureValue, class TExecutor, class TCallback>
 struct GetResultTraits<TFutureValue, TExecutor, TCallback, CallbackArgKind::Maybe> {
   using Type = ResultTraits<
       TFutureValue,
-      decltype(
-          std::declval<TExecutor>().Invoke(std::declval<TCallback>(), std::declval<Mso::Maybe<TFutureValue> &&>()))>;
+      decltype(std::declval<TExecutor>()
+                   .Invoke(std::declval<TCallback>(), std::declval<Mso::Maybe<TFutureValue> &&>()))>;
   static_assert(
       noexcept(
           std::declval<TExecutor>().Invoke(std::declval<TCallback>(), std::declval<Mso::Maybe<TFutureValue> &&>())),
@@ -87,8 +87,8 @@ template <class TFutureValue, class TExecutor, class TCallback>
 struct GetResultTraits<TFutureValue, TExecutor, TCallback, CallbackArgKind::MaybeRef> {
   using Type = ResultTraits<
       TFutureValue,
-      decltype(
-          std::declval<TExecutor>().Invoke(std::declval<TCallback>(), std::declval<Mso::Maybe<TFutureValue> &>()))>;
+      decltype(std::declval<TExecutor>()
+                   .Invoke(std::declval<TCallback>(), std::declval<Mso::Maybe<TFutureValue> &>()))>;
   static_assert(
       noexcept(std::declval<TExecutor>().Invoke(std::declval<TCallback>(), std::declval<Mso::Maybe<TFutureValue> &>())),
       "Executor's Invoke method must not throw");

package/Mso/future/details/whenAllInl.h

@@ -289,7 +289,7 @@ inline Future<void> WhenAll(Future<void> (&futures)[size]) noexcept {
 }
 
 template <class T0, class... Ts>
-Future<std::tuple<T0, Ts...>> WhenAll(const Future<T0> &future0, const Future<Ts> &... futures) noexcept {
+Future<std::tuple<T0, Ts...>> WhenAll(const Future<T0> &future0, const Future<Ts> &...futures) noexcept {
   using ResultType = std::tuple<T0, Ts...>;
   using TaskType = Mso::Futures::WhenAllFutureTask<void>;
   const size_t taskSize = TaskType::GetTaskSize(1 + sizeof...(Ts));

package/Mso/future/future.h

@@ -86,11 +86,11 @@ Future<void> MakeCompletedFuture() noexcept;
 
 //! Create an instance of completed Mso::Future<T> from a value created in-place.
 template <class T, class... TArgs>
-Future<T> MakeCompletedFutureEmplaced(TArgs &&... args) noexcept;
+Future<T> MakeCompletedFutureEmplaced(TArgs &&...args) noexcept;
 
 //! Create an instance of completed Mso::Future<T> from a value created in-place.
 template <class T, class U, class... TArgs>
-Future<T> MakeCompletedFutureEmplaced(std::initializer_list<U> il, TArgs &&... args) noexcept;
+Future<T> MakeCompletedFutureEmplaced(std::initializer_list<U> il, TArgs &&...args) noexcept;
 
 //! Create an instance of succeeded Mso::Future<T> from a provided value.
 template <class T>
@@ -101,11 +101,11 @@ Future<void> MakeSucceededFuture() noexcept;
 
 //! Create an instance of completed Mso::Future<T> from a value created in-place.
 template <class T, class... TArgs>
-Future<T> MakeSucceededFutureEmplaced(TArgs &&... args) noexcept;
+Future<T> MakeSucceededFutureEmplaced(TArgs &&...args) noexcept;
 
 //! Create an instance of completed Mso::Future<T> from a value created in-place.
 template <class T, class U, class... TArgs>
-Future<T> MakeSucceededFutureEmplaced(std::initializer_list<U> il, TArgs &&... args) noexcept;
+Future<T> MakeSucceededFutureEmplaced(std::initializer_list<U> il, TArgs &&...args) noexcept;
 
 //! Create an instance of Mso::Future<T> with error code.
 template <class T>
@@ -169,7 +169,7 @@ LIBLET_PUBLICAPI Future<void> WhenAll(const std::vector<Future<void>> &futures)
 //! Each future may have its own return type.
 //! Returns Future<std::tuple<Ts...>>.
 template <class T0, class... Ts>
-Future<std::tuple<T0, Ts...>> WhenAll(const Future<T0> &future0, const Future<Ts> &... futures) noexcept;
+Future<std::tuple<T0, Ts...>> WhenAll(const Future<T0> &future0, const Future<Ts> &...futures) noexcept;
 
 //=============================================================================
 // Mso::WhenAny overloads.
@@ -315,19 +315,19 @@ struct Promise {
 
   /// Sets the value in-place and completes the Promise. It can be called only once. Otherwise it crashes the app.
   template <class... TArgs>
-  void EmplaceValue(TArgs &&... args) const noexcept;
+  void EmplaceValue(TArgs &&...args) const noexcept;
 
   /// Sets the value in-place and completes the PromiseGroup. It can be called only once. Otherwise it VECs.
   template <class TArg, class... TArgs>
-  void EmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept;
+  void EmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept;
 
   /// Tries to set the value in-place if Promise is not completed or abandoned yet. Returns true on success.
   template <class... TArgs>
-  bool TryEmplaceValue(TArgs &&... args) const noexcept;
+  bool TryEmplaceValue(TArgs &&...args) const noexcept;
 
   /// Tries to set the value in-place if PromiseGroup is not completed or abandoned yet. Returns true on success.
   template <class TArg, class... TArgs>
-  bool TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept;
+  bool TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept;
 
   /// Tries to set Promise to Failed state with a CancellationError. Returns true if cancel succeeded.
   bool TryCancel() const noexcept;
@@ -809,19 +809,19 @@ struct PromiseGroup {
 
   /// Sets the value in-place and completes the PromiseGroup. It can be called only once. Otherwise it VECs.
   template <class... TArgs>
-  void EmplaceValue(TArgs &&... args) const noexcept;
+  void EmplaceValue(TArgs &&...args) const noexcept;
 
   /// Sets the value in-place and completes the PromiseGroup. It can be called only once. Otherwise it VECs.
   template <class TArg, class... TArgs>
-  void EmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept;
+  void EmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept;
 
   /// Tries to set the value in-place if PromiseGroup is not completed or abandoned yet. Returns true on success.
   template <class... TArgs>
-  bool TryEmplaceValue(TArgs &&... args) const noexcept;
+  bool TryEmplaceValue(TArgs &&...args) const noexcept;
 
   /// Tries to set the value in-place if PromiseGroup is not completed or abandoned yet. Returns true on success.
   template <class TArg, class... TArgs>
-  bool TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&... args) const noexcept;
+  bool TryEmplaceValue(std::initializer_list<TArg> init, TArgs &&...args) const noexcept;
 
   /// Tries to set Promise to Failed state with the provided error code. It crashes app if error code cannot be set.
   void SetError(const ErrorCode &errorCode) const noexcept;

package/Mso/guid/msoGuidDetails.h

@@ -16,7 +16,7 @@ See the msoGuid.h for the usage guidelines.
 #endif
 
 // Clang compiler must target C++11 or up to support MSO*GUIDs because we rely on constexpr.
-//#if (__cplusplus >= 201103L) && !defined(MSO_GUID_DISABLED)
+// #if (__cplusplus >= 201103L) && !defined(MSO_GUID_DISABLED)
 
 #include "compilerAdapters/compilerFeatures.h"
 #if MS_TARGET_POSIX

package/Mso/memoryApi/memoryApi.h

@@ -30,7 +30,7 @@ namespace Details {
 template <typename T>
 struct Emplacer {
   template <typename... Args>
-  static void Place(_Inout_updates_bytes_all_(sizeof(T)) void *mem, Args &&... args) noexcept {
+  static void Place(_Inout_updates_bytes_all_(sizeof(T)) void *mem, Args &&...args) noexcept {
     new (mem) T(std::forward<Args>(args)...);
   }
 };
@@ -38,7 +38,7 @@ struct Emplacer {
 template <typename T, size_t N>
 struct Emplacer<T[N]> {
   template <typename... Args>
-  static void Place(void *mem, Args &&... args) {
+  static void Place(void *mem, Args &&...args) {
     new (mem) T[N]{std::forward<Args>(args)...};
   }
 };
@@ -116,14 +116,18 @@ Disambiguator used to ensure a throwing new
 new (Mso::Memory::throwNew) Zoo();
 */
 OACR_WARNING_SUPPRESS(SPECIFY_SELECTANY, "Not needed for marker type")
-static const struct throwNew_t { throwNew_t() noexcept = default; } throwNew;
+static const struct throwNew_t {
+  throwNew_t() noexcept = default;
+} throwNew;
 
 /**
 Disambiguator used to ensure a crashing new
 new (Mso::Memory::failFast) Zoo();
 */
 OACR_WARNING_SUPPRESS(SPECIFY_SELECTANY, "Not needed for marker type")
-static const struct failFast_t { failFast_t() noexcept = default; } failFast;
+static const struct failFast_t {
+  failFast_t() noexcept = default;
+} failFast;
 
 /**
 Construct a object of type `T` stored at `mem`.
@@ -131,7 +135,7 @@ Construct a object of type `T` stored at `mem`.
 Arguments are forwarded to constructor of `T`.
 */
 template <typename T, typename... Args>
-static void Place(__inout_bcount(sizeof(T)) void *mem, Args &&... args) {
+static void Place(__inout_bcount(sizeof(T)) void *mem, Args &&...args) {
   Details::Emplacer<T>::Place(mem, std::forward<Args>(args)...);
 }
 
@@ -187,7 +191,9 @@ namespace NoThrow {
         new (Mso::Memory::NoThrow::MarkingLeak) Zoo();
 */
 OACR_WARNING_SUPPRESS(SPECIFY_SELECTANY, "Not needed for marker type")
-static const struct MarkingLeak_t { MarkingLeak_t() noexcept = default; } MarkingLeak;
+static const struct MarkingLeak_t {
+  MarkingLeak_t() noexcept = default;
+} MarkingLeak;
 
 } // namespace NoThrow
 } // namespace Memory
@@ -205,7 +211,7 @@ Allocates object T by passing args to its constructor.
 */
 template <typename T, typename... TArgs>
 OACR_WARNING_SUPPRESS(NULL_ON_NON_POINTER, "false positive")
-_Ret_notnull_ T *New(TArgs &&... t) {
+_Ret_notnull_ T *New(TArgs &&...t) {
   Debug(Mso::Memory::AutoShutdownLeakScope scope);
   T *pT = new (std::nothrow) T(std::forward<TArgs>(t)...);
   if (pT == nullptr)

package/Mso/motifCpp/gTestAdapter.h

@@ -32,7 +32,7 @@ struct GTestFixture : ::testing::Test {
   std::unique_ptr<TestClass> m_test;
 };
 
-template <int &... ExplicitParameterBarrier, typename Factory>
+template <int &...ExplicitParameterBarrier, typename Factory>
 inline auto RegisterTest(
     const char *test_suite_name,
     const char *test_name,

package/Mso/motifCpp/testInfo.h

@@ -31,15 +31,13 @@
                                                                                                                 \
   struct className : Mso::UnitTests::Internal::TestClassBase<className, TestClassInfo_##className>
 
-#define TEST_METHOD(methodName)                                                                                  \
-  struct TestMethodInfo_##methodName final                                                                       \
-      : Mso::UnitTests::Internal::TestMethodInfoReg<TestMethodInfo_##methodName> {                               \
-    TestMethodInfo_##methodName() : TestMethodInfoRegType{TestClassInfoType::Instance, #methodName, __LINE__} {} \
-                                                                                                                 \
-    void Invoke(Mso::UnitTests::TestClass &test) const override {                                                \
-      static_cast<TestClassType &>(test).methodName();                                                           \
-    }                                                                                                            \
-  };                                                                                                             \
+#define TEST_METHOD(methodName)                                                                                      \
+  struct TestMethodInfo_##methodName final                                                                           \
+      : Mso::UnitTests::Internal::TestMethodInfoReg<TestMethodInfo_##methodName> {                                   \
+    TestMethodInfo_##methodName() : TestMethodInfoRegType{TestClassInfoType::Instance, #methodName, __LINE__} {}     \
+                                                                                                                     \
+    void Invoke(Mso::UnitTests::TestClass &test) const override { static_cast<TestClassType &>(test).methodName(); } \
+  };                                                                                                                 \
   virtual void methodName()
 
 #define TESTMETHOD_REQUIRES_SEH(methodName) TEST_METHOD(methodName)

package/Mso/object/make.h

@@ -23,7 +23,7 @@ namespace Mso {
                                 // ValidateObject throws, or when some of the lambda's are inlined...
 
 template <typename T, typename TResult = T, typename... TArgs>
-inline Mso::CntPtr<TResult> Make(TArgs &&... args) noexcept(T::MakePolicy::IsNoExcept) {
+inline Mso::CntPtr<TResult> Make(TArgs &&...args) noexcept(T::MakePolicy::IsNoExcept) {
   typename T::RefCountPolicy::template MemoryGuard<T> memoryGuard = {};
   T::RefCountPolicy::AllocateMemory(memoryGuard);
   VerifyAllocElseCrashTag(memoryGuard.ObjMemory, 0x01117748 /* tag_bex3i */);
@@ -47,7 +47,7 @@ inline Mso::CntPtr<TResult> Make(TArgs &&... args) noexcept(T::MakePolicy::IsNoE
   Method MakeAlloc is noexcept depending on the Make policy IsNoExcept value.
 */
 template <typename T, typename TResult = T, typename TAllocArg, typename... TArgs>
-inline Mso::CntPtr<TResult> MakeAlloc(TAllocArg &&allocArg, TArgs &&... args) noexcept(T::MakePolicy::IsNoExcept) {
+inline Mso::CntPtr<TResult> MakeAlloc(TAllocArg &&allocArg, TArgs &&...args) noexcept(T::MakePolicy::IsNoExcept) {
   typename T::RefCountPolicy::template MemoryGuard<T> memoryGuard = {};
   T::RefCountPolicy::AllocateMemory(memoryGuard, std::forward<TAllocArg>(allocArg));
   VerifyAllocElseCrashTag(memoryGuard.ObjMemory, 0x01117749 /* tag_bex3j */);
@@ -70,7 +70,7 @@ inline Mso::CntPtr<TResult> MakeAlloc(TAllocArg &&allocArg, TArgs &&... args) no
   Method MakeElseNull is noexcept depending on the Make policy IsNoExcept value.
 */
 template <typename T, typename TResult = T, typename... TArgs>
-inline Mso::CntPtr<TResult> MakeElseNull(TArgs &&... args) noexcept(T::MakePolicy::IsNoExcept) {
+inline Mso::CntPtr<TResult> MakeElseNull(TArgs &&...args) noexcept(T::MakePolicy::IsNoExcept) {
   Mso::CntPtr<TResult> result; // Hopefully we can benefit from NRVO
 
   typename T::RefCountPolicy::template MemoryGuard<T> memoryGuard = {};
@@ -98,7 +98,7 @@ inline Mso::CntPtr<TResult> MakeElseNull(TArgs &&... args) noexcept(T::MakePolic
   Method MakeAllocElseNull is noexcept depending on the Make policy IsNoExcept value.
 */
 template <typename T, typename TResult = T, typename TAllocArg, typename... TArgs>
-inline Mso::CntPtr<TResult> MakeAllocElseNull(TAllocArg &&allocArg, TArgs &&... args) noexcept(
+inline Mso::CntPtr<TResult> MakeAllocElseNull(TAllocArg &&allocArg, TArgs &&...args) noexcept(
     T::MakePolicy::IsNoExcept) {
   Mso::CntPtr<TResult> result; // Hopefully we can benefit from NRVO
 
@@ -125,7 +125,7 @@ struct ThrowCtor {
   static const bool IsNoExcept = false;
 
   template <typename T, typename TMemoryGuard, typename... TArgs>
-  static void Make(TMemoryGuard &memoryGuard, TArgs &&... args) {
+  static void Make(TMemoryGuard &memoryGuard, TArgs &&...args) {
     OACR_POSSIBLE_THROW;
     memoryGuard.Obj = ::new (memoryGuard.ObjMemory) T(std::forward<TArgs>(args)...);
     memoryGuard.ObjMemory = nullptr; // Memory is now controlled by the object. Set to null to avoid memory destruction.
@@ -139,7 +139,7 @@ struct NoThrowCtor {
   static const bool IsNoExcept = true;
 
   template <typename T, typename TMemoryGuard, typename... TArgs>
-  static void Make(TMemoryGuard &memoryGuard, TArgs &&... args) noexcept {
+  static void Make(TMemoryGuard &memoryGuard, TArgs &&...args) noexcept {
     memoryGuard.Obj = ::new (memoryGuard.ObjMemory) T(std::forward<TArgs>(args)...);
     memoryGuard.ObjMemory = nullptr; // Memory is now controlled by the object. Set to null to avoid memory destruction.
   }
@@ -152,7 +152,7 @@ struct ThrowCtorAndInitializeThis {
   static const bool IsNoExcept = false;
 
   template <typename T, typename TMemoryGuard, typename... TArgs>
-  static void Make(TMemoryGuard &memoryGuard, TArgs &&... args) {
+  static void Make(TMemoryGuard &memoryGuard, TArgs &&...args) {
     OACR_POSSIBLE_THROW;
     memoryGuard.Obj = ::new (memoryGuard.ObjMemory) T();
     memoryGuard.ObjMemory = nullptr; // Memory is now controlled by the object. Set to null to avoid memory destruction.
@@ -167,7 +167,7 @@ struct NoThrowCtorAndInitializeThis {
   static const bool IsNoExcept = true;
 
   template <typename T, typename TMemoryGuard, typename... TArgs>
-  static void Make(TMemoryGuard &memoryGuard, TArgs &&... args) noexcept {
+  static void Make(TMemoryGuard &memoryGuard, TArgs &&...args) noexcept {
     memoryGuard.Obj = ::new (memoryGuard.ObjMemory) T();
     memoryGuard.ObjMemory = nullptr; // Memory is now controlled by the object. Set to null to avoid memory destruction.
     memoryGuard.Obj->InitializeThis(std::forward<TArgs>(args)...);

package/Mso/object/objectRefCount.h

@@ -18,9 +18,7 @@
 
 #define MSO_OBJECT_SIMPLEREFCOUNT(TObject)                                                         \
  public:                                                                                           \
-  bool IsUniqueRef() const noexcept {                                                              \
-    return m_refCount.load(std::memory_order_acquire) == 1;                                        \
-  }                                                                                                \
+  bool IsUniqueRef() const noexcept { return m_refCount.load(std::memory_order_acquire) == 1; }    \
   Debug(uint32_t RefCount() const noexcept { return m_refCount.load(std::memory_order_acquire); }) \
                                                                                                    \
       template <typename UseMsoMakeInsteadOfOperatorNew>                                           \
@@ -169,7 +167,7 @@ class RefCountedWrapper : public RefCountedWrapperBase, public T {
   using MakePolicy = Mso::MakePolicy::ThrowCtor;
 
   template <typename... U>
-  RefCountedWrapper(U &&... args) noexcept : T(std::forward<U>(args)...) {}
+  RefCountedWrapper(U &&...args) noexcept : T(std::forward<U>(args)...) {}
 };
 
 /**
@@ -193,7 +191,7 @@ template <typename T>
 using RefCountedPtr = Mso::CntPtr<RefCountedWrapper<T>>;
 
 template <typename T, typename... U>
-RefCountedPtr<T> Make_RefCounted(U &&... args) noexcept {
+RefCountedPtr<T> Make_RefCounted(U &&...args) noexcept {
   return Mso::Make<RefCountedWrapper<T>>(std::forward<U>(args)...);
 }
 

package/Mso/object/objectWithWeakRef.h

@@ -65,18 +65,14 @@
 // CntPtr and get a pointer to the ObjectWeakRef using the base ObjectWithWeakRef class.
 //
 
-#define MSO_OBJECT_WEAKREFCOUNT(TObject)                                        \
- public:                                                                        \
-  Mso::ObjectWeakRef &GetWeakRef() const noexcept {                             \
-    return *Mso::Details::GetWeakRef(this);                                     \
-  }                                                                             \
-  bool IsUniqueRef() const noexcept {                                           \
-    return GetWeakRef().IsUniqueRef();                                          \
-  }                                                                             \
-  Debug(uint32_t RefCount() const noexcept { return GetWeakRef().RefCount(); }) \
-                                                                                \
-      template <typename UseMsoMakeInsteadOfOperatorNew>                        \
-      void *operator new(size_t, UseMsoMakeInsteadOfOperatorNew * = nullptr);   \
+#define MSO_OBJECT_WEAKREFCOUNT(TObject)                                                      \
+ public:                                                                                      \
+  Mso::ObjectWeakRef &GetWeakRef() const noexcept { return *Mso::Details::GetWeakRef(this); } \
+  bool IsUniqueRef() const noexcept { return GetWeakRef().IsUniqueRef(); }                    \
+  Debug(uint32_t RefCount() const noexcept { return GetWeakRef().RefCount(); })               \
+                                                                                              \
+      template <typename UseMsoMakeInsteadOfOperatorNew>                                      \
+      void *operator new(size_t, UseMsoMakeInsteadOfOperatorNew * = nullptr);                 \
   MSO_NO_COPY_CTOR_AND_ASSIGNMENT(TObject)
 
 namespace Mso {
@@ -315,7 +311,7 @@ template <
     typename TResult = T,
     typename TContainer = ObjectWeakRefContainer<T, ObjectWeakRef>,
     typename... TArgs>
-inline Mso::CntPtr<TResult> MakeWeakRefObject(TArgs &&... args) noexcept(T::MakePolicy::IsNoExcept) {
+inline Mso::CntPtr<TResult> MakeWeakRefObject(TArgs &&...args) noexcept(T::MakePolicy::IsNoExcept) {
   typename T::RefCountPolicy::template MemoryGuard<T, TContainer> memoryGuard = {};
   T::RefCountPolicy::AllocateMemory(memoryGuard);
   VerifyAllocElseCrashTag(memoryGuard.ObjMemory, 0x0111774a /* tag_bex3k */);
@@ -344,7 +340,7 @@ template <
     typename TContainer = ObjectWeakRefContainer<T, ObjectWeakRef>,
     typename TAllocArg,
     typename... TArgs>
-inline Mso::CntPtr<TResult> MakeAllocWeakRefObject(TAllocArg &&allocArg, TArgs &&... args) noexcept(
+inline Mso::CntPtr<TResult> MakeAllocWeakRefObject(TAllocArg &&allocArg, TArgs &&...args) noexcept(
     T::MakePolicy::IsNoExcept) {
   typename T::RefCountPolicy::template MemoryGuard<T, TContainer> memoryGuard = {};
   T::RefCountPolicy::AllocateMemory(memoryGuard, std::forward<TAllocArg>(allocArg));

package/Mso/object/queryCast.h

@@ -159,7 +159,7 @@ class DECLSPEC_NOVTABLE QueryCastChain : public T {
 
  protected:
   template <typename... TArgs>
-  QueryCastChain(TArgs &&... args) noexcept : T(std::forward<TArgs>(args)...) {}
+  QueryCastChain(TArgs &&...args) noexcept : T(std::forward<TArgs>(args)...) {}
 
  private:
   // Recursively calls itself while we have two or more base types.
@@ -205,7 +205,7 @@ class DECLSPEC_NOVTABLE QueryCastGuid : public T {
 
  protected:
   template <typename... TArgs>
-  QueryCastGuid(TArgs &&... args) noexcept : T(std::forward<TArgs>(args)...) {}
+  QueryCastGuid(TArgs &&...args) noexcept : T(std::forward<TArgs>(args)...) {}
 };
 
 /// Implements the QueryCast that does not do any query cast for a type.
@@ -221,7 +221,7 @@ class DECLSPEC_NOVTABLE QueryCastHidden : public T {
 
  protected:
   template <typename... TArgs>
-  QueryCastHidden(TArgs &&... args) noexcept : T(std::forward<TArgs>(args)...) {}
+  QueryCastHidden(TArgs &&...args) noexcept : T(std::forward<TArgs>(args)...) {}
 };
 
 /// Delegates the QueryCast implementation to the list of the base types.
@@ -235,7 +235,7 @@ class DECLSPEC_NOVTABLE QueryCastList : public TBase0, public TBases... {
 
  protected:
   template <typename... TArgs>
-  QueryCastList(TArgs &&... args) noexcept : TBase0(std::forward<TArgs>(args)...) {}
+  QueryCastList(TArgs &&...args) noexcept : TBase0(std::forward<TArgs>(args)...) {}
 
  public:
   void *QueryCast(const GUID &riid) noexcept {

package/Mso/object/refCountedObject.h

@@ -313,7 +313,7 @@ class RefCountedObject : public TBaseType0, public TBaseTypes... {
 
  protected:
   template <typename... TArgs>
-  RefCountedObject(TArgs &&... args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
+  RefCountedObject(TArgs &&...args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
 
   virtual ~RefCountedObject() = default;
 
@@ -351,7 +351,7 @@ class RefCountedObject<Mso::SimpleRefCountPolicy<TDeleter, TAllocator>, TBaseTyp
 
  protected:
   template <typename... TArgs>
-  RefCountedObject(TArgs &&... args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
+  RefCountedObject(TArgs &&...args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
 
   virtual ~RefCountedObject() = default;
 
@@ -382,7 +382,7 @@ class RefCountedObject<Mso::WeakRefCountPolicy<TDeleter, TAllocator>, TBaseType0
 
  protected:
   template <typename... TArgs>
-  RefCountedObject(TArgs &&... args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
+  RefCountedObject(TArgs &&...args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
 
   virtual ~RefCountedObject() = default;
 };
@@ -396,7 +396,7 @@ class RefCountedObject<Mso::RefCountStrategy::NoRefCount, TBaseType0, TBaseTypes
   MSO_OBJECT_NOREFCOUNT(RefCountedObject);
 
   template <typename... TArgs>
-  RefCountedObject(TArgs &&... args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
+  RefCountedObject(TArgs &&...args) noexcept : TBaseType0(std::forward<TArgs>(args)...) {}
 
   void AddRef() const noexcept override {}
   void Release() const noexcept override {}