Uploaded image for project: 'C++ Standard Library'
  1. C++ Standard Library
  2. STDCXX-288

Option to name include for config.h

Add voteWatch issue
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 4.1.3, 4.1.4, 4.2.0, 4.2.1
    • Fix Version/s: 4.3.0
    • Component/s: Configuration
    • Labels:
      None
    • Environment:

      current stdcxx, all environments i think (at least windows with interix and wgcc)

    • Severity:
      Usability

      Description

      see mail "Re: problem with config.h" from Martin Sebor:

      Yes, the name is far from ideal. I've been thinking about making it configurable, perhaps by replacing the _RWSTD_USE_CONFIG macro with _RWSTD_CONFIG=<config-header-pathname>. The patch below has all the source changes necessary to make it work (the rest of the changes need to go in the makefiles and VisualStudio scripts).
      I suggest you create an enhancement request in Jira for this to help us remember to get it done.

      Martin

      Index: /build/sebor/stdcxx/include/rw/_config.h
      ===================================================================
      — /build/sebor/stdcxx/include/rw/_config.h (revision 436919)
      +++ /build/sebor/stdcxx/include/rw/_config.h (working copy)
      @@ -31,7 +31,9 @@
      #ifndef _RWSTD_RW_CONFIG_H_INCLUDED
      #define _RWSTD_RW_CONFIG_H_INCLUDED

      -#ifdef _RWSTD_USE_CONFIG
      +#ifdef _RWSTD_CONFIG
      +# include _RWSTD_CONFIG
      +#elif defined (_RWSTD_USE_CONFIG)

      1. include <config.h>
        #else
      2. include <rw/config/rwconfig_std.h>

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                sebor Martin Sebor
                Reporter:
                mduft Markus Duft

                Dates

                • Created:
                  Updated:

                  Time Tracking

                  Estimated:
                  Original Estimate - 2h
                  2h
                  Remaining:
                  Remaining Estimate - 2h
                  2h
                  Logged:
                  Time Spent - Not Specified
                  Not Specified

                    Issue deployment