Skip to content

Upgrade CppInterOp to master #21961

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
aaronj0 wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Upgrade CppInterOp to master #21961

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
7983d5f
Upgrade CppInterOp to master
aaronj0 Apr 20, 2026
e6e3152
[roottest] Make TExceptionHandlerTests into a separate gtest executable
aaronj0 Apr 21, 2026
82ea7cd
[cppinterop] Fix TracingTests can't include CppInterOp.h in non-stand…
aaronj0 Apr 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion core/base/test/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
ROOT_ADD_GTEST(CoreBaseTests
TNamedTests.cxx
TQObjectTests.cxx
TExceptionHandlerTests.cxx
TStringTest.cxx
TUrlTest.cxx
TBitsTests.cxx
Expand All @@ -22,6 +21,8 @@ endif()
target_compile_options(CoreBaseTests PRIVATE -DEXPECTED_SHARED_LIBRARY_DIR="${localruntimedir}")
target_compile_options(CoreBaseTests PRIVATE -DEXPECTED_INCLUDE_DIR="${CMAKE_BINARY_DIR}/include")

ROOT_ADD_GTEST(CoreExceptionHandlerTests TExceptionHandlerTests.cxx LIBRARIES RIO Core)

ROOT_ADD_GTEST(CoreErrorTests TErrorTests.cxx LIBRARIES Core)

ROOT_ADD_GTEST(CoreSystemTests TSystemTests.cxx LIBRARIES Core)
Expand Down
2 changes: 2 additions & 0 deletions interpreter/CppInterOp/.clang-tidy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ Checks: >
-readability-implicit-bool-conversion,
-readability-magic-numbers,
-readability-named-parameter,
-modernize-concat-nested-namespaces,
-readability-avoid-return-with-void-value,
-readability-function-cognitive-complexity,
-readability-redundant-access-specifiers,
-cppcoreguidelines-avoid-magic-numbers,
Expand Down
2 changes: 1 addition & 1 deletion interpreter/CppInterOp/cppinterop-version.tag
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
039bf0423284b1139674525d097e656cb0feb783
b8ae9f666cde92bc0d848f6bf357c162980b7d7e
3 changes: 2 additions & 1 deletion interpreter/CppInterOp/include/CppInterOp/CppInterOp.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ using TCppIndex_t = size_t;
using TCppScope_t = void*;
using TCppConstScope_t = const void*;
using TCppType_t = void*;
using TCppConstType_t = const void*;
using TCppFunction_t = void*;
using TCppConstFunction_t = const void*;
using TCppFuncAddr_t = void*;
Expand Down Expand Up @@ -368,7 +369,7 @@ CPPINTEROP_API bool IsComplete(TCppScope_t scope);
CPPINTEROP_API size_t SizeOf(TCppScope_t scope);

/// Checks if it is a "built-in" or a "complex" type.
CPPINTEROP_API bool IsBuiltin(TCppType_t type);
CPPINTEROP_API bool IsBuiltin(TCppConstType_t type);

/// Checks if it is a templated class.
CPPINTEROP_API bool IsTemplate(TCppScope_t handle);
Expand Down
1 change: 1 addition & 0 deletions interpreter/CppInterOp/lib/CppInterOp/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,7 @@ add_llvm_library(clangCppInterOp BUILDTREE_ONLY
CppInterOp.cpp
Dispatch.cpp
CXCppInterOp.cpp
Tracing.cpp
${DLM}
LINK_LIBS
${link_libs}
Expand Down
Loading
Loading