Change the type of "rolling start number" and "rolling period" on "diagnosis keys" to int.

Current Implementation

Currently, the type of the rolling start number and rolling period on diagnosis keys is long, see here.

Suggested Enhancement

Change both to int. Also consider renaming “rolling start number” to “rolling start interval number”, as that was changed in the spec as well.

Expected Benefits

Newest Apple/Google spec changed the rolling start interval number of temporary exposure keys to int32 (see here), which is int in Java. With the proposed change, we can avoid unnecessary type conversions.

[stevesap]

@Akshatt yes that makes sense

[Akshatt]

Hi @stevesap, I’ve forked and cloned the repo and have made changes where rolling start number and rolling period were defined. Should I also change long to int where they are passed as argument (See here) and returned (See here ) as well? Thanks for helping!