Fix documentation build error.

doc/tutorial.qbk

@@ -117,7 +117,7 @@ platforms. The complete list of Bjam executables can be found
 [h2 Let's Jam!]
 __jam__
 
-[@../example/Jamroot Here] is our minimalist Jamroot
+[@../../../../example/tutorial/Jamroot Here] is our minimalist Jamroot
 file. Simply copy the file and tweak [^use-project boost] to where your
 boost root directory is and you're OK.
 

include/boost/python/detail/pymutex.hpp

@@ -1,103 +0,0 @@
-// Copyright 2025 Boost.Python Contributors
-// Distributed under the Boost Software License, Version 1.0. (See
-// accompanying file LICENSE_1_0.txt or copy at
-// http://www.boost.org/LICENSE_1_0.txt)
-
-#ifndef BOOST_PYTHON_DETAIL_PYMUTEX_HPP
-#define BOOST_PYTHON_DETAIL_PYMUTEX_HPP
-
-#include <boost/python/detail/prefix.hpp>
-#ifdef Py_GIL_DISABLED
-// needed for pymutex wrapper
-#include <atomic>
-#include <cstddef>
-#endif
-
-namespace boost { namespace python { namespace detail {
-
-#ifdef Py_GIL_DISABLED
-
-// Re-entrant wrapper around PyMutex for free-threaded Python
-// Similar to _PyRecursiveMutex or threading.RLock
-class pymutex {
-    PyMutex m_mutex;
-    std::atomic<unsigned long> m_owner;
-    std::size_t m_level;
-
-public:
-    pymutex() : m_mutex({}), m_owner(0), m_level(0) {}
-
-    // Non-copyable, non-movable
-    pymutex(const pymutex&) = delete;
-    pymutex& operator=(const pymutex&) = delete;
-
-    void lock() {
-        unsigned long thread = PyThread_get_thread_ident();
-        if (m_owner.load(std::memory_order_relaxed) == thread) {
-            m_level++;
-            return;
-        }
-        PyMutex_Lock(&m_mutex);
-        m_owner.store(thread, std::memory_order_relaxed);
-        // m_level should be 0 when we acquire the lock
-    }
-
-    void unlock() {
-        unsigned long thread = PyThread_get_thread_ident();
-        // Verify current thread owns the lock
-        if (m_owner.load(std::memory_order_relaxed) != thread) {
-            // This should never happen - programming error
-            return;
-        }
-        if (m_level > 0) {
-            m_level--;
-            return;
-        }
-        m_owner.store(0, std::memory_order_relaxed);
-        PyMutex_Unlock(&m_mutex);
-    }
-
-    bool is_locked_by_current_thread() const {
-        unsigned long thread = PyThread_get_thread_ident();
-        return m_owner.load(std::memory_order_relaxed) == thread;
-    }
-};
-
-
-// RAII lock guard for pymutex
-class pymutex_guard {
-    pymutex& m_mutex;
-
-public:
-    explicit pymutex_guard(pymutex& mutex) : m_mutex(mutex) {
-        m_mutex.lock();
-    }
-
-    ~pymutex_guard() {
-        m_mutex.unlock();
-    }
-
-    // Non-copyable, non-movable
-    pymutex_guard(const pymutex_guard&) = delete;
-    pymutex_guard& operator=(const pymutex_guard&) = delete;
-};
-
-// Global mutex for protecting all Boost.Python internal state
-// Similar to pybind11's internals.mutex
-BOOST_PYTHON_DECL pymutex& get_global_mutex();
-
-// Macro for acquiring the global lock
-// Similar to pybind11's PYBIND11_LOCK_INTERNALS
-#define BOOST_PYTHON_LOCK_STATE() \
-    ::boost::python::detail::pymutex_guard lock(::boost::python::detail::get_global_mutex())
-
-#else
-
-// No-op macro when not in free-threaded mode
-#define BOOST_PYTHON_LOCK_STATE()
-
-#endif // Py_GIL_DISABLED
-
-}}} // namespace boost::python::detail
-
-#endif // BOOST_PYTHON_DETAIL_PYMUTEX_HPP

include/boost/python/module_init.hpp

@@ -11,41 +11,11 @@
 
 # ifndef BOOST_PYTHON_MODULE_INIT
 
-namespace boost { namespace python {
+namespace boost { namespace python { namespace detail {
-
-#ifdef HAS_CXX11
-// Use to activate the Py_MOD_GIL_NOT_USED flag.
-class mod_gil_not_used {
-public:
-    explicit mod_gil_not_used(bool flag = true) : flag_(flag) {}
-    bool flag() const { return flag_; }
-
-private:
-    bool flag_;
-};
-
-namespace detail {
-
-inline bool gil_not_used_option() { return false; }
-template <typename F, typename... O>
-bool gil_not_used_option(F &&, O &&...o);
-template <typename... O>
-inline bool gil_not_used_option(mod_gil_not_used f, O &&...o) {
-    return f.flag() || gil_not_used_option(o...);
-}
-template <typename F, typename... O>
-inline bool gil_not_used_option(F &&, O &&...o) {
-    return gil_not_used_option(o...);
-}
-
-}
-#endif // HAS_CXX11
-
-namespace detail {
 
 #  if PY_VERSION_HEX >= 0x03000000
 
-BOOST_PYTHON_DECL PyObject* init_module(PyModuleDef&, void(*)(), bool gil_not_used = false);
+BOOST_PYTHON_DECL PyObject* init_module(PyModuleDef&, void(*)());
 
 #else
 
@@ -57,37 +27,7 @@ BOOST_PYTHON_DECL PyObject* init_module(char const* name, void(*)());
 
 #  if PY_VERSION_HEX >= 0x03000000
 
-#   ifdef HAS_CXX11
+#   define _BOOST_PYTHON_MODULE_INIT(name) \
-#    define _BOOST_PYTHON_MODULE_INIT(name, ...) \
-  PyObject* BOOST_PP_CAT(PyInit_, name)()  \
-  { \
-    static PyModuleDef_Base initial_m_base = { \
-        PyObject_HEAD_INIT(NULL) \
-        0, /* m_init */ \
-        0, /* m_index */ \
-        0 /* m_copy */ };  \
-    static PyMethodDef initial_methods[] = { { 0, 0, 0, 0 } }; \
- \
-    static struct PyModuleDef moduledef = { \
-        initial_m_base, \
-        BOOST_PP_STRINGIZE(name), \
-        0, /* m_doc */ \
-        -1, /* m_size */ \
-        initial_methods, \
-        0,  /* m_reload */ \
-        0, /* m_traverse */ \
-        0, /* m_clear */ \
-        0,  /* m_free */ \
-    }; \
- \
-    return boost::python::detail::init_module( \
-        moduledef, BOOST_PP_CAT(init_module_, name), \
-        boost::python::detail::gil_not_used_option(__VA_ARGS__) ); \
-  } \
-  void BOOST_PP_CAT(init_module_, name)()
-
-#   else // !HAS_CXX11
-#    define _BOOST_PYTHON_MODULE_INIT(name) \
   PyObject* BOOST_PP_CAT(PyInit_, name)()  \
   { \
     static PyModuleDef_Base initial_m_base = { \
@@ -113,7 +53,6 @@ BOOST_PYTHON_DECL PyObject* init_module(char const* name, void(*)());
         moduledef, BOOST_PP_CAT(init_module_, name) ); \
   } \
   void BOOST_PP_CAT(init_module_, name)()
-#   endif // HAS_CXX11
 
 #  else
 
@@ -127,15 +66,9 @@ BOOST_PYTHON_DECL PyObject* init_module(char const* name, void(*)());
 
 #  endif
 
-#  ifdef HAS_CXX11
+#  define BOOST_PYTHON_MODULE_INIT(name)                       \
-#   define BOOST_PYTHON_MODULE_INIT(name, ...)                  \
-  void BOOST_PP_CAT(init_module_,name)();                      \
-extern "C" BOOST_SYMBOL_EXPORT _BOOST_PYTHON_MODULE_INIT(name, __VA_ARGS__)
-#  else
-#   define BOOST_PYTHON_MODULE_INIT(name)                       \
   void BOOST_PP_CAT(init_module_,name)();                      \
 extern "C" BOOST_SYMBOL_EXPORT _BOOST_PYTHON_MODULE_INIT(name)
-#  endif // HAS_CXX11
 
 # endif
 

include/boost/python/numpy/dtype.hpp

@@ -17,7 +17,6 @@
 #include <boost/python/numpy/numpy_object_mgr_traits.hpp>
 #include <boost/mpl/for_each.hpp>
 #include <boost/python/detail/type_traits.hpp>
-#include <boost/type_traits/is_unsigned.hpp>
 
 namespace boost { namespace python { namespace numpy {
 

include/boost/python/object_core.hpp

@@ -419,16 +419,6 @@ inline api::object_base& api::object_base::operator=(api::object_base const& rhs
 
 inline api::object_base::~object_base()
 {
-#ifdef Py_GIL_DISABLED
-    // This is a not very elegant fix for a problem that occurs with the
-    // free-threaded build of Python.  If this is called when the interpreter
-    // has already been finalized, the thread-state can be null.  Unlike the
-    // GIL-enabled build, Py_DECREF() requires a valid thread-state.  This
-    // causes a memory leak, rather than crash, which seems preferable.
-    if (PyThreadState_GetUnchecked() == NULL) {
-        return;
-    }
-#endif
     assert( Py_REFCNT(m_ptr) > 0 );
     Py_DECREF(m_ptr);
 }

src/converter/from_python.cpp

@@ -11,7 +11,6 @@
 
 #include <boost/python/handle.hpp>
 #include <boost/python/detail/raw_pyobject.hpp>
-#include <boost/python/detail/pymutex.hpp>
 #include <boost/python/cast.hpp>
 
 #include <vector>
@@ -146,8 +145,6 @@ namespace
 
   inline bool visit(rvalue_from_python_chain const* chain)
   {
-      BOOST_PYTHON_LOCK_STATE();
-
       visited_t::iterator const p = std::lower_bound(visited.begin(), visited.end(), chain);
       if (p != visited.end() && *p == chain)
           return false;
@@ -163,8 +160,6 @@ namespace
       
       ~unvisit()
       {
-          BOOST_PYTHON_LOCK_STATE();
-
           visited_t::iterator const p = std::lower_bound(visited.begin(), visited.end(), chain);
           assert(p != visited.end());
           visited.erase(p);

src/converter/registry.cpp

@@ -5,7 +5,6 @@
 #include <boost/python/converter/registry.hpp>
 #include <boost/python/converter/registrations.hpp>
 #include <boost/python/converter/builtin_converters.hpp>
-#include <boost/python/detail/pymutex.hpp>
 
 #include <set>
 #include <stdexcept>
@@ -182,8 +181,6 @@ namespace // <unnamed>
 
   entry* get(type_info type, bool is_shared_ptr = false)
   {
-      BOOST_PYTHON_LOCK_STATE();
-
 #  ifdef BOOST_PYTHON_TRACE_REGISTRY
       registry_t::iterator p = entries().find(entry(type));
       
@@ -296,8 +293,6 @@ namespace registry
 
   registration const* query(type_info type)
   {
-      BOOST_PYTHON_LOCK_STATE();
-
       registry_t::iterator p = entries().find(entry(type));
 #  ifdef BOOST_PYTHON_TRACE_REGISTRY
       std::cout << "querying " << type

src/converter/type_id.cpp

@@ -5,7 +5,6 @@
 
 #include <boost/python/type_id.hpp>
 #include <boost/python/detail/decorated_type_id.hpp>
-#include <boost/python/detail/pymutex.hpp>
 #include <utility>
 #include <vector>
 #include <algorithm>
@@ -93,7 +92,6 @@ namespace
 
 bool cxxabi_cxa_demangle_is_broken()
 {
-    BOOST_PYTHON_LOCK_STATE();
     static bool was_tested = false;
     static bool is_broken = false;
     if (!was_tested) {
@@ -111,8 +109,6 @@ namespace detail
 {
   BOOST_PYTHON_DECL char const* gcc_demangle(char const* mangled)
   {
-      BOOST_PYTHON_LOCK_STATE();
-
       typedef std::vector<
           std::pair<char const*, char const*>
       > mangling_map;

src/dict.cpp

@@ -68,16 +68,8 @@ object dict_base::get(object_cref k) const
 {
     if (check_exact(this))
     {
-#ifdef Py_GIL_DISABLED
-        PyObject* result;
-        if (PyDict_GetItemRef(this->ptr(),k.ptr(),&result) < 0) {
-            throw_error_already_set();
-        }
-        return object(detail::new_reference(result ? result : Py_None));
-#else
         PyObject* result = PyDict_GetItem(this->ptr(),k.ptr());
         return object(detail::borrowed_reference(result ? result : Py_None));
-#endif
     }
     else
     {

src/errors.cpp

@@ -10,21 +10,9 @@
 #include <boost/python/errors.hpp>
 #include <boost/cast.hpp>
 #include <boost/python/detail/exception_handler.hpp>
-#include <boost/python/detail/pymutex.hpp>
 
 namespace boost { namespace python {
 
-#ifdef Py_GIL_DISABLED
-namespace detail {
-    // Global mutex for protecting all Boost.Python internal state
-    pymutex& get_global_mutex()
-    {
-        static pymutex mutex;
-        return mutex;
-    }
-}
-#endif
-
 error_already_set::~error_already_set() {}
 
 // IMPORTANT: this function may only be called from within a catch block!
@@ -32,13 +20,8 @@ BOOST_PYTHON_DECL bool handle_exception_impl(function0<void> f)
 {
     try
     {
-        detail::exception_handler* handler_chain = nullptr;
+        if (detail::exception_handler::chain)
-        {
+            return detail::exception_handler::chain->handle(f);
-            BOOST_PYTHON_LOCK_STATE();
-            handler_chain = detail::exception_handler::chain;
-        }
-        if (handler_chain)
-            return handler_chain->handle(f);
         f();
         return false;
     }
@@ -97,7 +80,6 @@ exception_handler::exception_handler(handler_function const& impl)
     : m_impl(impl)
     , m_next(0)
 {
-    BOOST_PYTHON_LOCK_STATE();
     if (chain != 0)
         tail->m_next = this;
     else

src/module.cpp

@@ -38,17 +38,10 @@ BOOST_PYTHON_DECL void scope_setattr_doc(char const* name, object const& x, char
 
 #if PY_VERSION_HEX >= 0x03000000
 
-BOOST_PYTHON_DECL PyObject* init_module(PyModuleDef& moduledef,
+BOOST_PYTHON_DECL PyObject* init_module(PyModuleDef& moduledef, void(*init_function)())
-    void(*init_function)(), bool gil_not_used)
 {
-    PyObject *mod = PyModule_Create(&moduledef);
-#ifdef Py_GIL_DISABLED
-    if (mod != NULL && gil_not_used) {
-        PyUnstable_Module_SetGIL(mod, Py_MOD_GIL_NOT_USED);
-    }
-#endif
     return init_module_in_scope(
-        mod,
+        PyModule_Create(&moduledef),
         init_function);
 }
 

src/object/function_doc_signature.cpp

@@ -135,15 +135,7 @@ namespace boost { namespace python { namespace objects {
             str name(get_qualname(py_type));
             if ( py_type->tp_flags & Py_TPFLAGS_HEAPTYPE ) {
                 // Qualify the type name if it is defined in a different module.
-                PyObject *type_module_name;
+                PyObject *type_module_name = PyDict_GetItemString(py_type->tp_dict, "__module__");
-#if PY_VERSION_HEX >= 0x030D0000
-                if (PyDict_GetItemStringRef(py_type->tp_dict, "__module__", &type_module_name) < 0) {
-                    throw_error_already_set();
-                }
-#else
-                type_module_name = PyDict_GetItemString(py_type->tp_dict, "__module__");
-                Py_XINCREF(type_module_name);
-#endif
                 if (
                     type_module_name
                     && PyObject_RichCompareBool(
@@ -152,11 +144,8 @@ namespace boost { namespace python { namespace objects {
                         Py_NE
                     ) != 0
                 ) {
-                    str result = str("%s.%s" % make_tuple(handle<>(type_module_name), name));
+                    return str("%s.%s" % make_tuple(handle<>(borrowed(type_module_name)), name));
-                    return result;
                 }
-                // Clean up the strong reference if we didn't use it
-                Py_XDECREF(type_module_name);
             }
             return name;
         } else {

src/object/inheritance.cpp

@@ -4,7 +4,6 @@
 // http://www.boost.org/LICENSE_1_0.txt)
 #include <boost/python/object/inheritance.hpp>
 #include <boost/python/type_id.hpp>
-#include <boost/python/detail/pymutex.hpp>
 #include <boost/graph/breadth_first_search.hpp>
 #if _MSC_FULL_VER >= 13102171 && _MSC_FULL_VER <= 13102179
 # include <boost/graph/reverse_graph.hpp>
@@ -391,8 +390,6 @@ namespace
 
   inline void* convert_type(void* const p, class_id src_t, class_id dst_t, bool polymorphic)
   {
-      BOOST_PYTHON_LOCK_STATE();
-
       // Quickly rule out unregistered types
       index_entry* src_p = seek_type(src_t);
       if (src_p == 0)
@@ -455,8 +452,6 @@ BOOST_PYTHON_DECL void* find_static_type(void* p, class_id src_t, class_id dst_t
 BOOST_PYTHON_DECL void add_cast(
     class_id src_t, class_id dst_t, cast_function cast, bool is_downcast)
 {
-    BOOST_PYTHON_LOCK_STATE();
-
     // adding an edge will invalidate any record of unreachability in
     // the cache.
     static std::size_t expected_cache_len = 0;
@@ -495,7 +490,6 @@ BOOST_PYTHON_DECL void add_cast(
 BOOST_PYTHON_DECL void register_dynamic_id_aux(
     class_id static_id, dynamic_id_function get_dynamic_id)
 {
-    BOOST_PYTHON_LOCK_STATE();
     tuples::get<kdynamic_id>(*demand_type(static_id)) = get_dynamic_id;
 }
 

src/wrapper.cpp

@@ -21,7 +21,7 @@ namespace detail
                           this->m_self, const_cast<char*>(name))))
           )
           {
-              PyObject* class_f = 0;
+              PyObject* borrowed_f = 0;
             
               if (
                   PyMethod_Check(m.get())
@@ -29,20 +29,12 @@ namespace detail
                   && class_object->tp_dict != 0
               )
               {
-#if PY_VERSION_HEX >= 0x030D0000
+                  borrowed_f = ::PyDict_GetItemString(
-                  if (::PyDict_GetItemStringRef(
-                      class_object->tp_dict, const_cast<char*>(name), &class_f) < 0) {
-                      throw_error_already_set();
-                  }
-#else
-                  class_f = ::PyDict_GetItemString(
                       class_object->tp_dict, const_cast<char*>(name));
-                  Py_XINCREF(class_f);
+
-#endif
+
               }
-              bool is_override = (class_f != PyMethod_GET_FUNCTION(m.get()));
+              if (borrowed_f != PyMethod_GET_FUNCTION(m.get()))
-              Py_XDECREF(class_f);
-              if (is_override)
                   return override(m);
           }
       }

test/fabscript

@@ -68,7 +68,6 @@ for t in [('injected',),
           ('raw_ctor',),
           ('exception_translator',),
           ('module_init_exception',),
-          ('module_nogil',),
           ('test_enum', ['enum_ext']),
           ('test_cltree', ['cltree']),
           ('newtest', ['m1', 'm2']),

test/module_nogil.cpp

@@ -1,25 +0,0 @@
-// Test for BOOST_PYTHON_MODULE with optional mod_gil_not_used argument
-
-#include <boost/python/module.hpp>
-#include <boost/python/def.hpp>
-
-// Simple function to export
-int get_value() {
-    return 1234;
-}
-
-#ifdef HAS_CXX11
-// C++11 build: test with mod_gil_not_used option
-BOOST_PYTHON_MODULE(module_nogil_ext, boost::python::mod_gil_not_used())
-{
-    using namespace boost::python;
-    def("get_value", get_value);
-}
-#else
-// C++98 build: test without optional arguments
-BOOST_PYTHON_MODULE(module_nogil_ext)
-{
-    using namespace boost::python;
-    def("get_value", get_value);
-}
-#endif

test/module_nogil.py

@@ -1,29 +0,0 @@
-"""
->>> from module_nogil_ext import *
->>> get_value()
-1234
->>> import sys, sysconfig
->>> Py_GIL_DISABLED = bool(sysconfig.get_config_var('Py_GIL_DISABLED'))
->>> if Py_GIL_DISABLED and sys._is_gil_enabled():
-...    print('GIL is enabled and should not be')
-... else:
-...    print('okay')
-okay
-"""
-
-from __future__ import print_function
-
-def run(args = None):
-    import sys
-    import doctest
-
-    if args is not None:
-        sys.argv = args
-    return doctest.testmod(sys.modules.get(__name__))
-
-if __name__ == '__main__':
-    print("running...")
-    import sys
-    status = run()[0]
-    if (status == 0): print("Done.")
-    sys.exit(status)

test/shared_ptr.py

@@ -38,7 +38,7 @@
 12
 >>> try: modify(p)
 ... except TypeError: pass
-... else: print('expected a TypeError')
+... else: 'print(expected a TypeError)'
 >>> look(None)
 -1
 >>> store(p)
@@ -61,7 +61,7 @@ bye
 13
 >>> try: modify(z)
 ... except TypeError: pass
-... else: print('expected a TypeError')
+... else: 'print(expected a TypeError)'
 
 >>> Z.get() # should be None
 >>> store(z)
@@ -84,7 +84,7 @@ bye
 17
 >>> try: modify(x)
 ... except TypeError: pass
-... else: print('expected a TypeError')
+... else: 'print(expected a TypeError)'
 >>> look(None)
 -1
 >>> store(x)