Skip to content

[cppyy] fix: instantiation of templated functions#19450

Merged
Vipul-Cariappa merged 1 commit intoroot-project:masterfrom
Vipul-Cariappa:fix/TemplateProxy-NonInit-ToStr
Jul 28, 2025
Merged

[cppyy] fix: instantiation of templated functions#19450
Vipul-Cariappa merged 1 commit intoroot-project:masterfrom
Vipul-Cariappa:fix/TemplateProxy-NonInit-ToStr

Conversation

@Vipul-Cariappa
Copy link
Copy Markdown
Contributor

When instantiating with another templated function

@Vipul-Cariappa
Copy link
Copy Markdown
Contributor Author

@siliataider, @vepadulano, You might be interested in this.
This enables what we were discussing yesterday.

Comment thread interpreter/cling/lib/Interpreter/LookupHelper.cpp
@dpiparo
Copy link
Copy Markdown
Member

dpiparo commented Jul 23, 2025

Very nice. @Vipul-Cariappa would you mind perhaps adding a test?

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 23, 2025

Test Results

    21 files      21 suites   3d 8h 56m 57s ⏱️
 3 218 tests  3 218 ✅ 0 💤 0 ❌
65 804 runs  65 804 ✅ 0 💤 0 ❌

Results for commit 52a7437.

♻️ This comment has been updated with latest results.

@Vipul-Cariappa Vipul-Cariappa force-pushed the fix/TemplateProxy-NonInit-ToStr branch from dfbe8ab to 8e00338 Compare July 24, 2025 06:58
@Vipul-Cariappa
Copy link
Copy Markdown
Contributor Author

Vipul-Cariappa commented Jul 24, 2025 via email

Comment thread bindings/pyroot/cppyy/cppyy/test/test_templates.py Outdated
Comment thread bindings/pyroot/cppyy/cppyy/test/test_templates.py Outdated
Comment thread bindings/pyroot/cppyy/CPyCppyy/src/TemplateProxy.cxx Outdated
@Vipul-Cariappa Vipul-Cariappa force-pushed the fix/TemplateProxy-NonInit-ToStr branch 2 times, most recently from 6f6bf91 to db169ce Compare July 25, 2025 13:53
Copy link
Copy Markdown
Member

@vepadulano vepadulano left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

when instantiating with another templated function
@Vipul-Cariappa Vipul-Cariappa force-pushed the fix/TemplateProxy-NonInit-ToStr branch from db169ce to 52a7437 Compare July 28, 2025 08:14
Comment thread bindings/pyroot/cppyy/CPyCppyy/src/TemplateProxy.cxx
@Vipul-Cariappa Vipul-Cariappa merged commit 00565c6 into root-project:master Jul 28, 2025
45 of 48 checks passed
@Vipul-Cariappa Vipul-Cariappa deleted the fix/TemplateProxy-NonInit-ToStr branch July 28, 2025 13:39
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.

4 participants