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

Fixed compile error related to std::string in JSONOutputArchive #717

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Bouloulou1138
Copy link

JSONOutputArchive expected an std::string param to be passed saveValue().
This caused the compilation to fail when the more generic std::basic_string was passed and not an std::string.
This fix changes the signature of saveValue to expect an std::basic_string

JSONOutputArchive expected an std::string param to be passed saveValue().
This caused the compilation to fail when the more generic std::basic_string was passed and not an std::string.
This fix changes the signature of saveValue to expect an std::basic_string
JSONOutputArchive expected an std::string param to be passed to loadValue().
This caused the compilation to fail when the more generic std::basic_string was passed and not an std::string.
This fix changes the signature of loadValue to expect an std::basic_string
@Bouloulou1138
Copy link
Author

Hello @AzothAmmo, could this simple fix be added to the v1.3.1 release?

@AzothAmmo AzothAmmo added the bug label Sep 27, 2021
@AzothAmmo AzothAmmo added this to the v1.3.1 milestone Sep 27, 2021
@AzothAmmo AzothAmmo modified the milestones: v1.3.1, v1.3.2 Jan 10, 2022
@AzothAmmo AzothAmmo modified the milestones: v1.3.2, v1.3.3 Feb 28, 2022
@Boulougou
Copy link

I am also affected by this compilation issue, would this fix be included in the next release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants