helium / blockchain-core

Apache License 2.0
213 stars 86 forks source link

add fixedwidth bitvector encoding and var to control its use in miner #1260

Closed evanmcc closed 2 years ago

evanmcc commented 2 years ago

this code initially assumed that all votes would be present, but we failed to enforce that, so we need to provide a new version that can only be used safely. while we don't use this function in core, a var is required because otherwise we will produce halting blocks during the upgrade process.

evanmcc commented 2 years ago

closing in favor of #1263