-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
SetDefaultParameter
mangles strings with whitespaces
#217
Comments
I am unable to track this down fully right now so just writing some notes: This was brought up previously in eic/EICrecon PR 472 and was addressed in PR #191 . It should be getting checked with the test here:
It's not clear what is different in EICrecon from the JANA test so will need to be tracked down starting with a reproducing example from there. |
Hello @wdconinc , We added the test case that shows that a c++ command is correctly parsed when provided as a string input to JParameter. Let me know if there is something that we are missing. |
…strings_with_whitespaces Address issue #217: Test JParameterManager handling of strings containing whitespace
In EICrecon, we end up having to implement workarounds like eic/EICrecon#575, and those are only possible because our problem allows for strings without spaces to be interpreted correctly.
The text was updated successfully, but these errors were encountered: