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

Remove all Implementation from Header files #30

Closed
katyhuff opened this issue Jul 11, 2012 · 3 comments
Closed

Remove all Implementation from Header files #30

katyhuff opened this issue Jul 11, 2012 · 3 comments

Comments

@katyhuff
Copy link
Member

Robert brought up the fact that this would greatly speed up build time during subsequent builds. It'll add more to the implementation file, but the implementation file will directly match the header file.

@katyhuff
Copy link
Member Author

This issue came from https://github.com/cyclus/core/issues/111 . Go there for previous comments.

It was originally opened by @gidden .

@scopatz
Copy link
Member

scopatz commented Oct 24, 2013

Has this been completed in our recent refactorizations?

@gidden
Copy link
Member

gidden commented Oct 24, 2013

Not precisely, but I don't think this is a specific enough issue to keep open. I'll reopen something if it comes up.

@gidden gidden closed this as completed Oct 24, 2013
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