Return Weight type for max_weight_to_satisfy methods

[yancyribbens]

Return Weight type for max_weight_to_satisfy methods.

[yancyribbens]

I'm not sure if there should be a more specific error added or if there is a better one to use then the general one I picked here.

[apoelstra]

I'm not sure if there should be a more specific error added or if there is a better one to use then the general one I picked here.

Fine to use the general one for now. I have an ambitious PR in the works that will try to add span information etc to errors so any "extra" work you do now will just get overwritten when I finish that.

[apoelstra]

Could you rebase this though? The rebase will be trivial, it's just that #661 made me change the lockfiles in my local CI so now they don't work without it. (I could change them back on my end but I'm being lazy and I know you're comfortable rebasing stuff.)

utACK 3adf211

[yancyribbens]

Rebased.

[apoelstra]

You need to run cargo +nightly fmt in this repo.

[yancyribbens]

You need to run cargo +nightly fmt in this repo.

Thanks, done.

[apoelstra]

Now this appears to be based on the tip of #661 rather than on master.

[yancyribbens]

Now this appears to be based on the tip of #661 rather than on master.

Now sure how I managed that. I'm now showing 4aaedf is now the previous commit.

[apoelstra]

ACK 7c5f3dc