alexvanin/certificates
1
0
Fork 0
forked from TrueCloudLab/certificates
Code Pull requests Activity

Simplify abs, it performs even better.

Browse source
This commit is contained in:
Mariano Cano 2019-06-25 11:04:48 -07:00
parent e66272d6f0
commit 5f4217ca4c
1 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
authority/provisioner/keystore.go
View file

@ -138,12 +138,10 @@ func getExpirationTime(age time.Duration) time.Time {
return time.Now().Truncate(time.Second).Add(age) return time.Now().Truncate(time.Second).Add(age)
} }
// abs returns the absolute value of n using the two's complement form. // abs returns the absolute value of n.
//
// It will overflow with math.MinInt64 in the same way a branching version
// would, this is not a problem because maxAgeRegex will block negative numbers
// and the logic will never produce that number.
func abs(n time.Duration) time.Duration { func abs(n time.Duration) time.Duration {
y := n >> 63 if n < 0 {
return (n ^ y) - y return -n
}
return n
} }