Re: list of required C++ features

2007-09-20 Thread mark.g.brown
Martin Sebor wrote: I'd like us to start to think about and putting together a list of widely implemented core C++ features whose absence we currently work around with the help of configuration tests and macros and that are contributing in a significant way to the maintenance and development effo

Re: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread mark.g.brown
Travis Vitek wrote: Martin Sebor wrote: Travis Vitek wrote: If that is the case, then why would we possibly need this same code in any of the other methods that are used to extend the original string? I don't think we do, really. I suspect the main reason why the code is in all other (out-

Re: welcome Brad Lemings and Mark Brown

2007-09-20 Thread mark.g.brown
Martin Sebor wrote: I'd like to extend a belated welcome to Brad (AKA Eric) and Mark, the newest stdcxx committers. The Incubator PMC voted and approved them on 9/5. I've asked both Brad and Mark to sign and send/fax in their CLA's. As soon as they have been processed we'll go ahead and request t

Re: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: Stupid outlook. [21.3.5.2 p4] Effects: Determines the effective length rlen of the string to append as the smaller of n and str.size() - pos. The function then throws length_error if size() >= npos - rlen. The append that I'm invoking is described to behave as if it had ca

RE: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
Stupid outlook. > >[21.3.5.2 p4] Effects: Determines the effective length rlen of the >string to append as the smaller of n and str.size() - pos. The function >then throws length_error if size() >= npos - rlen. > >The append that I'm invoking is described to behave as if it had called >that func

RE: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
Martin Sebor wrote: > >Travis Vitek wrote: >> >> template >> class Xallocator >> { >> public: >> typedef unsigned char size_type; > >I suspect the problem might actually be here. Once you define >size_type to a type with a more generous range the test case >passes. I made this and a f

Re: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: Martin Sebor wrote: Travis Vitek wrote: If that is the case, then why would we possibly need this same code in any of the other methods that are used to extend the original string? I don't think we do, really. I suspect the main reason why the code is in all other (out-

RE: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
Martin Sebor wrote: > >Travis Vitek wrote: >> >> If that is the case, then why would we possibly need this >> same code in any of the other methods that are used to extend >> the original string? > >I don't think we do, really. I suspect the main reason why the >code is in all other (out-of-li

Re: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: Martin, I think that you are right in the case of push_back, or when the string to append is short. My last testcase didn't prove anything. This one does. If you use the current trunk, this prints '4 ' to the console. i.e. appending 20 characters to a buffer with 240 s

Re: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: [...] The change also seems unnecessary -- when size() equals capacity() we check that it doesn't exceed max_size() before allocating more memory in append(). Otherwise, when size() is less than capacity() (or rather capacity() - 1), there should be no reason to check against

RE: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
Martin, I think that you are right in the case of push_back, or when the string to append is short. My last testcase didn't prove anything. This one does. If you use the current trunk, this prints '4 ' to the console. i.e. appending 20 characters to a buffer with 240 should not get you 4, s

RE: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
Martin Sebor wrote: > >Travis Vitek wrote: >> It appears that recent changes to string have accidentally >> removed some overflow checking that used to be in the >> basic_string::append() and push_back() methods. The following >> patch adds the checks back in. > >Does this fix a test failure? O

[jira] Closed: (STDCXX-567) [Sun C++/Linux/x86] assembler error on xchg in i86/atomic.s

2007-09-20 Thread Martin Sebor (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Sebor closed STDCXX-567. --- Resolution: Fixed Fix Version/s: (was: trunk) 4.2 Fixed thus: http://sv

[jira] Updated: (STDCXX-567) [Sun C++/Linux/x86] assembler error on xchg in i86/atomic.s

2007-09-20 Thread Martin Sebor (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Sebor updated STDCXX-567: Priority: Blocker (was: Major) Affects Version/s: (was: trunk)

[jira] Assigned: (STDCXX-567) [Sun C++/Linux/x86] assembler error on xchg in i86/atomic.s

2007-09-20 Thread Martin Sebor (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Sebor reassigned STDCXX-567: --- Assignee: Martin Sebor > [Sun C++/Linux/x86] assembler error on xchg in i86/atomic.s > --

[jira] Created: (STDCXX-567) [Sun C++/Linux/x86] assembler error on xchg in i86/atomic.s

2007-09-20 Thread Martin Sebor (JIRA)
[Sun C++/Linux/x86] assembler error on xchg in i86/atomic.s --- Key: STDCXX-567 URL: https://issues.apache.org/jira/browse/STDCXX-567 Project: C++ Standard Library Issue Type: Bug

Re: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: It appears that recent changes to string have accidentally removed some overflow checking that used to be in the basic_string::append() and push_back() methods. The following patch adds the checks back in. Does this fix a test failure? Or some regression? (If the former, whi

RE: [PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
Travis Vitek wrote: > >It appears that recent changes to string have accidentally removed some >overflow checking that used to be in the basic_string::append() and >push_back() methods. The following patch adds the checks back in. > >Travis > Lets try that again. This one has the correct method

[PATCH] Add overflow checking to basic_string append and push_back

2007-09-20 Thread Travis Vitek
It appears that recent changes to string have accidentally removed some overflow checking that used to be in the basic_string::append() and push_back() methods. The following patch adds the checks back in. Travis 2007-09-20 Travis Vitek <[EMAIL PROTECTED]> * string (append): add inte

Re: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: Martin Sebor wrote: Travis Vitek wrote: It appears that svn has a pre-commit hook that could be used to require that the correct change format be used for all submissions. That would avoid this problem in the future. That'd be cool! Maybe we could even use it to reject

merging trunk to 4.2.0

2007-09-20 Thread Martin Sebor
I'm planning to merge all of the trunk to branches/4.2.0 as soon as the recently introduced build failures have cleared up in the build results. This could be as early as tomorrow and hopefully no later than early next week. Until then, I'd like to ask all committers to be extra careful before che

Re: Intel C++/Windows XP build failures

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: Martin Sebor wrote: I've asked Tim to get the compilers installed. That appears to have done the trick. I've successfully built the library and the test driver without trouble, and the tests are compiling now. Great! There are a couple of successful Intel C++ 10.0 bui

welcome Brad Lemings and Mark Brown

2007-09-20 Thread Martin Sebor
I'd like to extend a belated welcome to Brad (AKA Eric) and Mark, the newest stdcxx committers. The Incubator PMC voted and approved them on 9/5. I've asked both Brad and Mark to sign and send/fax in their CLA's. As soon as they have been processed we'll go ahead and request that accounts be set u

RE: svn commit: r576543 - in /incubator/stdcxx/trunk/etc/config/windows: msvc-7.0.config msvc-9.0-x64.config msvc-9.0.config

2007-09-20 Thread Farid Zaripov
> -Original Message- > From: Farid Zaripov [mailto:[EMAIL PROTECTED] > Sent: Thursday, September 20, 2007 8:19 PM > To: stdcxx-dev@incubator.apache.org > Subject: RE: svn commit: r576543 - in > /incubator/stdcxx/trunk/etc/config/windows: msvc-7.0.config > msvc-9.0-x64.config msvc-9.0.con

[jira] Commented: (STDCXX-566) update/generate incubation status file

2007-09-20 Thread Martin Sebor (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12529158 ] Martin Sebor commented on STDCXX-566: - The source XML file should also be updated with the latest status report a

RE: svn commit: r576543 - in /incubator/stdcxx/trunk/etc/config/windows: msvc-7.0.config msvc-9.0-x64.config msvc-9.0.config

2007-09-20 Thread Farid Zaripov
> -Original Message- > From: Martin Sebor [mailto:[EMAIL PROTECTED] > Sent: Thursday, September 20, 2007 8:01 PM > To: stdcxx-dev@incubator.apache.org > Subject: Re: svn commit: r576543 - in > /incubator/stdcxx/trunk/etc/config/windows: msvc-7.0.config > msvc-9.0-x64.config msvc-9.0.conf

[jira] Resolved: (STDCXX-561) add ASL headers to all etc/nls files

2007-09-20 Thread Farid Zaripov (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-561?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Farid Zaripov resolved STDCXX-561. -- Resolution: Fixed > add ASL headers to all etc/nls files >

RE: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Travis Vitek
Martin Sebor wrote: >Travis Vitek wrote: >> >> It appears that svn has a pre-commit hook that could be used >> to require that the correct change format be used for all >> submissions. That would avoid this problem in the future. > >That'd be cool! Maybe we could even use it to reject badly fo

Re: svn commit: r576543 - in /incubator/stdcxx/trunk/etc/config/windows: msvc-7.0.config msvc-9.0-x64.config msvc-9.0.config

2007-09-20 Thread Martin Sebor
[EMAIL PROTECTED] wrote: Author: faridz Date: Mon Sep 17 10:45:33 2007 New Revision: 576543 URL: http://svn.apache.org/viewvc?rev=576543&view=rev Log: 2007-09-17 Farid Zaripov <[EMAIL PROTECTED]> * msvc-7.0.config: Comments extended with information on MSVC 9.0 (Orcas). * msvc-

[jira] Resolved: (STDCXX-560) add ASL headers to all config tests

2007-09-20 Thread Farid Zaripov (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Farid Zaripov resolved STDCXX-560. -- Resolution: Fixed > add ASL headers to all config tests > --- >

Re: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Martin Sebor
Travis Vitek wrote: Travis Vitek wrote: Martin Sebor wrote: This Change Log seems to have lost its leading asterisks. I'll be generating ChangeLogs for stdcxx before the release so I could add them then but I wonder if we could fix it in Subversion now? That's my fault. I forgot to put the

RE: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Travis Vitek
Travis Vitek wrote: > >Martin Sebor wrote: >> >>This Change Log seems to have lost its leading asterisks. I'll be >>generating ChangeLogs for stdcxx before the release so I could add >>them then but I wonder if we could fix it in Subversion now? >> > >That's my fault. I forgot to put them in the

[jira] Reopened: (STDCXX-560) add ASL headers to all config tests

2007-09-20 Thread Farid Zaripov (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Farid Zaripov reopened STDCXX-560: -- etc/config/windows/*.config files should also have the ASL headers. > add ASL headers to all confi

RE: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Farid Zaripov
> -Original Message- > From: Martin Sebor [mailto:[EMAIL PROTECTED] On Behalf Of Martin Sebor > Sent: Thursday, September 20, 2007 6:28 PM > To: stdcxx-dev@incubator.apache.org > Subject: Re: svn commit: r58 [1/20] - in > /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ > po

RE: Intel C++/Windows XP build failures

2007-09-20 Thread Travis Vitek
Martin Sebor wrote: > >I've asked Tim to get the compilers installed. > That appears to have done the trick. I've successfully built the library and the test driver without trouble, and the tests are compiling now. Travis

RE: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Travis Vitek
Martin Sebor wrote: > >[EMAIL PROTECTED] wrote: >> Author: faridz >> Date: Thu Sep 20 07:50:22 2007 >> New Revision: 58 >> >> URL: http://svn.apache.org/viewvc?rev=58&view=rev >> Log: >> 2007-09-20 Travis Vitek <[EMAIL PROTECTED]> >> >> STDCXX-561 >> ANSI_X3.110-1983: Add ASL

Re: svn commit: r577778 [1/20] - in /incubator/stdcxx/trunk/etc/nls: charmaps/ posix/charmaps/ posix/src/ src/

2007-09-20 Thread Martin Sebor
[EMAIL PROTECTED] wrote: Author: faridz Date: Thu Sep 20 07:50:22 2007 New Revision: 58 URL: http://svn.apache.org/viewvc?rev=58&view=rev Log: 2007-09-20 Travis Vitek <[EMAIL PROTECTED]> STDCXX-561 ANSI_X3.110-1983: Add ASL Header ANSI_X3.4-1968: Same. AS

[jira] Assigned: (STDCXX-566) update/generate incubation status file

2007-09-20 Thread Farid Zaripov (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-566?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Farid Zaripov reassigned STDCXX-566: Assignee: Farid Zaripov > update/generate incubation status file >

[jira] Resolved: (STDCXX-560) add ASL headers to all config tests

2007-09-20 Thread Farid Zaripov (JIRA)
[ https://issues.apache.org/jira/browse/STDCXX-560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Farid Zaripov resolved STDCXX-560. -- Resolution: Fixed Tested on Windows and no problems were found. > add ASL headers to all confi