Conversation
|
Looks good overall. Just a couple of small comments. |
|
Noting here that our Ramtares CI system that forward-tests our workflow with the bleeding edge of Trick's We had a successful test at b6061e7 which is the direct parent of 3674388 so there appears to be something about this PR that is causing this. This isn't currently breaking us as our production branches are hanging out at much older 16828a9 and will be for some time, just posting here in case others in the Trick community are seeing something similar. Will update this thread with info as we dig in. |
|
for context: they are doing this: this stops working in newest swig anyway: which should work but also doesn't work: print(trick.command_line_args_get_argc())
print(trick.command_line_args_get_argv()[0])
print(trick.command_line_args_get_argv()[1])produces: 2
Traceback (most recent call last):
File "RUN_test/input.py", line 30, in <module>
print(trick.command_line_args_get_argv()[0])
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
SystemError: <built-in function swig_ref___getitem__> returned NULL without setting an exception |
|
Thanks for the logs, y'all. I'll take a look |
|
It's always the PRs you least expect |
argvis not supposed to be sim-readable/writable => shouldn't use TMM allocatorTrick::CommandLineArgumentsfrom usingTMM_declare_var_1d("char*")to astd::vector<std::string>char**viastd::vector::datastaticto ensure thechar**outlives functionAdditionally:
#includes in CommandLineArguments.cpp