organicmaps/icu
13
Fork 0
mirror of https://github.com/unicode-org/icu.git synced 2025-04-10 07:39:16 +00:00
Code Issues 1 Projects Releases Wiki Activity Actions 1

ICU-8077 Remove isLenient/setLenient from Format, etc. (revert #7725)

Browse source 
X-SVN-Rev: 28916
This commit is contained in:
Peter Edberg 2010-10-26 18:11:13 +00:00
parent 64466fd4e1
commit c8765bfe77
12 changed files with 26 additions and 141 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
icu4c/source/i18n/format.cpp
View file

@ -65,7 +65,6 @@ FieldPosition::clone() const {
Format::Format()
: UObject()
, fLenient(TRUE)
{
*validLocale = *actualLocale = 0;
}
@ -94,7 +93,6 @@ Format::operator=(const Format& that)
if (this != &that) {
uprv_strcpy(validLocale, that.validLocale);
uprv_strcpy(actualLocale, that.actualLocale);
fLenient = that.fLenient;
}
return *this;
}
@ -208,16 +206,6 @@ Format::setLocaleIDs(const char* valid, const char* actual) {
locBased.setLocaleIDs(valid, actual);
}
void
Format::setLenient(UBool lenient) {
fLenient = lenient;
}
UBool
Format::isLenient() const {
return fLenient;
}
U_NAMESPACE_END
#endif /* #if !UCONFIG_NO_FORMATTING */

1
icu4c/source/i18n/numfmt.cpp
View file

@ -193,7 +193,6 @@ NumberFormat::NumberFormat()
fParseIntegerOnly(FALSE)
{
fCurrency[0] = 0;
Format::setLenient(FALSE);
}
// -------------------------------------

23
icu4c/source/i18n/rbnf.cpp
View file

@ -655,6 +655,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(const UnicodeString& description,
, locale(alocale)
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
, noParse(FALSE) //TODO: to be removed after #6895
@ -671,6 +672,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(const UnicodeString& description,
, locale(Locale::getDefault())
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
, noParse(FALSE) //TODO: to be removed after #6895
@ -687,6 +689,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(const UnicodeString& description,
, locale(alocale)
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
, noParse(FALSE) //TODO: to be removed after #6895
@ -702,6 +705,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(const UnicodeString& description,
, locale(Locale::getDefault())
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
, noParse(FALSE) //TODO: to be removed after #6895
@ -718,6 +722,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(const UnicodeString& description,
, locale(aLocale)
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
, noParse(FALSE) //TODO: to be removed after #6895
@ -731,6 +736,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(URBNFRuleSetTag tag, const Locale&
, locale(alocale)
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
{
@ -804,6 +810,7 @@ RuleBasedNumberFormat::RuleBasedNumberFormat(const RuleBasedNumberFormat& rhs)
, locale(rhs.locale)
, collator(NULL)
, decimalFormatSymbols(NULL)
, lenient(FALSE)
, lenientParseRules(NULL)
, localizations(NULL)
{
@ -818,7 +825,7 @@ RuleBasedNumberFormat::operator=(const RuleBasedNumberFormat& rhs)
UErrorCode status = U_ZERO_ERROR;
dispose();
locale = rhs.locale;
Format::setLenient( rhs.isLenient() );
lenient = rhs.lenient;
UnicodeString rules = rhs.getRules();
UParseError perror;
@ -852,7 +859,7 @@ RuleBasedNumberFormat::clone(void) const
delete result;
result = 0;
} else {
result->Format::setLenient( isLenient() );
result->lenient = lenient;
//TODO: remove below when we fix the parse bug - See #6895 / #6896
result->noParse = noParse;
@ -870,7 +877,7 @@ RuleBasedNumberFormat::operator==(const Format& other) const
if (typeid(*this) == typeid(other)) {
const RuleBasedNumberFormat& rhs = (const RuleBasedNumberFormat&)other;
if (locale == rhs.locale &&
isLenient() == rhs.isLenient() &&
lenient == rhs.lenient &&
(localizations == NULL
? rhs.localizations == NULL
: (rhs.localizations == NULL
@ -1204,19 +1211,13 @@ RuleBasedNumberFormat::parse(const UnicodeString& text,
void
RuleBasedNumberFormat::setLenient(UBool enabled)
{
Format::setLenient(enabled);
lenient = enabled;
if (!enabled && collator) {
delete collator;
collator = NULL;
}
}
UBool
RuleBasedNumberFormat::isLenient(void) const {
return Format::isLenient();
}
#endif
void
@ -1542,7 +1543,7 @@ RuleBasedNumberFormat::getCollator() const
}
// lazy-evaulate the collator
if (collator == NULL && isLenient()) {
if (collator == NULL && lenient) {
// create a default collator based on the formatter's locale,
// then pull out that collator's rules, append any additional
// rules specified in the description, and create a _new_

28
icu4c/source/i18n/unicode/format.h
View file

@ -86,11 +86,6 @@ U_NAMESPACE_BEGIN
* If there is no match when parsing, a parse failure UErrorCode is
* retured for methods which take no ParsePosition. For the method
* that takes a ParsePosition, the index parameter is left unchanged.
* The Format class can support either strict or lenient parsing where
* appropriate (see isLenient, setLenient). By default parsing is
* lenient in the Format base class; subclasses may have a different
* default (for example, in NumberFormat, parsing is strict by default
* for backwards compatibility).
* <P>
* <em>User subclasses are not supported.</em> While clients may write
* subclasses, such code will not necessarily work and will not be
@ -261,28 +256,6 @@ public:
*/
const char* getLocaleID(ULocDataLocaleType type, UErrorCode &status) const;
/**
* Controls lenient parse behavior. At the Format class level, lenient parse
* is on by default. Format subclasses may have different default behavior
* for lenient parse.
*
* @see #isLenient
* @param lenient Sets lenient parse mode on (if TRUE) or off (if FALSE).
* @draft ICU 4.6
*/
virtual void setLenient(UBool lenient);
/**
* Returns the status of lenient parse mode. At the Format class level,
* lenient parse is on by default. Format subclasses may have different
* default behavior for lenient parse.
*
* @see #setLenient
* @return Lenient parse mode status: TRUE if on, FALSE if off.
* @draft ICU 4.6
*/
virtual UBool isLenient(void) const;
protected:
/** @stable ICU 2.8 */
void setLocaleIDs(const char* valid, const char* actual);
@ -320,7 +293,6 @@ protected:
private:
char actualLocale[ULOC_FULLNAME_CAPACITY];
char validLocale[ULOC_FULLNAME_CAPACITY];
UBool fLenient;
};
U_NAMESPACE_END

3
icu4c/source/i18n/unicode/numfmt.h
View file

@ -102,9 +102,6 @@ class StringEnumeration;
* nf->parse(myString, result, success);
* \endcode
* </pre>
* Note that while lenient parsing is on by default for the Format base class,
* it is off by default for NumberFormat and its subclasses.
* <P>
* Use createInstance to get the normal number format for that country.
* There are other static factory methods available. Use getCurrency
* to get the currency number format for that country. Use getPercent

12
icu4c/source/i18n/unicode/rbnf.h
View file

@ -927,7 +927,7 @@ public:
* @see #setLenient
* @stable ICU 2.0
*/
virtual UBool isLenient(void) const;
virtual inline UBool isLenient(void) const;
#endif
@ -994,6 +994,7 @@ private:
Locale locale;
Collator* collator;
DecimalFormatSymbols* decimalFormatSymbols;
UBool lenient;
UnicodeString* lenientParseRules;
LocalizationInfo* localizations;
@ -1035,6 +1036,15 @@ RuleBasedNumberFormat::parse(const UnicodeString& text, Formattable& result, UEr
NumberFormat::parse(text, result, status);
}
#if !UCONFIG_NO_COLLATION
inline UBool
RuleBasedNumberFormat::isLenient(void) const {
return lenient;
}
#endif
inline NFRuleSet*
RuleBasedNumberFormat::getDefaultRuleSet() const {
return defaultRuleSet;

17
icu4c/source/test/intltest/dcfmapts.cpp
View file

@ -189,23 +189,6 @@ void IntlTestDecimalFormatAPI::testAPI(/*char *par*/)
logln((UnicodeString)"Testing getters and setters");
// check default value of lenient
if ( def.isLenient() ) {
errln("ERROR: isLenient() not FALSE by default for DecimalFormat");
}
DecimalFormat *defMod = new DecimalFormat(def);
// check setting & comparison of lenient
if (defMod != NULL) {
if ( !(*defMod == def) ) {
errln("ERROR: operator == is FALSE, should be TRUE afer copy");
}
defMod->setLenient(!def.isLenient());
if( defMod->isLenient() == def.isLenient()) {
errln("ERROR: isLenient() after setLenient(!isLenient()) failed");
} // don't check that == fails, not relevant for subclass that does not check lenient
delete defMod;
}
const DecimalFormatSymbols *syms = pat.getDecimalFormatSymbols();
DecimalFormatSymbols *newSyms = new DecimalFormatSymbols(*syms);
def.setDecimalFormatSymbols(*newSyms);

11
icu4c/source/test/intltest/dtfmapts.cpp
View file

@ -218,16 +218,7 @@ if (fr != NULL && it != NULL && de != NULL)
name = locales[i].getName();
logln(name);
}
// check default value of lenient, and setting/testing
if ( !fr->isLenient() ) {
errln("ERROR: isLenient() not TRUE by default for DateFormat");
} else {
fr->setLenient(FALSE);
if ( fr->isLenient() ) {
errln("ERROR: isLenient() after setLenient(FALSE) failed");
}
}
fr->setLenient(it->isLenient());
if(fr->isLenient() != it->isLenient()) {
errln("ERROR: setLenient() failed");

13
icu4c/source/test/intltest/dtifmtts.cpp
View file

@ -114,14 +114,9 @@ void DateIntervalFormatTest::testAPI() {
return;
}
// not deleted, test clone
// check default value of lenient
if ( !dtitvfmt->isLenient() ) {
errln("ERROR: isLenient() not TRUE by default for DateIntervalFormat");
}
// ====== Test clone() and setLenient() (latter from Format superclass)
// ====== Test clone()
status = U_ZERO_ERROR;
logln("Testing DateIntervalFormat clone");
@ -130,11 +125,7 @@ void DateIntervalFormatTest::testAPI() {
dataerrln("ERROR: clone failed");
}
another->setLenient(!dtitvfmt->isLenient());
if( another->isLenient() == dtitvfmt->isLenient()) {
errln("ERROR: isLenient() after setLenient(!isLenient()) failed");
} // don't check that == fails, not relevant for subclass that does not check lenient
// ====== Test getDateIntervalInfo, setDateIntervalInfo, adoptDateIntervalInfo
status = U_ZERO_ERROR;
logln("Testing DateIntervalFormat getDateIntervalInfo");

14
icu4c/source/test/intltest/itrbnf.cpp
View file

@ -152,12 +152,6 @@ IntlTestRBNF::TestAPI() {
}
logln("RBNF API test starting");
// check default value of lenient
if ( formatter->isLenient() ) {
errln("ERROR: isLenient() not FALSE by default for RuleBasedNumberFormat");
}
// test clone
{
logln("Testing Clone");
@ -166,14 +160,6 @@ IntlTestRBNF::TestAPI() {
if(!(*rbnfClone == *formatter)) {
errln("Clone should be semantically equivalent to the original!");
}
rbnfClone->setLenient(!formatter->isLenient());
if( rbnfClone->isLenient() == formatter->isLenient()) {
errln("ERROR: isLenient() after setLenient(!isLenient()) failed");
#if !UCONFIG_NO_COLLATION
} else if (*rbnfClone == *formatter) {
errln("ERROR: operator == is TRUE, should be FALSE if isLenient() differs");
#endif
}
delete rbnfClone;
} else {
errln("Cloning failed!");

14
icu4c/source/test/intltest/nmfmapts.cpp
View file

@ -194,20 +194,6 @@ if (fr != NULL && def != NULL)
logln(name);
}
// check default value of lenient, and setting/testing
if ( def->isLenient() ) {
errln("ERROR: isLenient() not FALSE by default for NumberFormat");
} else {
def->setLenient(TRUE);
if ( !def->isLenient() ) {
errln("ERROR: isLenient() after setLenient(TRUE) failed");
}
}
def->setLenient(fr->isLenient());
if(def->isLenient() != fr->isLenient()) {
errln("ERROR: setLenient() failed");
}
fr->setParseIntegerOnly( def->isParseIntegerOnly() );
if(fr->isParseIntegerOnly() != def->isParseIntegerOnly() ) {
errln("ERROR: setParseIntegerOnly() failed");

19
icu4c/source/test/intltest/sdtfmtts.cpp
View file

@ -175,25 +175,6 @@ void IntlTestSimpleDateFormatAPI::testAPI(/*char *par*/)
logln("Testing getters and setters");
// check default value of lenient
if ( !def.isLenient() ) {
errln("ERROR: isLenient() not TRUE by default for SimpleDateFormat");
}
SimpleDateFormat *defMod = new SimpleDateFormat(def);
// check setting & comparison of lenient
if (defMod != NULL) {
if ( !(*defMod == def) ) {
errln("ERROR: operator == is FALSE, should be TRUE after copy");
}
defMod->setLenient(!def.isLenient());
if( defMod->isLenient() == def.isLenient()) {
errln("ERROR: isLenient() after setLenient(!isLenient()) failed");
} else if ( *defMod == def ) {
errln("ERROR: operator == is TRUE, should be FALSE if isLenient() differs");
}
delete defMod;
}
const DateFormatSymbols *syms = pat.getDateFormatSymbols();
if(!syms) {
errln("Couldn't obtain DateFormatSymbols. Quitting test!");