Remove LWMAPOS dead code #148

Open
opened 2 years ago by duke · 5 comments
duke commented 2 years ago
Owner

yep

yep
Collaborator

what does acronym LWMAPOS stand for?

what does acronym LWMAPOS stand for?
Poster
Owner

LWMAPOS is Linear Weighted Moving Average Proof Of Stake, quite a horrible acronym. Don't blame me, I didn't make it up 😄

It's "dead code" that is only used for POS chains, which Hush code has not supported for a long time.

LWMAPOS is Linear Weighted Moving Average Proof Of Stake, quite a horrible acronym. Don't blame me, I didn't make it up :smile: It's "dead code" that is only used for POS chains, which Hush code has not supported for a long time.
Collaborator

is that the "ASSETCHAINS_STAKED" stuff?

is that the "ASSETCHAINS_STAKED" stuff?
Poster
Owner

This is related to the variable ASSETCHAINS_LWMAPOS which is only used when ASSETCHAINS_STAKED is not 0, which cannot happen in our codebase.

This is related to the variable `ASSETCHAINS_LWMAPOS` which is only used when `ASSETCHAINS_STAKED` is not 0, which cannot happen in our codebase.
Poster
Owner

The trick is to delete this dead code without changing any other behavior (consensus or not)

The trick is to delete this dead code without changing any other behavior (consensus or not)
duke added the
low priority
label 1 year ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.