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

Improve dependencies between compilation units #74

Open
mathbagu opened this issue Apr 2, 2020 · 0 comments
Open

Improve dependencies between compilation units #74

mathbagu opened this issue Apr 2, 2020 · 0 comments
Projects

Comments

@mathbagu
Copy link

mathbagu commented Apr 2, 2020

  • Do you want to request a feature or report a bug?
    Arguable

  • What is the current behavior?
    Sometimes, depending on the changes, almost whole project is rebuild after a change. I think there is some troubles in the dependencies between the different compilation units, specially with the template implementations.

It's really easy to have circular dependencies, because we have a lot of bidirectional links between our classes. I think some #include statement could be replaced by a forward declaration. We have to write a coding style about headers management.

  • If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
    Do a complete build
    Change the implementation of a template method
    Run the compilation again

    • if the method was a classic method, in a CPP, only this CPP would be compiled
    • a lot of CPP files would be compiled
  • What is the expected behavior?
    The project have to be cleaned to be compiled faster.

  • What is the motivation / use case for changing the behavior?
    Be more productive.

  • Please tell us about your environment:

    • PowSyBl Version: ...
    • OS Version: ...
  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, spectrum, etc)

(if a question doesn't apply, you can delete it)

@mathbagu mathbagu added this to To do in IIDM v1.4.1 via automation Nov 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant