Merge rust-bitcoin/rust-miniscript#583: Clean up miniscript/mod.rs

a60c642a54d685450aa9e117d854c624108ff7e5 Refactor satisfy* functions (Tobin C. Harding)
e1e95d38f53485d205eced9e5fe040e9d00cf1cc Compress patter match in script_size (Tobin C. Harding)
23897d11be018bba68eb640c397fa928abe17570 Put the two Miniscript impl blocks together (Tobin C. Harding)
5fd22a6ac611c0810f2b2af3238db7fe0f1a389b Move Miniscript satisfy* functions into main block (Tobin C. Harding)
32342de7f4213ccffeb4c9279ae15197927f5087 Move Miniscript max_satisfaction* to main block (Tobin C. Harding)
e3c1b7c3a7968d7686f3bc0e1fdd4d6ce55cce2d Move Miniscript encode/script_size into main block (Tobin C. Harding)
dbf9a62e89b5269ec988983b1339c78d4c87ce03 Combine Miniscript conversion and constructor blocks (Tobin C. Harding)
6c5db91a779d1d4ab78011e2ee5379323fac7869 Move Miniscript impl block under type (Tobin C. Harding)
fd09756d1a629dfff8c4736354d764ccd048ca27 Make Miniscript cmp/ord/hash docs uniform (Tobin C. Harding)
1d1ac370d6a294080fb703199d66a3bfec971a4b Move Miniscript Hash impl underneath others (Tobin C. Harding)
ccb05b5ae4308964ca0801475402d0826e84fdf2 Improve rustdocs on Miniscript type (Tobin C. Harding)

Pull request description:

  Make an effort to clean up the `miniscript/mod.rs` file. I could not see a reason for all the separate impl blocks, is there one? Please note this PR does not fix all the docs, just does ones that should not be too hard to review.

ACKs for top commit:
  apoelstra:
    ACK a60c642a54d685450aa9e117d854c624108ff7e5

Tree-SHA512: 2fb555f5f399fd037b177c6d2e1224915d303c6bc13de8eb2d51143e48caf7a61e6bdd8bf0cbdac7fb95a29c00ef13bf8fe0aac1a3b9d9ca47ce59e63ced041e

Author: heap-coder