BOOST_NO_CTYPE_FUNCTIONS and BSD OS's

[brad0]

While looking at include/boost/config/platform/bsd.hpp and updating the OpenBSD bits I noticed this
bit of the header.

//
// The BSD <ctype.h> has macros only, no functions:
//
#if !defined(__OpenBSD__) || defined(__DragonFly__)
#  define BOOST_NO_CTYPE_FUNCTIONS
#endif

I was wondering if this needed any further changes for the current state, but looking at each respective
ctype.h header for the BSD's I am not sure if this should be set or not. Could someone who is familiar
with this area comment after taking a look at the headers? I have provided URLs below,

https://gitweb.dragonflybsd.org/dragonfly.git/blob_plain/HEAD:/include/ctype.h
https://raw.githubusercontent.com/freebsd/freebsd-src/main/include/ctype.h
http://cvsweb.netbsd.org/bsdweb.cgi/~checkout~/src/include/ctype.h?rev=1.35&only_with_tag=MAIN
http://cvsweb.openbsd.org/cgi-bin/cvsweb/~checkout~/src/include/ctype.h?rev=1.25

[jzmaddock]

Test case is here: https://github.com/boostorg/config/blob/develop/test/boost_no_ctype_functions.ipp

[Kojoley]

Test case is here: https://github.com/boostorg/config/blob/develop/test/boost_no_ctype_functions.ipp

I think the test case is invalid, it takes addresses of non-addressable standard library functions
[namespace.constraints]/6

[jzmaddock]

I'm not sure when that constraint was added, but it's not in C++11.

The point is that these should be real functions and not macros as they originally were on BSD.

[Kojoley]

I'm not sure when that constraint was added, but it's not in C++11.

It is C++20, P0551.

The point is that these should be real functions and not macros as they originally were on BSD.

Converting the test to C must be the simplest solution. IIRC doing (isalpha)('x') should done the job too.

[jzmaddock]

The test case has to be C++, I'll change it, now that C++20 has apparently broken it.

[jzmaddock]

Test case corrected for C++20 in develop.

[jzmaddock]

Can you check the revised test case on BSD?

[brad0]

Sorry I forgot about this. How do I go about running the test individually or the whole test suite?

[Kojoley]

From the root of Boost, where bootstrap.sh is located: b2 libs/config/test to run the whole test suite of the Boost.Config library and b2 libs/config/test//name_of_the_test to run a test named name_of_the_test of the Boost.Config library.

[Kojoley]

Actually, it seems that I am wrong and you need to do b2 libs/config/test/all to run the test suit that includes that test, and b2 libs/config/test/all//BOOST_NO_CTYPE_FUNCTIONS to run the test only,

@jzmaddock is it done intentionally? I am asking because I do not see those test to run on https://www.boost.org/development/tests/develop/developer/config.html

[brad0]

Boost 1.80

humpty$ b2 libs/config/test/all//BOOST_NO_CTYPE_FUNCTIONS
Performing configuration checks

    - default address-model    : 64-bit (cached) [1]
    - default architecture     : x86 (cached) [1]

[1] clang-13
...found 130 targets...
...updating 19 targets...
clang-linux.compile.c++ bin.v2/libs/config/test/all/no_ctype_functions_fail.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_fail.o
libs/config/test/all/../no_ctype_functions_fail.cpp:27:2: error: "this file should not compile"
#error "this file should not compile"
 ^
libs/config/test/all/../no_ctype_functions_fail.cpp:32:11: error: use of undeclared identifier 'boost_no_ctype_functions'
   return boost_no_ctype_functions::test();
          ^
2 errors generated.
(failed-as-expected) bin.v2/libs/config/test/all/no_ctype_functions_fail.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_fail.o
**passed** bin.v2/libs/config/test/all/no_ctype_functions_fail.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_fail.test
clang-linux.compile.c++ bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass.o
clang-linux.link bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass
testing.capture-output bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass.run
**passed** bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass.test
...updated 19 targets...

From the source dir after running a build of Boost itself.

[Kojoley]

Boost 1.80

humpty$ b2 libs/config/test/all//BOOST_NO_CTYPE_FUNCTIONS
Performing configuration checks

    - default address-model    : 64-bit (cached) [1]
    - default architecture     : x86 (cached) [1]

[1] clang-13
...found 130 targets...
...updating 19 targets...
clang-linux.compile.c++ bin.v2/libs/config/test/all/no_ctype_functions_fail.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_fail.o
libs/config/test/all/../no_ctype_functions_fail.cpp:27:2: error: "this file should not compile"
#error "this file should not compile"
 ^
libs/config/test/all/../no_ctype_functions_fail.cpp:32:11: error: use of undeclared identifier 'boost_no_ctype_functions'
   return boost_no_ctype_functions::test();
          ^
2 errors generated.
(failed-as-expected) bin.v2/libs/config/test/all/no_ctype_functions_fail.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_fail.o
**passed** bin.v2/libs/config/test/all/no_ctype_functions_fail.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_fail.test
clang-linux.compile.c++ bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass.o
clang-linux.link bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass
testing.capture-output bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass.run
**passed** bin.v2/libs/config/test/all/no_ctype_functions_pass.test/clang-linux-13/debug/threading-multi/visibility-hidden/no_ctype_functions_pass.test
...updated 19 targets...

This means that the macro state is correct for your platform.

From the source dir after running a build of Boost itself.

Building Boost manually is not necessary in case of running tests, b2 would do that automatically when needed.