Remove confusing P1008R1 violation in ATMPArgs

src/validation.cpp

@@ -489,6 +489,9 @@ class MemPoolAccept
          */
         const bool m_package_submission;
 
+        // Must use static functions below to construct objects, NOT aggregate
+        // initialization.
+
         /** Parameters for single transaction mempool validation. */
         static ATMPArgs SingleAccept(const CChainParams& chainparams, int64_t accept_time,
                                      bool bypass_limits, std::vector<COutPoint>& coins_to_uncache,
@@ -528,9 +531,6 @@ class MemPoolAccept
                             /* m_package_submission */ true,
             };
         }
-        // No default ctor to avoid exposing details to clients and allowing the possibility of
-        // mixing up the order of the arguments. Use static functions above instead.
-        ATMPArgs() = delete;
     };
 
     // Single transaction acceptance