Now that I've looked closer at both tests, I think you're right: the 22.locale.ctype.narrow tests already exercises everything in the 22.locale.ctype.widen test. Question is, do we want to just remove the 22.locale.ctype.widen test and leave the narrow test as it is or split the narrow test into narrow and widen counterparts?
More generally, do we plan to integrate these new tests into subsequent 4.2.x releases or should they wait for 4.3? Brad. > -----Original Message----- > From: Martin Sebor [mailto:[EMAIL PROTECTED] > Sent: Monday, April 28, 2008 2:40 PM > To: [email protected] > Subject: Re: [jira] Commented: (STDCXX-866) add > 22.locale.ctype.widen.cpp > > Brad, I've reopened the issue until we know that the new test > really is needed (i.e., that it exercises functionality that's > not also tested by 22.locale.ctype.narrow.cpp). > > Martin Sebor wrote: > > Brad, > > > > You probably missed my question below: do we really need this test? > > > > Martin > > > > Martin Sebor (JIRA) wrote: > >> [ > >> > https://issues.apache.org/jira/browse/STDCXX-866?page=com.atla > ssian.jira.plugin.system.issuetabpanels:comment-tabpanel&focus > edCommentId=12590141#action_12590141 > >> ] > >> Martin Sebor commented on STDCXX-866: > >> ------------------------------------- > >> > >> I just noticed that > >> > [{{22.locale.ctype.narrow.cpp}}|http://svn.apache.org/repos/as > f/stdcxx/trunk/tests/localization/22.locale.ctype.narrow.cpp] > >> also exercises {{ctype::widen()}}. Does this new test do anything > >> {{22.locale.ctype.narrow.cpp}} doesn't? > >> > >>> add 22.locale.ctype.widen.cpp > >>> ----------------------------- > >>> > >>> Key: STDCXX-866 > >>> URL: > https://issues.apache.org/jira/browse/STDCXX-866 > >>> Project: C++ Standard Library > >>> Issue Type: Sub-task > >>> Components: Tests > >>> Affects Versions: 4.1.2, 4.1.3, 4.1.4, 4.2.0 > >>> Reporter: Martin Sebor > >>> Fix For: 4.2.1 > >>> > >>> Attachments: 22.locale.ctype.widen.cpp, 22_ctype_widen.cpp > >>> > >>> Original Estimate: 2h > >>> Time Spent: 1h > >>> Remaining Estimate: 1h > >>> > >>> The test 22_ctype_widen.cpp needs to be migrated from the > Rogue Wave > >>> source code repository into Subversion under the name > >>> > [localization/22.locale.ctype.widen.cpp|http://svn.apache.org/ > repos/asf/stdcxx/trunk/tests/localization/22.locale.ctype.widen.cpp]. > >>> > >> > > > >
