diff --git a/dom/quota/CipherKeyManager.h b/dom/quota/CipherKeyManager.h index abafc25dd9da..6634f9bb4b27 100644 --- a/dom/quota/CipherKeyManager.h +++ b/dom/quota/CipherKeyManager.h @@ -39,7 +39,7 @@ class CipherKeyManager { using CipherKey = typename CipherStrategy::KeyType; public: - explicit CipherKeyManager(const char* aName) : mCipherKeys(aName){}; + explicit CipherKeyManager(const char* aName) : mCipherKeys(aName) {}; Maybe Get(const nsACString& aKeyId = "default"_ns) { auto lockedCipherKeys = mCipherKeys.Lock(); diff --git a/dom/quota/Flatten.h b/dom/quota/Flatten.h index 2eb29c6010c2..4525c038e877 100644 --- a/dom/quota/Flatten.h +++ b/dom/quota/Flatten.h @@ -21,11 +21,13 @@ using std::begin; using std::end; template -auto Flatten(NestedRange&& aRange) -> std::enable_if_t< - std::is_same_v()))::value_type>>, - std::conditional_t, - std::decay_t, NestedRange>> { +auto Flatten(NestedRange&& aRange) + -> std::enable_if_t< + std::is_same_v()))::value_type>>, + std::conditional_t, + std::decay_t, NestedRange>> { return std::forward(aRange); } @@ -98,11 +100,12 @@ struct FlatRange { }; template -auto Flatten(NestedRange&& aRange) -> std::enable_if_t< - !std::is_same_v< - T, std::decay_t&>()))::value_type>>, - FlatRange> { +auto Flatten(NestedRange&& aRange) + -> std::enable_if_t< + !std::is_same_v&>()))::value_type>>, + FlatRange> { return FlatRange{std::forward(aRange)}; } diff --git a/dom/quota/QuotaCommon.h b/dom/quota/QuotaCommon.h index 166e8de969d4..66419871a046 100644 --- a/dom/quota/QuotaCommon.h +++ b/dom/quota/QuotaCommon.h @@ -589,15 +589,13 @@ struct IpcFailCustomRetVal { // read slightly better than plain numbers. // See also // https://stackoverflow.com/questions/3046889/optional-parameters-with-c-macros -#define QM_TRY_META(...) \ - { \ - MOZ_ARG_7(, ##__VA_ARGS__, \ - QM_TRY_CUSTOM_RET_VAL_WITH_CLEANUP_AND_PREDICATE(__VA_ARGS__), \ - QM_TRY_CUSTOM_RET_VAL_WITH_CLEANUP(__VA_ARGS__), \ - QM_TRY_CUSTOM_RET_VAL(__VA_ARGS__), \ - QM_TRY_PROPAGATE_ERR(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__), \ - QM_MISSING_ARGS(__VA_ARGS__)) \ - } +#define QM_TRY_META(...) \ + {MOZ_ARG_7(, ##__VA_ARGS__, \ + QM_TRY_CUSTOM_RET_VAL_WITH_CLEANUP_AND_PREDICATE(__VA_ARGS__), \ + QM_TRY_CUSTOM_RET_VAL_WITH_CLEANUP(__VA_ARGS__), \ + QM_TRY_CUSTOM_RET_VAL(__VA_ARGS__), \ + QM_TRY_PROPAGATE_ERR(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__), \ + QM_MISSING_ARGS(__VA_ARGS__))} // Generates unique variable name. This extra internal macro (along with // __COUNTER__) allows nesting of the final macro. @@ -745,14 +743,12 @@ struct IpcFailCustomRetVal { // Chooses the final implementation macro for given argument count. // See also the comment for QM_TRY_META. -#define QM_TRY_RETURN_META(...) \ - { \ - MOZ_ARG_6(, ##__VA_ARGS__, \ - QM_TRY_RETURN_CUSTOM_RET_VAL_WITH_CLEANUP(__VA_ARGS__), \ - QM_TRY_RETURN_CUSTOM_RET_VAL(__VA_ARGS__), \ - QM_TRY_RETURN_PROPAGATE_ERR(__VA_ARGS__), \ - QM_MISSING_ARGS(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__)) \ - } +#define QM_TRY_RETURN_META(...) \ + {MOZ_ARG_6(, ##__VA_ARGS__, \ + QM_TRY_RETURN_CUSTOM_RET_VAL_WITH_CLEANUP(__VA_ARGS__), \ + QM_TRY_RETURN_CUSTOM_RET_VAL(__VA_ARGS__), \ + QM_TRY_RETURN_PROPAGATE_ERR(__VA_ARGS__), \ + QM_MISSING_ARGS(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__))} // Generates unique variable name. This extra internal macro (along with // __COUNTER__) allows nesting of the final macro. @@ -826,12 +822,10 @@ struct IpcFailCustomRetVal { // Chooses the final implementation macro for given argument count. // See also the comment for QM_TRY_META. -#define QM_REPORTONLY_TRY_META(...) \ - { \ - MOZ_ARG_6(, ##__VA_ARGS__, QM_REPORTONLY_TRY_WITH_CLEANUP(__VA_ARGS__), \ - QM_REPORTONLY_TRY(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__), \ - QM_MISSING_ARGS(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__)) \ - } +#define QM_REPORTONLY_TRY_META(...) \ + {MOZ_ARG_6(, ##__VA_ARGS__, QM_REPORTONLY_TRY_WITH_CLEANUP(__VA_ARGS__), \ + QM_REPORTONLY_TRY(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__), \ + QM_MISSING_ARGS(__VA_ARGS__), QM_MISSING_ARGS(__VA_ARGS__))} // Generates unique variable name. This extra internal macro (along with // __COUNTER__) allows nesting of the final macro. @@ -1134,8 +1128,8 @@ auto ErrToDefaultOk(const nsresult aValue) -> Result { } template -auto CreateAndRejectMozPromise(StaticString aFunc, const RejectValueT& aRv) - -> decltype(auto) { +auto CreateAndRejectMozPromise(StaticString aFunc, + const RejectValueT& aRv) -> decltype(auto) { if constexpr (std::is_same_v) { return MozPromiseType::CreateAndReject(aRv, aFunc); } else if constexpr (std::is_same_v) { @@ -1227,8 +1221,8 @@ auto Reduce(Range&& aRange, T aInit, const BinaryOp& aBinaryOp) { } template -auto CollectEachInRange(Range&& aRange, const Body& aBody) - -> Result { +auto CollectEachInRange(Range&& aRange, + const Body& aBody) -> Result { for (auto&& element : aRange) { MOZ_TRY(aBody(element)); } @@ -1624,8 +1618,7 @@ Result CollectEachFile(nsIFile& aDirectory, template Result CollectEachFile(nsIFile& aDirectory, const Body& aBody) { - return detail::CollectEachFile( - aDirectory, [] { return false; }, aBody); + return detail::CollectEachFile(aDirectory, [] { return false; }, aBody); } template