Issue830

Title one registry to manage PluginRegistry<T>/PluginTypeRegistry/PluginGroupRegistry
Priority feature Status resolved
Superseder Nosy List cedric, jendrik, malte, patfer, salome
Assigned To patfer Keywords
Optional summary

Created on 2018-09-19.11:24:23 by patfer, last changed by patfer.

Messages
msg7616 (view) Author: patfer Date: 2018-09-20.10:00:40
New comments applied and merged.
msg7601 (view) Author: malte Date: 2018-09-19.19:13:15
I left one additional (trivial) comment on bitbucket.
From my perspective, this looks good to merge. :-)
msg7587 (view) Author: patfer Date: 2018-09-19.16:23:34
I applied your suggestions
msg7577 (view) Author: jendrik Date: 2018-09-19.15:40:56
I left some comments on Bitbucket.
msg7576 (view) Author: malte Date: 2018-09-19.15:09:18
I started looking at this, but the pull request diff shows some merge conflicts.
Can you resolve these? I think what you need to do is merge from default and
resolve any merge conflicts, but someone with more experience with this (e.g.
Jendrik) might be able to give you better advice.
msg7572 (view) Author: patfer Date: 2018-09-19.14:52:20
Rework ready for review:
https://bitbucket.org/PatFer/downward/pull-requests/5/issue830/diff
msg7560 (view) Author: patfer Date: 2018-09-19.11:24:22
Rework the registry system to have one registry managing all three,
PluginRegistry, PluginTypeRegistry, PluginGroupRegistry.

One Registry to rule them all.
History
Date User Action Args
2018-09-20 10:00:40patfersetstatus: reviewing -> resolved
messages: + msg7616
2018-09-19 19:13:15maltesetmessages: + msg7601
2018-09-19 16:23:34patfersetmessages: + msg7587
2018-09-19 15:40:56jendriksetmessages: + msg7577
2018-09-19 15:09:18maltesetmessages: + msg7576
2018-09-19 14:52:20patfersetstatus: unread -> reviewing
messages: + msg7572
2018-09-19 11:24:23patfercreate