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

Wrap some more calls to delete with unique_ptr. #10860

Merged
merged 1 commit into from
Sep 1, 2020

Conversation

drwells
Copy link
Member

@drwells drwells commented Aug 29, 2020

Not the most important feature, but we are actually quite close to getting rid of all new/delete pairs.

@drwells
Copy link
Member Author

drwells commented Aug 29, 2020

/rebuild

@drwells
Copy link
Member Author

drwells commented Aug 30, 2020

The CI fails when I don't use std::make_unique - I guess I am still used to thinking of C++11 :)

Copy link
Member

@bangerth bangerth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Almost there.

source/fe/fe_abf.cc Outdated Show resolved Hide resolved
source/fe/fe_abf.cc Outdated Show resolved Hide resolved
source/fe/fe_abf.cc Outdated Show resolved Hide resolved
@drwells
Copy link
Member Author

drwells commented Aug 31, 2020

Fixed!

@peterrum
Copy link
Member

peterrum commented Sep 1, 2020

Something is wrong with CI. Could you rebase and push again!?

@masterleinad
Copy link
Member

/rebuild

@masterleinad masterleinad merged commit 7d4b2c2 into dealii:master Sep 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants