Refactor/508 change variable in StorageUnit from long int to int64_t

[LorenzzoQM]

• Tickets addressed: bsk-508
• Review: By commit
• Merge strategy: Merge (no squash)

Description

Variable types long long int were updated to int64_t to guarantee the same size of integers across platforms.

Verification

Current unit tests are still valid after the changes.

Documentation

N/A

Future work

N/A

[schaubh]

Thanks for the change.

[patkenneally]

@LorenzzoQM I encourage you to read the three articles at the bottom of the CONTRIBUTING.md. They outline many of the expectations and reasons for preparing a PR in the way we do.

[schaubh]

@patkenneally , are you ok with this branch then? I didn't see issues with his commits here. to make the requested change all these files had to be changed at once or the branch would not compile. Are you talking about him including the "removal of unneeded brackets" in these commits? He could have done that separately, I agree.

[patkenneally]

Are you talking about him including the "removal of unneeded brackets" in these commits? He could have done that separately, I agree.

@schaubh Yes I'm referring to the inclusion of nonfunctional changes in the first commit. It's of course not a problem. I was hoping to help Lorenzzo learn these little things as we go. I don't advocate changing this PR.