incubator-stdcxx-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Lemings" <Eric.Lemi...@roguewave.com>
Subject RE: [jira] Commented: (STDCXX-866) add 22.locale.ctype.widen.cpp
Date Tue, 29 Apr 2008 21:00:31 GMT
 
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:sebor@roguewave.com] 
> Sent: Monday, April 28, 2008 2:40 PM
> To: dev@stdcxx.apache.org
> 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]. 
> >>>
> >>
> > 
> 
> 

Mime
View raw message