diff options
author | 2011-09-14 00:33:20 -0700 | |
---|---|---|
committer | 2011-09-14 00:33:20 -0700 | |
commit | 80f73ccf18b2d88b045f5321990a3fea8d3f1ea4 (patch) | |
tree | d535f8108f4990adf5c9936a7a4f66ba55d6e168 /native/src/correction.cpp | |
parent | 1586dc6edffc75f3ca5cf9a4b632c0972820dcf8 (diff) | |
parent | 032cfeef5abe3a5ddf2e9ef20d834869f8090bee (diff) | |
download | latinime-80f73ccf18b2d88b045f5321990a3fea8d3f1ea4.tar.gz latinime-80f73ccf18b2d88b045f5321990a3fea8d3f1ea4.tar.xz latinime-80f73ccf18b2d88b045f5321990a3fea8d3f1ea4.zip |
Merge "Delete unused function and add TODO comment for a potential bug."
Diffstat (limited to 'native/src/correction.cpp')
-rw-r--r-- | native/src/correction.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/native/src/correction.cpp b/native/src/correction.cpp index 5a0e608b8..d5bfed017 100644 --- a/native/src/correction.cpp +++ b/native/src/correction.cpp @@ -429,6 +429,9 @@ inline static void multiplyIntCapped(const int multiplier, int *base) { if (multiplier == 2) { *base = TWO_31ST_DIV_2 >= temp ? temp << 1 : S_INT_MAX; } else { + // TODO: This overflow check gives a wrong answer when, for example, + // temp = 2^16 + 1 and multiplier = 2^17 + 1. + // Fix this behavior. const int tempRetval = temp * multiplier; *base = tempRetval >= temp ? tempRetval : S_INT_MAX; } |