[cppyy] Use cppdef for variable in test#22365
Conversation
guitargeek
left a comment
There was a problem hiding this comment.
LGTM, thanks!
By the way, I think cppexec should work in principle, because it's the wrapper around gInterpreter->ProcessLine. And statements like gInterpreter->ProcessLine("int interactive_b = 4") do work in C++, so I don't see why they should not work in Python. You have any thoughts on this? I would like to understand what's going on there, to understand the interpreter better 🙂 Not a problem for this PR though, because it doesn't matter if we use cppexec or cppdef to implement the test for the interactive access to the Cling global scope.
|
FWIW, I also don't understand why |
Test Results 21 files 21 suites 3d 4h 53m 31s ⏱️ For more details on these failures, see this check. Results for commit 2241db2. ♻️ This comment has been updated with latest results. |
This fixes test22_cppexec which was marked as xfail if not IS_WINDOWS, which is the opposite of the given reason "Fails on Windows".
|
My understanding is that |
This fixes
test22_cppexecwhich was marked asxfailifnot IS_WINDOWS, which is the opposite of the given reason "Fails on Windows".