Page MenuHomePhabricator

[refactor multisig] remove redundant counters
ClosedPublic

Authored by markblundeberg on Jul 15 2019, 01:10.

Details

Summary

This is perhaps debatable but the ikey, isig indices are always intended
to iterate in step with the number of remaining objects, and the numbers
remaining are the important values needed to know when to stop iteration.
The ikey, isig thus only serve to avoid some (fast) arithmetic.

Depends on D3625

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jul 15 2019, 16:20
Mengerian added a subscriber: Mengerian.

This makes the code more readable for me, so it seems like a good change.