odelaneau / GLIMPSE

Low Coverage Calling of Genotypes
MIT License
141 stars 27 forks source link

phase: models: use signed char instead of char #243

Open rwk-unil opened 2 weeks ago

rwk-unil commented 2 weeks ago

Because the C/C++ standard does not specify the signedness of the char type the results will be compiler/machine dependent this can lead to unexpected behaviour.

Use signed char instead of char to avoid these cases.

References :

Fixes : https://github.com/odelaneau/GLIMPSE/issues/242