Move macro call

[tcharding]

During development of the v28 support we re-ordered some macro calls, so as not to hold up an external contributor with triviality I acked and merged as it was.

Put the macro calls in the same order so that when one diffs v27_api.rs and v28_api.rs there are no false positives in the diff.

Internal change only.

[tcharding]

Hmm, that is a surprising CI failure.

[tnull]

Btw, related to "move" and "macro call": it seems that since all macros include separate impl blocks the docs render pretty weirdly. Might be worth dropping these impl blocks and only adding them once in the respective module that uses the macros.

[tcharding]

the docs render pretty weirdly

Which docs? This whole crate is pretty weird, so I'm not totally surprised no matter which docs you link me to. The Client docs seem to be reasonably normal.

https://docs.rs/bitcoind-json-rpc-regtest/0.3.0/bitcoind_json_rpc_regtest/struct.Client.html

Might be worth dropping these impl blocks and only adding them once in the respective module that uses the macros

The whole point of the macros is that they are used in every version (goes for the test macros and the client macros.

Or am I misunderstanding you?

[tnull]

seem to be reasonably normal.

Well, other than the docs are broken up as each method listed has its own impl Client section. :)

[tcharding]

Oh yeah I see, I did not notice that before.