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

Option to name include for config.h

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 4.1.3, 4.1.4, 4.2.0, 4.2.1
    • 4.3.0
    • Configuration
    • None
    • current stdcxx, all environments i think (at least windows with interix and wgcc)

    • 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

              sebor Martin Sebor
              mduft Markus Duft
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:

                Time Tracking

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