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

ierr and serr module of origin #119

Closed
tjfulle opened this issue Nov 29, 2017 · 6 comments
Closed

ierr and serr module of origin #119

tjfulle opened this issue Nov 29, 2017 · 6 comments

Comments

@tjfulle
Copy link
Collaborator

tjfulle commented Nov 29, 2017

ierr and serr are defined in the forteuchos module. It would be convenient if, instead, they lived in their own module, independent of forteuchos. Perhaps forerrcnt? See this comment for a use case where this would be helpful.

@aprokop @sethrj

@aprokop aprokop mentioned this issue Dec 4, 2017
7 tasks
@sethrj
Copy link
Collaborator

sethrj commented Dec 4, 2017

Sure, I just put them in forteuchos because (to my knowledge) everything in trilinos is downstream of it. The instantiations of std_x etc. might also be good to have in there as well.

@aprokop
Copy link
Collaborator

aprokop commented Dec 15, 2017

The thing I don't like here is having an extra module to be included every time one of ForTrilinos modules is included. Is there a method to avoid that?

@aprokop
Copy link
Collaborator

aprokop commented Dec 16, 2017

This is going to be fixed by #169.

@tjfulle
Copy link
Collaborator Author

tjfulle commented Dec 16, 2017

@aprokop said:

The thing I don't like here is having an extra module to be included every time one of ForTrilinos modules is included. Is there a method to avoid that?

That is already a requirement, right? For any package other than teuchos, you have to use forteuchos to get access to ierr and serr. I think #include “ForTrilinos.h” is the cleanest, and most intuitive, option. The header can include any mandatory use statements.

@aprokop
Copy link
Collaborator

aprokop commented Dec 16, 2017

@tjfulle I agree with you and that's why I followed your advice in the PR. I'm OK to always include ForTrilinos.h.

@aprokop
Copy link
Collaborator

aprokop commented Dec 19, 2017

For the most part, done in #169.

@aprokop aprokop closed this as completed Dec 19, 2017
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants