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

Update to C++11 #208

Merged
merged 1 commit into from
Sep 13, 2018
Merged

Update to C++11 #208

merged 1 commit into from
Sep 13, 2018

Conversation

dhardy
Copy link
Collaborator

@dhardy dhardy commented Sep 13, 2018

Memory management has been updated to use C++11 techniques in some places,
but not in all. A few TODO items have been left.

The update revealed a couple of bugs, likely insignificant (other than
performance).

Results changed in many tests, presumably because some copies were avoided.

Memory management has been updated to use C++11 techniques in some places,
but not in all. A few TODO items have been left.

The update revealed a couple of bugs, likely insignificant (other than
performance).

Results changed in many tests, presumably because some copies were avoided.
@dhardy
Copy link
Collaborator Author

dhardy commented Sep 13, 2018

I should mention motivation: recently, OpenMalaria has become more difficult to compile on (some) Linux distributions, since the packaged version of Xerces-C is configured for more recent compilers. This should help with that.

Caveat: untested on Windows / VisualC++. I don't know if anyone still relies on this even?

@dhardy dhardy merged commit b205311 into master Sep 13, 2018
@dhardy dhardy deleted the c++11 branch September 13, 2018 13:10
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

Successfully merging this pull request may close these issues.

1 participant