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
Remove unused option core #5192
Remove unused option core #5192
Conversation
@@ -109,12 +109,8 @@ namespace aspect | |||
if (geometry_model.has_curved_elements()) | |||
return std::make_unique<MappingQCache<dim>>(4); | |||
|
|||
#if !DEAL_II_VERSION_GTE(9,4,0) || DEAL_II_VERSION_GTE(9,4,1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just talking myself through this: The condition means that deal.II either (i) is prior to 9.4, or (ii) 9.4.1 or later. The former cannot happen any more, but it is conceivable that someone uses deal.II 9.4.0. So I think the right change here is to replace this line by
#if DEAL_II_VERSION_GTE(9,4,1)
and leave the rest of the code we have here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh right, looked over the '!'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, looks good now!
Do you know how to "squash" the six commits into one? This way, there would only be one commit that encapsulates everything, and the various detours you took will no longer be visible :-) |
0444394
to
39319f8
Compare
39319f8
to
5061df5
Compare
Pull Request Checklist. Please read and check each box with an X. Delete any part not applicable. Ask on the forum if you need help with any step.
Describe what you did in this PR and why you did it.
Before your first pull request:
For all pull requests:
For new features/models or changes of existing features: