Fix #26: string_methods.cpp does not support ChaiScript 6+ by leftibot · Pull Request #29 · ChaiScript/ChaiScript_Extras · GitHub
Skip to content

Fix #26: string_methods.cpp does not support ChaiScript 6+#29

Merged
lefticus merged 1 commit intoChaiScript:masterfrom
leftibot:fix/issue-26-string-methods-cpp-does-not-support-chai
Apr 16, 2026
Merged

Fix #26: string_methods.cpp does not support ChaiScript 6+#29
lefticus merged 1 commit intoChaiScript:masterfrom
leftibot:fix/issue-26-string-methods-cpp-does-not-support-chai

Conversation

@leftibot
Copy link
Copy Markdown
Contributor

Automated fix by @leftibot.

What changed

Fix #26: Update to ChaiScript 6.1.0 and fix vector_type API
ChaiScript 6+ changed the vector_type signature from returning a ModulePtr
to taking a Module& parameter and returning void. The ChaiScript constructor
also no longer accepts a ModulePtr for stdlib. Updated bootstrap() to register
the vector_type for std::vectorstd::string directly within the module, and
updated tests to use the new ChaiScript 6+ API. Fixed Catch SIGSTKSZ
compile error on newer glibc where SIGSTKSZ is no longer a constant expression.
Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Files

 cmake/chaiscript.cmake                       | 2 +-
 include/chaiscript/extras/string_methods.hpp | 5 ++++-
 tests/catch.hpp                              | 6 +++---
 tests/math.cpp                               | 4 +---
 tests/string_methods.cpp                     | 5 +----
 5 files changed, 10 insertions(+), 12 deletions(-)

Closes #26

Triggered by @lefticus.

ChaiScript 6+ changed the vector_type signature from returning a ModulePtr
to taking a Module& parameter and returning void. The ChaiScript constructor
also no longer accepts a ModulePtr for stdlib. Updated bootstrap() to register
the vector_type for std::vector<std::string> directly within the module, and
updated tests to use the new ChaiScript 6+ API. Fixed Catch SIGSTKSZ
compile error on newer glibc where SIGSTKSZ is no longer a constant expression.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@RobLoach RobLoach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks about right.

@lefticus lefticus merged commit fbe744e into ChaiScript:master Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

string_methods.cpp does not support ChaiScript 6+

3 participants