Skip to content
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

cmnData<std::string>::SerializeText adds escape characters before spaces -- breaks some code #9

Open
pkazanzides opened this issue Sep 3, 2014 · 0 comments
Assignees

Comments

@pkazanzides
Copy link
Contributor

cmnData<std::string>::SerializeText adds an escape character, '', before any spaces in the string being serialized. This breaks code in many argument constructors because they attempt to parse multiple parameters from a single string (in the FromStreamRaw method). This is only used by the cisstComponentManager, however, so it is not a major issue. My opinion is that we should preserve any escape characters already in the string, but not add any in SerializeText. We could have a separate method (perhaps a helper function) to add escape characters to a string.

@pkazanzides pkazanzides self-assigned this Sep 10, 2014
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

No branches or pull requests

1 participant