algorand / algorand/go-algorand

check alignment of all atomic vars

Ouverte
#5,001 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
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.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.