Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[KILO]: DynComms Prefactor: Refactor/evaluate htlc view #9097

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
+579 −1,094
Open

[KILO]: DynComms Prefactor: Refactor/evaluate htlc view #9097

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
b42209d
lnwallet: pack paymentDescriptor add/remove heights into Duals
ProofOfKeags Jul 16, 2024
5cbdcd4
lnwallet: consolidate redundant cases using Dual.ForParty
ProofOfKeags Jul 16, 2024
7c30de3
lnwallet: eliminate inner-most layer of evil mutateState nonsense
ProofOfKeags Jul 17, 2024
13074fb
lnwallet: bring processFeeUpdate in line with process[Add|Remove]Entry
ProofOfKeags Jul 17, 2024
1d86aa5
lnwallet: inline processUpdateFee and remove the function entirely
ProofOfKeags Jul 19, 2024
c7c22c5
lnwallet: remove extraneous argument from process[Add|Remove]Entry
ProofOfKeags Jul 19, 2024
b05b762
lnwallet: remove mutateState from evaluateHTLCView
ProofOfKeags Jul 19, 2024
d721834
lnwallet: use fn.Set API directly instead of empty struct map.
ProofOfKeags Jul 22, 2024
8807f6d
lnwallet: change bool isIncoming to new lntypes.ChannelParty
ProofOfKeags Jul 22, 2024
b6ed0f2
lnwallet: pack htlcView.{OurUpdates|TheirUpdates} into Dual.
ProofOfKeags Jul 24, 2024
606bbbf
lnwallet: simplify fee calculation in evaluateHTLCView
ProofOfKeags Jul 24, 2024
cc73517
lnwallet: consolidate loops in evaluateHTLCView
ProofOfKeags Jul 24, 2024
96bf63d
lnwallet: remove continue statements from evaluateHTLCView loops
ProofOfKeags Jul 24, 2024
18861ba
lnwallet: remove useless argument from process[Add|Remove]Entry
ProofOfKeags Jul 25, 2024
851cf55
lnwallet: inline and remove process[Add|Remove]Entry
ProofOfKeags Jul 25, 2024
d2e2ef6
lnwallet: return balance changes rather than modifying references
ProofOfKeags Jul 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ require (
github.com/lightningnetwork/lightning-onion v1.2.1-0.20240712235311-98bd56499dfb
github.com/lightningnetwork/lnd/cert v1.2.2
github.com/lightningnetwork/lnd/clock v1.1.1
github.com/lightningnetwork/lnd/fn v1.2.1
github.com/lightningnetwork/lnd/fn v1.2.2
github.com/lightningnetwork/lnd/healthcheck v1.2.5
github.com/lightningnetwork/lnd/kvdb v1.4.10
github.com/lightningnetwork/lnd/queue v1.1.1
Expand Down
8 changes: 4 additions & 4 deletions lnwallet/aux_signer.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,10 +109,10 @@ func newAuxHtlcDescriptor(p *paymentDescriptor) AuxHtlcDescriptor {
ParentIndex: p.ParentIndex,
EntryType: p.EntryType,
CustomRecords: p.CustomRecords.Copy(),
addCommitHeightRemote: p.addCommitHeightRemote,
addCommitHeightLocal: p.addCommitHeightLocal,
removeCommitHeightRemote: p.removeCommitHeightRemote,
removeCommitHeightLocal: p.removeCommitHeightLocal,
addCommitHeightRemote: p.addCommitHeights.Remote,
addCommitHeightLocal: p.addCommitHeights.Local,
removeCommitHeightRemote: p.removeCommitHeights.Remote,
removeCommitHeightLocal: p.removeCommitHeights.Local,
}
}

Expand Down
Loading
Loading