Skip to content

Commit 537820e

Browse files
committed
change int to uint32 for indices in pedersen dkg
1 parent ee9efa3 commit 537820e

1 file changed

Lines changed: 8 additions & 9 deletions

File tree

share/dkg/pedersen/status.go

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -81,23 +81,22 @@ func (s *StatusMatrix) Get(dealer, share uint32) Status {
8181

8282
func (s *StatusMatrix) String() string {
8383
// get dealer indexes
84-
dealerIdx := make([]int, 0, len((*s)))
85-
//todo what are these indexes? Should they be uint32?
84+
dealerIdx := make([]uint32, 0, len((*s)))
8685
for didx := range *s {
87-
dealerIdx = append(dealerIdx, int(didx))
86+
dealerIdx = append(dealerIdx, didx)
8887
}
8988
// get share holder indexes
90-
sharesIdx := make([]int, 0, len((*s)[uint32(dealerIdx[0])]))
91-
for sidx := range (*s)[uint32(dealerIdx[0])] {
92-
sharesIdx = append(sharesIdx, int(sidx))
89+
sharesIdx := make([]uint32, 0, len((*s)[dealerIdx[0]]))
90+
for sidx := range (*s)[dealerIdx[0]] {
91+
sharesIdx = append(sharesIdx, sidx)
9392
}
94-
sort.Ints(dealerIdx)
95-
sort.Ints(sharesIdx)
93+
sort.Slice(dealerIdx, func(i, j int) bool { return dealerIdx[i] < dealerIdx[j] })
94+
sort.Slice(sharesIdx, func(i, j int) bool { return sharesIdx[i] < sharesIdx[j] })
9695
var str = ""
9796
for _, dealerIndex := range dealerIdx {
9897
var statuses []string
9998
for _, shareIndex := range sharesIdx {
100-
status := (*s)[uint32(dealerIndex)][uint32(shareIndex)]
99+
status := (*s)[dealerIndex][shareIndex]
101100
var st string
102101
if status == Success {
103102
st = fmt.Sprintf(" %d: ok", shareIndex)

0 commit comments

Comments
 (0)