From d40ad2cd57569f6a2924df9716bb31fa50a24629 Mon Sep 17 00:00:00 2001 From: paolo Date: Fri, 24 Oct 2003 12:21:18 +0000 Subject: [PATCH] 2003-10-24 Paolo Carlini * include/bits/locale_facets.tcc (money_get::do_get(..., string_type&): Minor tweak to the previous commit. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@72892 138bc75d-0d04-0410-961f-82ee72b054a4 --- libstdc++-v3/ChangeLog | 5 ++++ libstdc++-v3/include/bits/locale_facets.tcc | 45 ++++++++++++++--------------- 2 files changed, 26 insertions(+), 24 deletions(-) diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 7842ab5..2b96b00 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,6 +1,11 @@ 2003-10-24 Paolo Carlini * include/bits/locale_facets.tcc (money_get::do_get(..., + string_type&): Minor tweak to the previous commit. + +2003-10-24 Paolo Carlini + + * include/bits/locale_facets.tcc (money_get::do_get(..., string_type&): Disregard the previous commit: doesn't hurt but doesn't accomplish anything useful either. This is the right one, speeding up greatly the function in case of early fail. diff --git a/libstdc++-v3/include/bits/locale_facets.tcc b/libstdc++-v3/include/bits/locale_facets.tcc index f025c51..5e8128e 100644 --- a/libstdc++-v3/include/bits/locale_facets.tcc +++ b/libstdc++-v3/include/bits/locale_facets.tcc @@ -1284,7 +1284,7 @@ namespace std __testvalid = false; } - if (__testvalid) + if (__testvalid && __tmp_units.size()) { const char_type __zero = __ctype.widen('0'); @@ -1297,33 +1297,30 @@ namespace std __tmp_units.erase(0, __only_zeros ? __tmp_units.size() - 1 : __first); } + + // 22.2.6.1.2, p4 + if (__sign.size() && __sign == __neg_sign + && __tmp_units[0] != __zero) + __tmp_units.insert(__tmp_units.begin(), __ctype.widen('-')); - if (__tmp_units.size()) + // Test for grouping fidelity. + if (__grouping.size() && __grouping_tmp.size()) { - // 22.2.6.1.2, p4 - if (__sign.size() && __sign == __neg_sign - && __tmp_units[0] != __zero) - __tmp_units.insert(__tmp_units.begin(), __ctype.widen('-')); - - // Test for grouping fidelity. - if (__grouping.size() && __grouping_tmp.size()) - { - if (!std::__verify_grouping(__grouping, __grouping_tmp)) - __testvalid = false; - } - - // Iff not enough digits were supplied after the decimal-point. - if (__testdecfound) - { - const int __frac = __intl ? __mpt.frac_digits() - : __mpf.frac_digits(); - if (__frac > 0 && __sep_pos != __frac) - __testvalid = false; - } + if (!std::__verify_grouping(__grouping, __grouping_tmp)) + __testvalid = false; + } + + // Iff not enough digits were supplied after the decimal-point. + if (__testdecfound) + { + const int __frac = __intl ? __mpt.frac_digits() + : __mpf.frac_digits(); + if (__frac > 0 && __sep_pos != __frac) + __testvalid = false; } - else - __testvalid = false; } + else + __testvalid = false; // Iff no more characters are available. if (__c == __eof) -- 2.7.4