go-kosu: update scaling methods to be consistent #243
+13
−10
go-kosu: fix copy of big.Ints
- Loading branch information
| @@ -77,7 +77,7 @@ func posterIterator(app *App, totalBalance *big.Int) func(string, *types.Poster) | ||
| // calculates a poster's period limit based on their balance and the total poster balance | ||
| func posterLimit(periodLimit uint64, posterBalance, totalBalance *big.Int) uint64 { | ||
| // copy periodLimit (pl), posterBalance (pb), totalBalance (tb) | ||
| var pl, pb, tb *big.Int | ||
| pl, pb, tb := &big.Int{}, &big.Int{}, &big.Int{} | ||
|
This conversation was marked as resolved
by gchaincl
|
||
| pl.SetUint64(periodLimit) | ||
| pb.Set(posterBalance) | ||
| tb.Set(totalBalance) | ||
ProTip!
Use n and p to navigate between commits in a pull request.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
what's the difference between this and using non-pointers ?