lightning

5074 Commits

55 Branches

107 Tags

38 MiB

Tree: de682f5806

Commit Graph

Author	SHA1	Message	Date
Rusty Russell	1ede7bc55b	wallet_tx: make wtx_select_utxos return command_result. It can fail the command, so it should return accordingly. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>	6 years ago
Rusty Russell	52303029aa	fundchannel: cap 'all' at 2^24-1 satoshi. The easiest way to do this is to play with the 'wallet_tx' semantics and have 'amount' have meaning even when 'all_funds' is set. Note that we change the string 'Cannot afford funding transaction' to 'Cannot afford transaction' as this code is also used for withdrawls. Inspired-by: molz on #c-lightning Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>	7 years ago
Mark Beckwith	7767b68ee9	Removed redundancies in withdraw and fundchannel. No new functionality, just a continuation of my work toward completing #665. I removed the common members of `struct withdrawal` and `struct fund_channel` and placed them in a new `struct wallet_tx`. Then it was fairly straightforward to reimplement the existing code in terms of `wallet_tx`. Since I made some structural changes I wanted to get this approved before I go any farther. Added 'all' to fundchannel help message.	7 years ago

Author

SHA1

Message

Date

Rusty Russell

1ede7bc55b

wallet_tx: make wtx_select_utxos return command_result.

It can fail the command, so it should return accordingly.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

6 years ago

Rusty Russell

52303029aa

fundchannel: cap 'all' at 2^24-1 satoshi.

The easiest way to do this is to play with the 'wallet_tx' semantics
and have 'amount' have meaning even when 'all_funds' is set.

Note that we change the string 'Cannot afford funding transaction' to
'Cannot afford transaction' as this code is also used for withdrawls.

Inspired-by: molz on #c-lightning
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

7 years ago

Mark Beckwith

7767b68ee9

Removed redundancies in withdraw and fundchannel.

No new functionality, just a continuation of my work toward completing #665.

I removed the common members of `struct withdrawal` and `struct fund_channel`
and placed them in a new `struct wallet_tx`.  Then it was fairly straightforward
to reimplement the existing code in terms of `wallet_tx`.

Since I made some structural changes I wanted to get this approved before I
go any farther.

Added 'all' to fundchannel help message.

7 years ago

3 Commits (de682f580628f817185c90c15b0aa852994df995)