Bug 1626761 - Fixing readability-else-after-return bug, found by clang-tidy. r=jfkthame

Differential Revision: https://phabricator.services.mozilla.com/D77591
This commit is contained in:
bharath bmsce 2020-06-01 11:48:19 +00:00
parent f747fd8867
commit 498fc0d0dc

View File

@ -55,12 +55,11 @@ char16_t HandleNumberInChar(char16_t aChar, bool aPrevCharArabic,
// for clipboard handling
// XXX do we really want to convert numerals when copying text?
if (aPrevCharArabic) {
if (aNumFlag == IBMBIDI_NUMERAL_PERSIANCONTEXT)
return NUM_TO_PERSIAN(aChar);
else
return NUM_TO_HINDI(aChar);
} else
return NUM_TO_ARABIC(aChar);
return aNumFlag == IBMBIDI_NUMERAL_PERSIANCONTEXT
? NUM_TO_PERSIAN(aChar)
: NUM_TO_HINDI(aChar);
}
return NUM_TO_ARABIC(aChar);
case IBMBIDI_NUMERAL_NOMINAL:
default:
return aChar;