diff options
author | Eike Rathke <erack@redhat.com> | 2014-04-16 00:32:33 +0200 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2014-04-16 00:33:37 +0200 |
commit | e51ca506076a566bdd587f35cc0a8c58a50cdc2c (patch) | |
tree | 0082f04be276b27b9f0fc1969a5842853fcd95db /i18nlangtag | |
parent | CID#736182 guard against a dubious call from SearchAndReplace() (diff) | |
download | core-e51ca506076a566bdd587f35cc0a8c58a50cdc2c.tar.gz core-e51ca506076a566bdd587f35cc0a8c58a50cdc2c.zip |
CID#1202769 comfort coverity
Change-Id: I84214dde71883065f1ca3737180f0d739b49d8ee
Diffstat (limited to 'i18nlangtag')
-rw-r--r-- | i18nlangtag/source/languagetag/languagetag.cxx | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/i18nlangtag/source/languagetag/languagetag.cxx b/i18nlangtag/source/languagetag/languagetag.cxx index 6541c90fa45d..3ae1f437d05d 100644 --- a/i18nlangtag/source/languagetag/languagetag.cxx +++ b/i18nlangtag/source/languagetag/languagetag.cxx @@ -1976,10 +1976,16 @@ bool LanguageTagImpl::isIsoODF() const { const_cast<LanguageTagImpl*>(this)->synCanonicalize(); if (!LanguageTag::isIsoScript( getScript())) - return ((meIsIsoODF = DECISION_NO) == DECISION_YES); + { + meIsIsoODF = DECISION_NO; + return false; + } // The usual case is lll-CC so simply check that first. if (isIsoLocale()) - return ((meIsIsoODF = DECISION_YES) == DECISION_YES); + { + meIsIsoODF = DECISION_YES; + return true; + } // If this is not ISO locale for which script must not exist it can // still be ISO locale plus ISO script lll-Ssss-CC, but not ll-vvvv ... // ll-vvvvvvvv |