Eric Lemings (JIRA) wrote:
[ https://issues.apache.org/jira/browse/STDCXX-810?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12601298#action_12601298 ]
Eric Lemings commented on STDCXX-810:
-------------------------------------
I noticed the prefix {{rwt_}} used in some places. As long as we're renaming
header files, wouldn't this prefix further distinguish RWTest headers from
other headers (e.g. internal library headers)? Or would that be a little too
invasive?
That was the original intent of the prefix, but we somehow managed
to drop the 't' (I don't think it was deliberate). AFAIK, there's
just one file that uses the rwt_ prefix. The rest of the driver
uses rw_, so I'd say changing it would definitely be very invasive.
Martin
inconsistent testuite header names
----------------------------------
Key: STDCXX-810
URL: https://issues.apache.org/jira/browse/STDCXX-810
Project: C++ Standard Library
Issue Type: Improvement
Components: Test Driver
Affects Versions: 4.1.2, 4.1.3, 4.1.4, 4.2.0
Reporter: Martin Sebor
Assignee: Eric Lemings
Priority: Minor
Fix For: 4.3
Original Estimate: 1h
Remaining Estimate: 1h
The names of some test suite headers start with the {{rw_}} prefix (such as
[rw_alarm.h|http://svn.apache.org/repos/asf/stdcxx/trunk/tests/include/rw_alarm.h])
but others don't follow this convention (e.g.,
[21.strings.h|http://svn.apache.org/repos/asf/stdcxx/trunk/tests/include/21.strings.h]
or
[driver.h|http://svn.apache.org/repos/asf/stdcxx/trunk/tests/include/driver.h]).
For consistency, we should add the {{rw_}} prefix to all test suite headers
that are missing it.
