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

package/Mso/errorCode/errorProvider.h

@@ -28,7 +28,7 @@ class ErrorProvider final : public IErrorProvider {
   ErrorCode MakeErrorCode(const T &errorInfo) const noexcept;
 
   template <class TValue, class... TArgs>
-  Maybe<TValue> MakeMaybe(TArgs &&... errorArgs) const noexcept;
+  Maybe<TValue> MakeMaybe(TArgs &&...errorArgs) const noexcept;
 
   // TODO: deprecate this method in favor of TryGetErrorInfo
   bool IsOwnedErrorCode(const ErrorCode &errorCode) const noexcept;
@@ -87,7 +87,7 @@ ErrorCode ErrorProvider<T, GuidToken>::MakeErrorCode(const T &errorInfo) const n
 
 template <class T, class GuidToken>
 template <class TValue, class... TArgs>
-Maybe<TValue> ErrorProvider<T, GuidToken>::MakeMaybe(TArgs &&... errorArgs) const noexcept {
+Maybe<TValue> ErrorProvider<T, GuidToken>::MakeMaybe(TArgs &&...errorArgs) const noexcept {
   return Mso::Maybe<TValue>(MakeErrorCode(T(std::forward<TArgs>(errorArgs)...)));
 }
 

package/Mso/errorCode/maybe.h

@@ -46,14 +46,14 @@ class Maybe {
   _Allow_implicit_ctor_ Maybe(T &&value) noexcept;
 
   template <class... TArgs>
-  explicit Maybe(InPlaceTag, TArgs &&... args) noexcept;
+  explicit Maybe(InPlaceTag, TArgs &&...args) noexcept;
 
   template <
       class TItem,
       class... TArgs,
       class TDummy =
           typename std::enable_if<std::is_constructible<T, std::initializer_list<TItem> &, TArgs &&...>::value>::type>
-  explicit Maybe(InPlaceTag, std::initializer_list<TItem> il, TArgs &&... args) noexcept;
+  explicit Maybe(InPlaceTag, std::initializer_list<TItem> il, TArgs &&...args) noexcept;
 
   _Allow_implicit_ctor_ Maybe(const ErrorCode &error) noexcept;
   _Allow_implicit_ctor_ Maybe(ErrorCode &&error) noexcept;
@@ -163,13 +163,13 @@ Maybe<T>::Maybe(T &&value) noexcept {
 
 template <class T>
 template <class... TArgs>
-Maybe<T>::Maybe(InPlaceTag, TArgs &&... args) noexcept {
+Maybe<T>::Maybe(InPlaceTag, TArgs &&...args) noexcept {
   ::new (reinterpret_cast<T *>(&reinterpret_cast<char &>(m_value))) T(std::forward<TArgs>(args)...);
 }
 
 template <class T>
 template <class TItem, class... TArgs, class TDummy>
-Maybe<T>::Maybe(InPlaceTag, std::initializer_list<TItem> il, TArgs &&... args) noexcept {
+Maybe<T>::Maybe(InPlaceTag, std::initializer_list<TItem> il, TArgs &&...args) noexcept {
   ::new (reinterpret_cast<T *>(&reinterpret_cast<char &>(m_value))) T(il, std::forward<TArgs>(args)...);
 }
 

package/Mso/functional/functor.h

@@ -161,7 +161,7 @@ class FunctionObjectWrapper final
   template <typename T>
   FunctionObjectWrapper(T &&func) noexcept : m_func(std::forward<T>(func)) {}
 
-  virtual TResult Invoke(TArgs &&... args) noexcept override {
+  virtual TResult Invoke(TArgs &&...args) noexcept override {
     return m_func(std::forward<TArgs>(args)...);
   }
 
@@ -180,7 +180,7 @@ class FunctionObjectWrapperThrow final
   template <typename T>
   FunctionObjectWrapperThrow(T &&func) noexcept : m_func(std::forward<T>(func)) {}
 
-  virtual TResult Invoke(TArgs &&... args) override {
+  virtual TResult Invoke(TArgs &&...args) override {
     return m_func(std::forward<TArgs>(args)...);
   }
 
@@ -199,7 +199,7 @@ class FunctionPointerWrapper final : public ConstexprFunctorBase<Mso::IFunctor<T
  public:
   constexpr FunctionPointerWrapper(TFunc func) noexcept : m_func(func) {}
 
-  TResult Invoke(TArgs &&... args) noexcept override {
+  TResult Invoke(TArgs &&...args) noexcept override {
     return (*m_func)(std::forward<TArgs>(args)...);
   }
 
@@ -216,7 +216,7 @@ class FunctionPointerWrapperThrow final : public ConstexprFunctorBase<Mso::IFunc
  public:
   constexpr FunctionPointerWrapperThrow(TFunc func) noexcept : m_func(func) {}
 
-  TResult Invoke(TArgs &&... args) override {
+  TResult Invoke(TArgs &&...args) override {
     return (*m_func)(std::forward<TArgs>(args)...);
   }
 
@@ -234,7 +234,7 @@ class StatelessFunctorWrapper final : public ConstexprFunctorBase<Mso::IFunctor<
  public:
   constexpr StatelessFunctorWrapper(TFunc &func) noexcept : m_func(func) {}
 
-  TResult Invoke(TArgs &&... args) noexcept override {
+  TResult Invoke(TArgs &&...args) noexcept override {
     return m_func(std::forward<TArgs>(args)...);
   }
 
@@ -248,7 +248,7 @@ class StatelessFunctorWrapperThrow final : public ConstexprFunctorBase<Mso::IFun
  public:
   constexpr StatelessFunctorWrapperThrow(TFunc &func) noexcept : m_func(func) {}
 
-  TResult Invoke(TArgs &&... args) override {
+  TResult Invoke(TArgs &&...args) override {
     return m_func(std::forward<TArgs>(args)...);
   }
 

package/Mso/functional/functorRef.h

@@ -178,7 +178,7 @@ class FunctorRef<TResult(TArgs...)>
     FunctorRefWrapper &operator=(FunctorRefWrapper const &) = delete;
     ~FunctorRefWrapper() = delete;
 
-    TResult Invoke(TArgs &&... args) const noexcept override {
+    TResult Invoke(TArgs &&...args) const noexcept override {
       // If you see OACR warning "Nothrow Func Throws" here then it means that the
       // provided lambda or function object's operator() are not marked as noexcept.
 
@@ -269,7 +269,7 @@ class FunctorRefThrow<TResult(TArgs...)> final {
     FunctorRefThrowWrapper &operator=(FunctorRefThrowWrapper const &) = delete;
     ~FunctorRefThrowWrapper() = delete;
 
-    TResult Invoke(TArgs &&... args) const override {
+    TResult Invoke(TArgs &&...args) const override {
       OACR_POSSIBLE_THROW;
       // We use const_cast to enable support for mutable lambdas
       return (*const_cast<FunctorRefThrowWrapper *>(this)->m_func)(std::forward<TArgs>(args)...);

package/Mso/future/details/executor.h

@@ -197,7 +197,7 @@ namespace Internal {
 
 struct ExecutorInvoker {
   template <class TCallback, class... TArgs>
-  auto Invoke(TCallback &&callback, TArgs &&... args) noexcept -> decltype(callback(std::forward<TArgs>(args)...)) {
+  auto Invoke(TCallback &&callback, TArgs &&...args) noexcept -> decltype(callback(std::forward<TArgs>(args)...)) {
     UNREFERENCED_OACR(this);
     static_assert(noexcept(callback(std::forward<TArgs>(args)...)), "Callback must not throw.");
     return callback(std::forward<TArgs>(args)...);
@@ -209,7 +209,7 @@ struct ThrowingExecutor : TBaseExecutor {
   using TBaseExecutor::TBaseExecutor;
 
   template <class TCallback, class... TArgs>
-  auto Invoke(TCallback &&callback, TArgs &&... args) noexcept {
+  auto Invoke(TCallback &&callback, TArgs &&...args) noexcept {
     using TResult = decltype(callback(std::forward<TArgs>(args)...));
     constexpr const bool isNoExcept = noexcept(callback(std::forward<TArgs>(args)...));
     return Mso::Futures::MaybeInvoker<TResult, isNoExcept>::Invoke(

package/Mso/future/details/futureFuncInl.h

@@ -32,14 +32,14 @@ inline Future<void> MakeCompletedFuture() noexcept {
 }
 
 template <class T, class... TArgs>
-inline Future<T> MakeCompletedFutureEmplaced(TArgs &&... args) noexcept {
+inline Future<T> MakeCompletedFutureEmplaced(TArgs &&...args) noexcept {
   Promise<T> promise;
   promise.EmplaceValue(std::forward<TArgs>(args)...);
   return promise.AsFuture();
 }
 
 template <class T, class U, class... TArgs>
-inline Future<T> MakeCompletedFutureEmplaced(std::initializer_list<U> il, TArgs &&... args) noexcept {
+inline Future<T> MakeCompletedFutureEmplaced(std::initializer_list<U> il, TArgs &&...args) noexcept {
   Promise<T> promise;
   promise.EmplaceValue(il, std::forward<TArgs>(args)...);
   return promise.AsFuture();
@@ -59,14 +59,14 @@ inline Future<void> MakeSucceededFuture() noexcept {
 }
 
 template <class T, class... TArgs>
-inline Future<T> MakeSucceededFutureEmplaced(TArgs &&... args) noexcept {
+inline Future<T> MakeSucceededFutureEmplaced(TArgs &&...args) noexcept {
   Promise<T> promise;
   promise.EmplaceValue(std::forward<TArgs>(args)...);
   return promise.AsFuture();
 }
 
 template <class T, class U, class... TArgs>
-inline Future<T> MakeSucceededFutureEmplaced(std::initializer_list<U> il, TArgs &&... args) noexcept {
+inline Future<T> MakeSucceededFutureEmplaced(std::initializer_list<U> il, TArgs &&...args) noexcept {
   Promise<T> promise;
   promise.EmplaceValue(il, std::forward<TArgs>(args)...);
   return promise.AsFuture();

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)...);