X Tutup
The Wayback Machine - https://web.archive.org/web/20201220223707/https://github.com/paceholder/nodeeditor/pull/171
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 registry / creator code #171

Open
wants to merge 9 commits into
base: master
from

Conversation

@Quincunx271
Copy link
Contributor

@Quincunx271 Quincunx271 commented May 11, 2018

Simplifies the registration code, removing duplication, by using tag dispatching, and minimizing the compile-time branch code.

The only difference between the two registerModelImpl functions was the way the name was being computed, so rather than using registerModelImpl to do the compile-time branching, make a new computeName function.

@Quincunx271 Quincunx271 changed the title Simplify registry code Improve registry / creator code May 11, 2018
@Quincunx271
Copy link
Contributor Author

@Quincunx271 Quincunx271 commented May 11, 2018

I also made the DataModelRegistry outlive the Connections and Nodes, so that e.g. a NodeDataModel could have a regular pointer (not shared) into something from its creator.

@Quincunx271 Quincunx271 force-pushed the Quincunx271:simplify-registry-code branch from ec0b1ba to 0ad8ba9 May 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.
X Tutup