Merge bitcoin/bitcoin#25772: test: Add missing static to IsStandardTx helper

fad5bc4 test: Add missing static to IsStandardTx helper (MacroFake)

Pull request description:

  Requested in bitcoin/bitcoin#25648 (comment)

  Also remove line break from the other two helpers.

ACKs for top commit:
  glozow:
    utACK fad5bc4
  aureleoules:
    ACK fad5bc4.
  theStack:
    ACK fad5bc4

Tree-SHA512: 771411e1fb5939a58491ecf719e1929ab0150b0faae2078ac72bd13117f1d4dcffdeed5027bfae53e4336af25a4f1db47d564abc06a5a2c9ec006a9f67bae104

Author: fanquake

src/test/script_p2sh_tests.cpp

@@ -18,20 +18,18 @@
 #include <boost/test/unit_test.hpp>
 
 // Helpers:
-bool IsStandardTx(const CTransaction& tx, std::string& reason)
+static bool IsStandardTx(const CTransaction& tx, std::string& reason)
 {
     return IsStandardTx(tx, std::nullopt, DEFAULT_PERMIT_BAREMULTISIG, CFeeRate{DUST_RELAY_TX_FEE}, reason);
 }
 
-static std::vector<unsigned char>
-Serialize(const CScript& s)
+static std::vector<unsigned char> Serialize(const CScript& s)
 {
     std::vector<unsigned char> sSerialized(s.begin(), s.end());
     return sSerialized;
 }
 
-static bool
-Verify(const CScript& scriptSig, const CScript& scriptPubKey, bool fStrict, ScriptError& err)
+static bool Verify(const CScript& scriptSig, const CScript& scriptPubKey, bool fStrict, ScriptError& err)
 {
     // Create dummy to/from transactions:
     CMutableTransaction txFrom;