Concordium / concordium-rosetta

A server implementing the Rosetta API for the Concordium blockchain.
Mozilla Public License 2.0
5 stars 1 forks source link

Ensure that all amount casts are sound #21

Closed bisgardo closed 2 years ago

bisgardo commented 2 years ago

Purpose

The currently used type i64 is wide enough for the time being but may not be so in the distant future. Using i128 instead ensures that any u64 value stays represented correctly.

Changes

Change all i64 variables for amounts to i128.

Rosetta represents amounts as strings so no changes to generated code are required.

Requested in this comment.