Remove an incorrect conversion from HKL to LANGID
This CL fixes the following warning
C4302: 'reinterpret_cast': truncation from 'HKL' to 'LANGID'
by removing the code in question.
This is actually an unintentional operation, which was introduced for a
long time ago. KLID and HKL are two different concepts. Converting one
with reinterpret_cast to the other is indeed an mistake.
That said, the impact on this code in question is supposed to be
limited, because it is only used for an early exit path. The likelihood
of accidentally falling into that path is not so high.
BUG=
TEST=
REF_BUG=26932823
REF_CL=114656281
REF_TIME=2016-02-14T17:44:54-08:00
REF_TIME_RAW=1455500694 -0800
Yohei Yukawa
8 years ago
| 0 | 0 |
MAJOR=2
|
| 1 | 1 |
MINOR=17
|
| 2 | |
BUILD=2483
|
|
2 |
BUILD=2484
|
| 3 | 3 |
REVISION=102
|
| 4 | 4 |
# NACL_DICTIONARY_VERSION is the target version of the system dictionary to be
|
| 5 | 5 |
# downloaded by NaCl Mozc.
|
| 198 | 198 |
LOG(ERROR) << "SystemParameterInfo failed: " << GetLastError();
|
| 199 | 199 |
return false;
|
| 200 | 200 |
}
|
| 201 | |
const LANGID langage_id = reinterpret_cast<LANGID>(hkl);
|
| 202 | |
const LANGID kJapaneseLangID =
|
| 203 | |
MAKELANGID(LANG_JAPANESE, SUBLANG_JAPANESE_JAPAN);
|
| 204 | |
|
| 205 | |
if (langage_id != kJapaneseLangID) {
|
| 206 | |
return false;
|
| 207 | |
}
|
| 208 | |
|
| 209 | 201 |
return ImmRegistrar::IsIME(hkl, ImmRegistrar::GetFileNameForIME());
|
| 210 | 202 |
}
|
| 211 | 203 |
|