algorand / algorand/go-algorand
check alignment of all atomic vars
Ouverte
new-feature-request
Team Carbon-11
- Langage dominant
- Go
- Étoiles
- 1.4k
- Forks
- 537
- Merge moyen
- 1 j 6 h
- PR mergées (30 j)
- 17
Description
## Problem
On 32 bit platforms (ARM) `sync/atomic` operations can crash if a 64 bit value is misaligned.
## Solution
Create unit tests that use `unsafe.Offset` to check if that `% 8 == 0`
See `network/wsPeer_test.go`
## Dependencies
None
## Urgency
Nice-to-have. 32 bit platforms aren't much used.
Guide de contribution
Ouvrir le guide de contribution
Évaluation
Cette issue n'a pas encore été évaluée.