From 842618f2711e7407e4c69c79301f6c9a767f9514 Mon Sep 17 00:00:00 2001
From: pme
I/O sentry ctor/dtor They can perform additional work - than the minimum required. I don't think we're currently taking +
[27.6.1.1.2],
+ [27.6.2.3] The I/O sentry ctor and dtor can perform
+ additional work than the minimum required. We are not currently taking
advantage of this yet.
[27.7.1.3]/16,
diff --git a/libstdc++-v3/docs/html/27_io/howto.html b/libstdc++-v3/docs/html/27_io/howto.html
index b4a5925..d7a984e 100644
--- a/libstdc++-v3/docs/html/27_io/howto.html
+++ b/libstdc++-v3/docs/html/27_io/howto.html
@@ -34,6 +34,7 @@
To minimize the time you have to wait on the compiler, it's good to + only include the headers you really need. Many people simply include + <iostream> when they don't need to -- and that can penalize + your runtime as well. Here are some tips on which header to use + for which situations, starting with the simplest. +
+<iosfwd> should be included whenever you simply + need the name of an I/O-related class, such as + "ofstream" or "basic_streambuf". Like the name + implies, these are forward declarations. (A word to all you fellow + old school programmers: trying to forward declare classes like + "class istream;" won't work. Look in the iosfwd header if + you'd like to know why.) For example, +
++ #include <iosfwd> + + class MyClass + { + .... + std::ifstream input_file; + }; + + extern std::ostream& operator<< (std::ostream&, MyClass&); ++
<ios> declares the base classes for the entire + I/O stream hierarchy, std::ios_base and std::basic_ios<charT>, the + counting types std::streamoff and std::streamsize, the file + positioning type std::fpos, and the various manipulators like + std::hex, std::fixed, std::noshowbase, and so forth. +
+The ios_base class is what holds the format flags, the state flags, + and the functions which change them (setf(), width(), precision(), + etc). You can also store extra data and register callback functions + through ios_base, but that has been historically underused. Anything + which doesn't depend on the type of characters stored is consolidated + here. +
+The template class basic_ios is the highest template class in the + hierarchy; it is the first one depending on the character type, and + holds all general state associated with that type: the pointer to the + polymorphic stream buffer, the facet information, etc. +
+<streambuf> declares the template class + basic_streambuf, and two standard instantiations, streambuf and + wstreambuf. If you need to work with the vastly useful and capable + stream buffer classes, e.g., to create a new form of storage + transport, this header is the one to include. +
+<istream>/<ostream> are + the headers to include when you are using the >>/<< + interface, or any of the other abstract stream formatting functions. + For example, +
++ #include <istream> + + std::ostream& operator<< (std::ostream& os, MyClass& c) + { + return os << c.data1() << c.data2(); + } ++
The std::istream and std::ostream classes are the abstract parents of + the various concrete implementations. If you are only using the + interfaces, then you only need to use the appropriate interface header. +
+<iomanip> provides "extractors and inserters
+ that alter information maintained by class ios_base and its dervied
+ classes," such as std::setprecision and std::setw. If you need
+ to write expressions like os << setw(3);
or
+ is >> setbase(8);
, you must include <iomanip>.
+
<sstream>/<fstream> + declare the six stringstream and fstream classes. As they are the + standard concrete descendants of istream and ostream, you will already + know about them. +
+Finally, <iostream> provides the eight standard + global objects (cin, cout, etc). To do this correctly, this header + also provides the contents of the <istream> and <ostream> + headers, but nothing else. The contents of this header look like +
++ #include <ostream> + #include <istream> + + namespace std + { + extern istream cin; + extern ostream cout; + .... + + // this is explained below + static ios_base::Init __foo; // not its real name + } ++
Now, the runtime penalty mentioned previously: the global objects + must be initialized before any of your own code uses them; this is + guaranteed by the standard. Like any other global object, they must + be initialized once and only once. This is typically done with a + construct like the one above, and the nested class ios_base::Init is + specified in the standard for just this reason. +
+How does it work? Because the header is included before any of your + code, the __foo object is constructed before any of + your objects. (Global objects are built in the order in which they + are declared, and destroyed in reverse order.) The first time the + constructor runs, the eight stream objects are set up. +
+The static
keyword means that each object file compiled
+ from a source file containing <iostream> will have its own
+ private copy of __foo. There is no specified order
+ of construction across object files (it's one of those pesky NP
+ problems that make life so interesting), so one copy in each object
+ file means that the stream objects are guaranteed to be set up before
+ any of your code which uses them could run, thereby meeting the
+ requirements of the standard.
+
The penalty, of course, is that after the first copy of + __foo is constructed, all the others are just wasted + processor time. The time spent is merely for an increment-and-test + inside a function call, but over several dozen or hundreds of object + files, that time can add up. (It's not in a tight loop, either.) +
+The lesson? Only include <iostream> when you need to use one of + the standard objects in that source file; you'll pay less startup + time. Only include the header files you need to in general; your + compile times will go down when there's less parsing work to do. +
+ diff --git a/libstdc++-v3/docs/html/documentation.html b/libstdc++-v3/docs/html/documentation.html index a4a79e3..8c40fb0 100644 --- a/libstdc++-v3/docs/html/documentation.html +++ b/libstdc++-v3/docs/html/documentation.html @@ -66,7 +66,7 @@Doc. no. | -J16/02-0027 = WG21 N1369 | +J16/02-0048 = WG21 N1390 |
Date: | -10 May 2002 | +10 Sep 2002 |
Project: | @@ -17,10 +17,10 @@||
Reply to: | -Matt Austern <austern@research.att.com> | +Matt Austern <austern@apple.com> |
Reference ISO/IEC IS 14882:1998(E)
Also see:
Public information as to how to obtain a copy of the C++ Standard, join the standards committee, submit an issue, or comment on an issue can be found in the C++ FAQ at http://www.research.att.com/~austern/csc/faq.html. - Public discussion of C++ Standard related issues occurs on news:comp.std.c++. + Public discussion of C++ Standard related issues occurs on news:comp.std.c++.
For committee members, files available on the committee's private @@ -88,6 +88,10 @@ directory as the issues list files.
-Section: 17.4.4.3 [lib.global.functions] Status: Open Submitter: Dave Abrahams Date: 01 Apr 2000
+Section: 17.4.4.3 [lib.global.functions] Status: Review Submitter: Dave Abrahams Date: 01 Apr 2000Are algorithms in std:: allowed to use other algorithms without qualification, so functions in user namespaces might be found through Koenig lookup?
For example, a popular standard library implementation includes this @@ -1586,13 +1590,14 @@ should have any effect.]
[Curaçao: An LWG-subgroup spent an afternoon working on issues 225, 226, and 229. Their conclusion was that the issues should be -separated into an LWG portion (Howard will write a proposal), and a +separated into an LWG portion (Howard's paper, N1387=02-0045), and a EWG portion (Dave will write a proposal). The LWG and EWG had -(separate) discussions of this plan the next day.]
+(separate) discussions of this plan the next day. The proposed +resolution for this issue is in accordance with Howard's paper.]-Section: 17.4.3.1 [lib.reserved.names] Status: Open Submitter: Dave Abrahams Date: 01 Apr 2000
+Section: 17.4.3.1 [lib.reserved.names] Status: Review Submitter: Dave Abrahams Date: 01 Apr 2000The issues are:
1. How can a 3rd party library implementor (lib1) write a version of a standard algorithm which is specialized to work with his own class template?
@@ -1677,6 +1682,9 @@ not provide an operator<< for std::pair<>.Proposed resolution:
+Adopt the wording in the Customization Points section of +Howard Hinnant's paper, N1387=02-0045.
+[Tokyo: Summary, "There is no conforming way to extend std::swap for user defined templates." The LWG agrees that there is a problem. Would like more information before @@ -1734,13 +1742,14 @@ try to put together a proposal before the next meeting.]
[Curaçao: An LWG-subgroup spent an afternoon working on issues 225, 226, and 229. Their conclusion was that the issues should be -separated into an LWG portion (Howard will write a proposal), and a +separated into an LWG portion (Howard's paper, N1387=02-0045), and a EWG portion (Dave will write a proposal). The LWG and EWG had -(separate) discussions of this plan the next day.]
+(separate) discussions of this plan the next day. The proposed +resolution is the one proposed by Howard.]-Section: 17.4.1.1 [lib.contents] Status: Open Submitter: Steve Clamage Date: 19 Apr 2000
+Section: 17.4.1.1 [lib.contents] Status: Review Submitter: Steve Clamage Date: 19 Apr 2000Throughout the library chapters, the descriptions of library entities refer to other library entities without necessarily qualifying the names.
@@ -1784,13 +1793,15 @@ but that the wording may not be clear enough to fall under[Curaçao: An LWG-subgroup spent an afternoon working on issues 225, 226, and 229. Their conclusion was that the issues should be -separated into an LWG portion (Howard will write a proposal), and a +separated into an LWG portion (Howard's paper, N1387=02-0045), and a EWG portion (Dave will write a proposal). The LWG and EWG had -(separate) discussions of this plan the next day.]
+(separate) discussions of this plan the next day. This paper resolves +issues 225 and 226. In light of that resolution, the proposed +resolution for the current issue makes sense.]-Section: 22.2.2.2.2 [lib.facet.num.put.virtuals] Status: Open Submitter: James Kanze, Stephen Clamage Date: 25 Apr 2000
+Section: 22.2.2.2.2 [lib.facet.num.put.virtuals] Status: Review Submitter: James Kanze, Stephen Clamage Date: 25 Apr 2000What is the following program supposed to output?
#include <iostream> @@ -1831,24 +1842,31 @@ etc. Plus, of course, if precision == 0 and flags & floatfield == of the anomalies of printf:-).+Proposed resolution:
-In 22.2.2.2.2 [lib.facet.num.put.virtuals], paragraph 11, change -"if (flags & fixed) != 0" to -"if (flags & floatfield) == fixed || - (flags & floatfield) == scientific" +Replace 22.2.2.2.2 [lib.facet.num.put.virtuals], paragraph 11, with the following +sentence:
++For conversion from a floating-point type, +str.precision() is specified in the conversion +specification. +Rationale:
The floatfield determines whether numbers are formatted as if with %f, %e, or %g. If the fixed bit is set, it's %f, if scientific it's %e, and if both bits are set, or -neither, it's %e.
+neither, it's %g.Turning to the C standard, a precision of 0 is meaningful -for %f and %e, but not for %g: for %g, precision 0 is taken -to be the same as precision 1.
-The proposed resolution has the effect that the output of -the above program will be "1e+00".
- -[Curaçao: Howard will send Matt improved wording dealing with -case not covered by current PR.]
+for %f and %e. For %g, precision 0 is taken to be the same as +precision 1. +The proposed resolution has the effect that if neither +fixed nor scientific is set we'll be +specifying a precision of 0, which will be internally +turned into 1. There's no need to call it out as a special +case.
+The output of the above program will be "1e+00".
+ +[Post-Curaçao: Howard provided improved wording covering the case +where precision is 0 and mode is %g.]
233. Insertion hints in associative containers
@@ -2354,6 +2372,28 @@ throw, the string must compare equal to the argument.
(Not all of these options are mutually exclusive.)
Proposed resolution:
+NAD/Future
+Rationale:
+ +Throwing a bad_alloc while trying to construct a message for another +exception-derived class is not necessarily a bad thing. And the +bad_alloc constructor already has a no throw spec on it (18.4.2.1).
+ ++The copy constructors of all exception-derived classes already have a +no throw spec. Reference 18.6.1, 19.1 and 15.4/13. +
+ +Future:
+ +All involved would like to see const char* constructors added, but +this should probably be done for C++0X as opposed to a DR.
+ +I believe the no throw specs currently decorating these functions +could be improved by some kind of static no throw spec checking +mechanism (in a future C++ language). As they stand, the copy +constructors might fail via a call to unexpected. I think what is +intended here is that the copy constructors can't fail.
[Toronto: some LWG members thought this was merely a QoI issue, but most believed that it was at least a borderline defect. There was @@ -2361,11 +2401,9 @@ more support for nonnormative advice to implementors than for a normative change.]
[Redmond: discussed, without definite conclusion. Most LWG -members thought there was a real defect lurking here. A small group -(Herb, Kevlin, Howard, Martin, Dave) will try to make a -recommendation.]
- -[Curaçao: Howard will nag the others to work on a recommendation.]
+members thought there was a real defect lurking here. The above +proposed resolution/rationale is from Howard, Herb, Kevlin, Martin, +and Dave.]
258. Missing allocator requirement
@@ -2553,7 +2591,7 @@ desirable to provide this feature in a different way.
282. What types does numpunct grouping refer to?
-Section: 22.2.2.2.2 [lib.facet.num.put.virtuals] Status: Open Submitter: Howard Hinnant Date: 5 Dec 2000
+Section: 22.2.2.2.2 [lib.facet.num.put.virtuals] Status: Review Submitter: Howard Hinnant Date: 5 Dec 2000Paragraph 16 mistakenly singles out integral types for inserting thousands_sep() characters. This conflicts with the syntax for floating @@ -2593,8 +2631,8 @@ floating-point input even though this is unambiguously required by the standard. ]
-[Curaçao: Howard will email Bill and other implementors to try to -move the issue forward.]
+[Post-Curaçao: the above proposed resolution is the consensus of +Howard, Bill, Pete, Benjamin, Nathan, Dietmar, Boris, and Martin.]
283. std::replace() requirement incorrect/insufficient
@@ -3099,22 +3137,42 @@ note in p24 (below) is that x be empty after the merge which is surely unintended in this case.
Proposed resolution:
+In 23.2.2.4 [lib.list.ops], replace paragraps 23-25 with:
+-Change 23.2.2.4, p23 to: +23 Effects: if (&x == this) does nothing; otherwise, merges the two +sorted ranges [begin(), end()) and [x.begin(), x.end()). The result +is a range in which the elements will be sorted in non-decreasing +order according to the ordering defined by comp; that is, for every +iterator i in the range other than the first, the condition comp(*i, +*(i - 1)) will be false.
--Effects: If &x == this, does nothing; otherwise, merges the -argument list into the list. + +-+24 Notes: Stable: if (&x != this), then for equivalent elements in the +two original ranges, the elements from the original range [begin(), +end()) always precede the elements from the original range [x.begin(), +x.end()). If (&x != this) the range [x.begin(), x.end()) is empty +after the merge. +
+ ++25 Complexity: At most size() + x.size() - 1 applications of comp if +(&x ! = this); otherwise, no applications of comp are performed. If +an exception is thrown other than by a comparison there are no +effects. +
+[Copenhagen: The proposed resolution does not fix all of the -problems in 23.2.2.4 [lib.list.ops], p22-25. Three different +
[Copenhagen: The original proposed resolution did not fix all of +the problems in 23.2.2.4 [lib.list.ops], p22-25. Three different paragraphs (23, 24, 25) describe the effects of merge. Changing p23, without changing the other two, appears to introduce contradictions. Additionally, "merges the argument list into the list" is excessively vague.]
-[Curaçao: Robert Klarer volunteers to work on this issue.]
+[Post-Curaçao: Robert Klarer provided new wording.]
304. Must *a return an lvalue when a is an input iterator?
@@ -5283,6 +5341,648 @@ rationale. basic_filebuf<charT,traits>* rdbuf(); const basic_filebuf<charT,traits>* rdbuf() const;
+Section: 25.2.7 [lib.alg.remove] Status: New Submitter: Anthony Williams Date: 13 May 2002
++remove_copy and remove_copy_if (25.2.7 [lib.alg.remove]) permit their +input range to be marked with Input Iterators. However, since two +operations are required against the elements to copy (comparison and +assigment), when the input range uses Input Iterators, a temporary +copy must be taken to avoid dereferencing the iterator twice. This +therefore requires the value type of the InputIterator to be +CopyConstructible. If the iterators are at least Forward Iterators, +then the iterator can be dereferenced twice, or a reference to the +result maintained, so the temporary is not required. +
+Proposed resolution:
++Add "If InputIterator does not meet the requirements of forward +iterator, then the value type of InputIterator must be copy +constructible. Otherwise copy constructible is not required." to +25.2.7 [lib.alg.remove] paragraph 6. +
++Section: 21.3.5.6 [lib.string::replace] Status: New Submitter: Beman Dawes Date: 3 Jun 2002
++21.3.5.6 [lib.string::replace] basic_string::replace, second +signature, given in paragraph 1, has two "Throws" paragraphs (3 and +5). +
+ ++In addition, the second "Throws" paragraph (5) includes specification +(beginning with "Otherwise, the function replaces ...") that should be +part of the "Effects" paragraph. +
+Proposed resolution:
++Section: 27.3 [lib.iostream.objects] Status: New Submitter: Ruslan Abdikeev Date: 8 Jul 2002
++Is it safe to use standard iostream objects from constructors of +static objects? Are standard iostream objects constructed and are +their associations established at that time? +
+ +Surpisingly enough, Standard does NOT require that.
+ ++27.3/2 [lib.iostream.objects] guarantees that standard iostream +objects are constructed and their associations are established before +the body of main() begins execution. It also refers to ios_base::Init +class as the panacea for constructors of static objects. +
+ ++However, there's nothing in 27.3 [lib.iostream.objects], +in 27.4.2 [lib.ios.base], and in 27.4.2.1.6 [lib.ios::Init], +that would require implementations to allow access to standard +iostream objects from constructors of static objects. +
+ +Details:
+ +Core text refers to some magic object ios_base::Init, which will +be discussed below:
+ ++ "The [standard iostream] objects are constructed, and their + associations are established at some time prior to or during + first time an object of class basic_ios<charT,traits>::Init + is constructed, and in any case before the body of main + begins execution." (27.3/2 [lib.iostream.objects]) ++ +
+The first non-normative footnote encourages implementations +to initialize standard iostream objects earlier than required. +
+ +However, the second non-normative footnote makes an explicit +and unsupported claim:
+ ++ "Constructors and destructors for static objects can access these + [standard iostream] objects to read input from stdin or write output + to stdout or stderr." (27.3/2 footnote 265 [lib.iostream.objects]) ++ +
+The only bit of magic is related to that ios_base::Init class. AFAIK, +the rationale behind ios_base::Init was to bring an instance of this +class to each translation unit which #included <iostream> or +related header. Such an inclusion would support the claim of footnote +quoted above, because in order to use some standard iostream object it +is necessary to #include <iostream>. +
+ ++However, while Standard explicitly describes ios_base::Init as +an appropriate class for doing the trick, I failed to found a +mention of an _instance_ of ios_base::Init in Standard. +
+Proposed resolution:
++At the end of header <iostream> synopsis in 27.3 [lib.iostream.objects] +
+ ++ namespace std + { + ... extern istream cin; ... ++ +
add the following lines
+ ++ namespace + { + ios_base::Init <some_implementation_defined_name>; + } + } ++
+Section: 27.6.1.3 [lib.istream.unformatted] Status: New Submitter: Ray Lischner Date: 15 Jul 2002
+Defect report for description of basic_istream::get (section 27.6.1.3 [lib.istream.unformatted]), paragraph 15. The description for the get function +with the following signature:
+ ++ basic_istream<charT,traits>& get(basic_streambuf<char_type,traits>& + sb); ++ +
is incorrect. It reads
+ ++ Effects: Calls get(s,n,widen('\n')) ++ +
which I believe should be:
+ ++ Effects: Calls get(sb,widen('\n')) ++
Proposed resolution:
+Change the Effects paragraph to:
++ Effects: Calls get(sb,widen('\n')) ++
+Section: 23.1 [lib.container.requirements] Status: New Submitter: Frank Compagner Date: 20 Jul 2002
++The requirements for multiset and multimap containers (23.1 +[lib.containers.requirements], 23.1.2 [lib.associative.reqmnts], +23.3.2 [lib.multimap] and 23.3.4 [lib.multiset]) make no mention of +the stability of the required (mutating) member functions. It appears +the standard allows these functions to reorder equivalent elements of +the container at will, yet the pervasive red-black tree implementation +appears to provide stable behaviour. +
+ +This is of most concern when considering the behaviour of erase(). +A stability requirement would guarantee the correct working of the +following 'idiom' that removes elements based on a certain predicate +function. +
+ ++ multimap<int, int> m; + multimap<int, int>::iterator i = m.begin(); + while (i != m.end()) { + if (pred(i)) + m.erase (i++); + else + ++i; + } ++ +
+Although clause 23.1.2/8 guarantees that i remains a valid iterator +througout this loop, absence of the stability requirement could +potentially result in elements being skipped. This would make +this code incorrect, and, furthermore, means that there is no way +of erasing these elements without iterating first over the entire +container, and second over the elements to be erased. This would +be unfortunate, and have a negative impact on both performance and +code simplicity. +
+ ++If the stability requirement is intended, it should be made explicit +(probably through an extra paragraph in clause 23.1.2). +
++If it turns out stability cannot be guaranteed, i'd argue that a +remark or footnote is called for (also somewhere in clause 23.1.2) to +warn against relying on stable behaviour (as demonstrated by the code +above). If most implementations will display stable behaviour, any +problems emerging on an implementation without stable behaviour will +be hard to track down by users. This would also make the need for an +erase_if() member function that much greater. +
+ +This issue is somewhat related to LWG issue 130.
+Proposed resolution:
++Section: 17.4.4.8 [lib.res.on.exception.handling], 18.6.1 [lib.exception], Status: New Submitter: Randy Maddox Date: 22 Jul 2002
+ +Paragraph 3 under clause 17.4.4.8 [lib.res.on.exception.handling], Restrictions on +Exception Handling, states that "Any other functions defined in the +C++ Standard Library that do not have an exception-specification may +throw implementation-defined exceptions unless otherwise specified." +This statement is followed by a reference to footnote 178 at the +bottom of that page which states, apparently in reference to the C++ +Standard Library, that "Library implementations are encouraged (but +not required) to report errors by throwing exceptions from (or derived +from) the standard exceptions."
+ +These statements appear to be in direct contradiction to clause +18.6.1 [lib.exception], which states "The class exception defines the +base class for the types of objects thrown as exceptions by the C++ +Standard library components ...".
+ +Is this inconsistent?
+ +Proposed resolution:
++Section: 27.6.1.2.1 [lib.istream.formatted.reqmts], 27.6.2.5.1 [lib.ostream.formatted.reqmts] Status: New Submitter: Keith Baker Date: 23 Jul 2002
+ ++In 27.6.1.2.1 [lib.istream.formatted.reqmts] and 27.6.2.5.1 [lib.ostream.formatted.reqmts] +(exception()&badbit) != 0 is used in testing for rethrow, yet +exception() is the constructor to class std::exception in 18.6.1 [lib.exception] that has no return type. Should member function +exceptions() found in 27.4.4 [lib.ios] be used instead? +
+ +Proposed resolution:
++
++Section: 22.2.6.3.1 [lib.locale.moneypunct.members], 22.2.6.3.2 [lib.locale.moneypunct.virtuals] Status: New Submitter: Ray Lischner Date: 8 Aug 2002
++In section 22.2.6.3.1 [lib.locale.moneypunct.members], frac_digits() returns type +"int". This implies that frac_digits() might return a negative value, +but a negative value is nonsensical. It should return "unsigned". +
+ ++Similarly, in section 22.2.6.3.2 [lib.locale.moneypunct.virtuals], do_frac_digits() +should return "unsigned". +
+ +Proposed resolution:
++Section: 27.7.1.3 [lib.stringbuf.virtuals] Status: New Submitter: Ray Lischner Date: 14 Aug 2002
++In Section 27.7.1.3 [lib.stringbuf.virtuals]: Table 90, Table 91, and paragraph +14 all contain references to "basic_ios::" which should be +"ios_base::". +
+Proposed resolution:
++Change all references to "basic_ios" in Table 90, Table 91, and +paragraph 14 to "ios_base". +
++Section: 27.7.1.3 [lib.stringbuf.virtuals] Status: New Submitter: Ray Lischner Date: 14 Aug 2002
++In Section 27.7.1.3 [lib.stringbuf.virtuals], Table 90, the implication is that +the four conditions should be mutually exclusive, but they are not. +The first two cases, as written, are subcases of the third. I think it +would be clearer if the conditions were rewritten as follows: +
+ +++ ++ (which & (ios_base::in|ios_base::out)) == ios_base::in +
+ ++ (which & (ios_base::in|ios_base::out)) == ios_base::out +
+ ++ (which & (ios_base::in|ios_base::out)) == +(ios_base::in|ios_base::out) + and way == either ios_base::beg or ios_base::end +
+ +Otherwise
+
+As written, it is unclear what should be the result if cases 1 & 2 +are true, but case 3 is false, e.g., +
+ ++ seekoff(0, ios_base::cur, ios_base::in | ios_base::out) ++ +
Proposed resolution:
++Section: 21.3.5.4 [lib.string::insert] Status: New Submitter: Ray Lischner Date: 16 Aug 2002
++Section 21.3.5.4 [lib.string::insert], paragraph 4, contains the following, +"Then throws length_error if size() >= npos - rlen." +
+ ++Related to DR 83, this sentence should probably be removed. +
+Proposed resolution:
++Section: 22.1.1 [lib.locale] Status: New Submitter: Martin Sebor Date: 6 Sep 2002
++I think there is a problem with 22.1.1, p6 which says that +
++ -6- An instance of locale is immutable; once a facet reference + is obtained from it, that reference remains usable as long + as the locale value itself exists. ++
+and 22.1.1.2, p4: +
++ const locale& operator=(const locale& other) throw(); + + -4- Effects: Creates a copy of other, replacing the current value. ++
+How can a reference to a facet obtained from a locale object remain +valid after an assignment that clearly must replace all the facets +in the locale object? Imagine a program such as this +
++ std::locale loc ("de_DE"); + const std::ctype<char> &r0 = std::use_facet<std::ctype<char> >(loc); + loc = std::locale ("en_US"); + const std::ctype<char> &r1 = std::use_facet<std::ctype<char> >(loc); ++
+Is r0 really supposed to be preserved and destroyed only when loc goes +out of scope? +
+Proposed resolution:
++Suggest to replace 22.1.1 [lib.locale], p6 with +
++ -6- Unless assigned a new value, locale objects are immutable; + once a facet reference is obtained from it, that reference + remains usable as long as the locale object itself exists + or until the locale object is assigned the value of another, + distinct locale object. ++
+Section: 22.2.1.1.2 [lib.locale.ctype.virtuals] Status: New Submitter: Martin Sebor Date: 6 Sep 2002
++The last sentence in 22.2.1.1.2, p11 below doesn't seem to make sense. +
++ charT do_widen (char c) const; + + -11- Effects: Applies the simplest reasonable transformation from + a char value or sequence of char values to the corresponding + charT value or values. The only characters for which unique + transformations are required are those in the basic source + character set (2.2). For any named ctype category with a + ctype<charT> facet ctw and valid ctype_base::mask value + M (is(M, c) || !ctw.is(M, do_widen(c))) is true. ++
+Shouldn't the last sentence instead read +
++ For any named ctype category with a ctype<char> facet ctc + and valid ctype_base::mask value M + (ctc.is(M, c) || !is(M, do_widen(c))) is true. ++
+I.e., if the narrow character c is not a member of a class of +characters then neither is the widened form of c. (To paraphrase +footnote 224.) +
+Proposed resolution:
++Replace the last sentence of 22.2.1.1.2 [lib.locale.ctype.virtuals], p11 with the +following text: +
++ For any named ctype category with a ctype<char> facet ctc + and valid ctype_base::mask value M + (ctc.is(M, c) || !is(M, do_widen(c))) is true. ++
+Section: 22.2.1.5.2 [lib.locale.codecvt.virtuals] Status: New Submitter: Martin Sebor Date: 6 Sep 2002
++Tables 53 and 54 in 22.2.1.5.2 [lib.locale.codecvt.virtuals] are both titled "convert +result values," when surely "do_in/do_out result values" must have +been intended for Table 53 and "do_unshift result values" for Table +54. +
++Table 54, row 3 says that the meaning of partial is "more characters +needed to be supplied to complete termination." The function is not +supplied any characters, it is given a buffer which it fills with +characters or, more precisely, destination elements (i.e., an escape +sequence). So partial means that space for more than (to_limit - to) +destination elements was needed to terminate a sequence given the +value of state. +
+Proposed resolution:
++Change the title of Table 53 to "do_in/do_out result values" and +the title of Table 54 to "do_unshift result values." +
++Change the text in Table 54, row 3, under the heading Meaning to +"space for more than (to_limit - to) destination elements was +needed to terminate a sequence given the value of state." +
++Section: 22.2.1.5.2 [lib.locale.codecvt.virtuals] Status: New Submitter: Martin Sebor Date: 6 Sep 2002
++All but one codecvt member functions that take a state_type argument +list as one of their preconditions that the state_type argument have +a valid value. However, according to 22.2.1.5.2, p6, +codecvt::do_unshift() is the only codecvt member that is supposed to +return error if the state_type object is invalid. +
+ ++It seems to me that the treatment of state_type by all codecvt member +functions should be the same and the current requirements should be +changed. Since the detection of invalid state_type values may be +difficult in general or computationally expensive in some specific +cases, I propose the following: +
+Proposed resolution:
++Add a new paragraph before 22.2.1.5.2, p5, and after the function +declaration below +
++ result do_unshift(stateT& state, + externT* to, externT* to_limit, externT*& to_next) const; ++
+as follows: +
++ Requires: (to <= to_end) well defined and true; state initialized, + if at the beginning of a sequence, or else equal to the result of + converting the preceding characters in the sequence. ++
+and change the text in Table 54, row 4, under the heading Meaning +from +
++ state has invalid value ++
+to +
++ an unspecified error has occurred ++
+The return value of error should allow implementers to detect and +report invalid state values but shouldn't require it, hence the +word "unspecified" in the new wording. +
++Section: 22.2.1.5 [lib.locale.codecvt] Status: New Submitter: Martin Sebor Date: 30 Aug 2002
++It seems that the descriptions of codecvt do_in() and do_out() leave +sufficient room for interpretation so that two implementations of +codecvt may not work correctly with the same filebuf. Specifically, +the following seems less than adequately specified: +
+ ++Finally, the conditions described at the end of 22.2.1.5.2 [lib.locale.codecvt.virtuals], p4 don't seem to be possible: +
++ "A return value of partial, if (from_next == from_end), + indicates that either the destination sequence has not + absorbed all the available destination elements, or that + additional source elements are needed before another + destination element can be produced." ++
+If the value is partial, it's not clear to me that (from_next +==from_end) could ever hold if there isn't enough room +in the destination buffer. In order for (from_next==from_end) to +hold, all characters in that range must have been successfully +converted (according to 22.2.1.5.2 [lib.locale.codecvt.virtuals], p2) and since there are no +further source characters to convert, no more room in the +destination buffer can be needed. +
++It's also not clear to me that (from_next==from_end) could ever +hold if additional source elements are needed to produce another +destination character (not element as incorrectly stated in the +text). partial is returned if "not all source characters have +been converted" according to Table 53, which also implies that +(from_next==from) does NOT hold. +
++Could it be that the intended qualifying condition was actually +(from_next != from_end), i.e., that the sentence was supposed +to read +
++ "A return value of partial, if (from_next != from_end),..." ++
+which would make perfect sense, since, as far as I understand it, +partial can only occur if (from_next != from_end)? +
+Proposed resolution:
++To address these issues, I propose that paragraphs 2, 3, and 4 +be rewritten as follows. The proposal incorporates the accepted +resolution of lwg issue 19. +
++-2- Effects: Converts characters in the range of source elements + [from, from_end), placing the results in sequential positions + starting at destination to. Converts no more than (from_endfrom) + source elements, and stores no more than (to_limitto) + destination elements. + + Stops if it encounters a sequence of source elements it cannot + convert to a valid destination character. It always leaves the + from_next and to_next pointers pointing one beyond the last + element successfully converted. + + [Note: If returns noconv, internT and externT are the same type + and the converted sequence is identical to the input sequence + [from, from_next). to_next is set equal to to, the value of + state is unchanged, and there are no changes to the values in + [to, to_limit). --end note] + +-3- Notes: Its operations on state are unspecified. + [Note: This argument can be used, for example, to maintain shift + state, to specify conversion options (such as count only), or to + identify a cache of seek offsets. --end note] + +-4- Returns: An enumeration value, as summarized in Table 53: + + Table 53 -- do_in/do_out result values + + Value Meaning + +---------+----------------------------------------------------+ + | ok | successfully completed the conversion of all | + | | complete characters in the source range | + +---------+----------------------------------------------------+ + | partial | the characters in the source range would, after | + | | conversion, require space greater than that | + | | available in the destination range | + +---------+----------------------------------------------------+ + | error | encountered either a sequence of elements in the | + | | source range forming a valid source character that | + | | could not be converted to a destination character, | + | | or a sequence of elements in the source range that | + | | could not possibly form a valid source character | + +---------+----------------------------------------------------+ + | noconv | internT and externT are the same type, and input | + | | sequence is identical to converted sequence | + +---------+----------------------------------------------------+ + + A return value of partial, i.e., if (from_next != from_end), + indicates that either the destination sequence has not absorbed + all the available destination elements, or that additional + source elements are needed before another destination character + can be produced. +
----- End of document -----