refactor: Simply include CTxMemPool::Options in CTxMemPool directly rather than duplicating definition

[luke-jr]

Instead of duplicating mempool options two places, just include the Options struct directly on the CTxMemPool

[DrahtBot]

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type	Reviewers
ACK	kristapsk, jonatack, achow101

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Conflicts

Reviewers, this pull request conflicts with the following ones:

• #29954 (RPC: Return permitbaremultisig and maxdatacarriersize in getmempoolinfo by kristapsk)
• #29942 (Remove redundant -datacarrier option by vostrnad)
• #29680 (wallet: fix unrelated parent conflict doesn't cause child tx to be marked as conflict by Eunovo)
• #29520 (add -limitdummyscriptdatasize option by Retropex)
• #29325 (consensus: Store transaction nVersion as uint32_t by achow101)
• #29309 (Add a -permitbarepubkey option by vostrnad)
• #29060 (Policy: Report reason inputs are non standard by ismaelsadeeq)
• #28984 (Cluster size 2 package rbf by instagibbs)
• #28830 ([refactor] Check CTxMemPool options in ctor by TheCharlatan)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

[glozow]

Looking at the conflicts with kernel, v3, cluster mempool, etc., is this the kind of mempool refactor we should defer for now?

[maflcko]

Closing for now due to inactivity. Leave a comment below to have it re-opened.

[luke-jr]

reopen please

[luke-jr]

Rebased

[kristapsk]

cr utACK cc67d33

[jonatack]

Nice cleanup.

ACK cc67d33

[achow101]

ACK cc67d33