STDCXX-978

[Eric Lemings-2]

 
Heads up.  Substantiative change in the works.  Affects the following
files in the 4.3.x branch.
 
M          677618   include/rw/_autoptr.h
M          677618   include/rw/_defs.h
M          677618   include/rw/_config-sunpro.h
M          677618   include/rw/_iterator.h
M          677618   include/rw/_iterbase.h
M          677618   include/deque
M          677618   include/loc/_locale.h
M          677618   include/set
M          677618   include/list.cc
M          677618   include/complex
M          677618   include/list
M          678525   include/bitset
M          677618   include/string
M          677618   include/string.cc
M          677618   include/vector
M          677618   include/vector.cc
M          677618   include/deque.cc
M          677618   tests/utilities/20.auto.ptr.cpp
M          677618   tests/containers/23.bitset.cpp
M          677618   tests/localization/22.locale.synopsis.cpp
M          677618   tests/strings/21.string.exceptions.cpp
M          677618   src/vecbool.cpp
M          677618   etc/config/src/implicit_inclusion_imp.h
M          677618   etc/config/src/IMPLICIT_INCLUSION.cpp
M          677618   etc/config/src/extern_template_imp.h
M          677618   etc/config/src/EXTERN_MEMBER_TEMPLATE.cpp
M          677618   etc/config/src/extern_template_imp.cpp
M          677618   etc/config/src/implicit_inclusion_imp.cc
M          677618   README
M          677618   examples/include/stocks.h

Should I post the diffs for review before committing them?
 
Brad.
 

[Martin Sebor]

Eric Lemings wrote:

>  
> Heads up.  Substantiative change in the works.  Affects the following
> files in the 4.3.x branch.
>  
> M          677618   include/rw/_autoptr.h
> M          677618   include/rw/_defs.h
> M          677618   include/rw/_config-sunpro.h
> M          677618   include/rw/_iterator.h
> M          677618   include/rw/_iterbase.h
> M          677618   include/deque
> M          677618   include/loc/_locale.h
> M          677618   include/set
> M          677618   include/list.cc
> M          677618   include/complex
> M          677618   include/list
> M          678525   include/bitset
> M          677618   include/string
> M          677618   include/string.cc
> M          677618   include/vector
> M          677618   include/vector.cc
> M          677618   include/deque.cc
> M          677618   tests/utilities/20.auto.ptr.cpp
> M          677618   tests/containers/23.bitset.cpp
> M          677618   tests/localization/22.locale.synopsis.cpp
> M          677618   tests/strings/21.string.exceptions.cpp
> M          677618   src/vecbool.cpp
> M          677618   etc/config/src/implicit_inclusion_imp.h
> M          677618   etc/config/src/IMPLICIT_INCLUSION.cpp
> M          677618   etc/config/src/extern_template_imp.h
> M          677618   etc/config/src/EXTERN_MEMBER_TEMPLATE.cpp
> M          677618   etc/config/src/extern_template_imp.cpp
> M          677618   etc/config/src/implicit_inclusion_imp.cc
> M          677618   README
> M          677618   examples/include/stocks.h
>
> Should I post the diffs for review before committing them?

Thanks for the heads up!

It probably wouldn't hurt, although I suspect the raw patch might
be too big for ezmlm (AFAIK, the limit is 100K). If it is, I think
you might just as well commit it (assuming it's tested with at
least a couple of compilers).

Btw., I assume you're also removing the MEMBER_TEMPLATES.cpp test,
right?

Martin

[Eric Lemings-2]

 

> -----Original Message-----
> From: Martin Sebor [mailto:msebor@...] On Behalf Of Martin Sebor
> Sent: Monday, July 21, 2008 2:25 PM
> To: dev@...
> Subject: Re: STDCXX-978
>
...
> It probably wouldn't hurt, although I suspect the raw patch might
> be too big for ezmlm (AFAIK, the limit is 100K). If it is, I think
> you might just as well commit it (assuming it's tested with at
> least a couple of compilers).

I'm building and running tests for 12D and 15D builds on Linux/gcc 4.3,
Solaris, and HP-UX IPF right now.

I think the changes are pretty much straightforward but someone should
take a look with a close eye to make sure.  There are after all lots of
changes to public headers.

>
> Btw., I assume you're also removing the MEMBER_TEMPLATES.cpp test,
> right?

Uhh thanks for the reminder...  :D

Brad.