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

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

Merged
merged 14 commits into from
Oct 14, 2024
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
b457b40
lnwallet: pack paymentDescriptor add/remove heights into Duals
ProofOfKeags Jul 16, 2024
71da6b5
lnwallet: consolidate redundant cases using Dual.ForParty
ProofOfKeags Jul 16, 2024
49add0f
lnwallet: eliminate inner-most layer of evil mutateState nonsense
ProofOfKeags Jul 17, 2024
05347c8
lnwallet: bring processFeeUpdate in line with process[Add|Remove]Entry
ProofOfKeags Jul 17, 2024
819239c
lnwallet: inline processUpdateFee and remove the function entirely
ProofOfKeags Jul 19, 2024
d82d028
lnwallet: remove mutateState from evaluateHTLCView
ProofOfKeags Jul 19, 2024
b902d08
lnwallet: use fn.Set API directly instead of empty struct map.
ProofOfKeags Jul 22, 2024
1b2cb14
lnwallet: change bool isIncoming to new lntypes.ChannelParty
ProofOfKeags Jul 22, 2024
4b2a4e3
lnwallet: pack htlcView.{OurUpdates|TheirUpdates} into Dual.
ProofOfKeags Jul 24, 2024
51c2849
lnwallet: simplify fee calculation in evaluateHTLCView
ProofOfKeags Jul 24, 2024
d5aab4a
lnwallet: consolidate loops in evaluateHTLCView
ProofOfKeags Jul 24, 2024
1222cb8
lnwallet: remove continue statements from evaluateHTLCView loops
ProofOfKeags Jul 24, 2024
f0eecfa
lnwallet: inline and remove process[Add|Remove]Entry
ProofOfKeags Jul 25, 2024
5307e7a
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
4 changes: 2 additions & 2 deletions go.sum
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -453,8 +453,8 @@ github.com/lightningnetwork/lnd/cert v1.2.2 h1:71YK6hogeJtxSxw2teq3eGeuy4rHGKcFf
github.com/lightningnetwork/lnd/cert v1.2.2/go.mod h1:jQmFn/Ez4zhDgq2hnYSw8r35bqGVxViXhX6Cd7HXM6U=
github.com/lightningnetwork/lnd/clock v1.1.1 h1:OfR3/zcJd2RhH0RU+zX/77c0ZiOnIMsDIBjgjWdZgA0=
github.com/lightningnetwork/lnd/clock v1.1.1/go.mod h1:mGnAhPyjYZQJmebS7aevElXKTFDuO+uNFFfMXK1W8xQ=
github.com/lightningnetwork/lnd/fn v1.2.1 h1:pPsVGrwi9QBwdLJzaEGK33wmiVKOxs/zc8H7+MamFf0=
github.com/lightningnetwork/lnd/fn v1.2.1/go.mod h1:SyFohpVrARPKH3XVAJZlXdVe+IwMYc4OMAvrDY32kw0=
github.com/lightningnetwork/lnd/fn v1.2.2 h1:rVtmGW1cQTmYce2XdUbRcc5qLDxqu+aQ6IGRpyspakk=
github.com/lightningnetwork/lnd/fn v1.2.2/go.mod h1:SyFohpVrARPKH3XVAJZlXdVe+IwMYc4OMAvrDY32kw0=
github.com/lightningnetwork/lnd/healthcheck v1.2.5 h1:aTJy5xeBpcWgRtW/PGBDe+LMQEmNm/HQewlQx2jt7OA=
github.com/lightningnetwork/lnd/healthcheck v1.2.5/go.mod h1:G7Tst2tVvWo7cx6mSBEToQC5L1XOGxzZTPB29g9Rv2I=
github.com/lightningnetwork/lnd/kvdb v1.4.10 h1:vK89IVv1oVH9ubQWU+EmoCQFeVRaC8kfmOrqHbY5zoY=
Expand Down
2 changes: 2 additions & 0 deletions lntypes/channel_party.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,3 +117,5 @@ func MapDual[A, B any](d Dual[A], f func(A) B) Dual[B] {
Remote: f(d.Remote),
}
}

var BothParties []ChannelParty = []ChannelParty{Local, Remote}
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