Remove NodeLinked

Created by: andreucm

Remove NodeLinked and check its implications :

• remove NodeTerminus
• Reimplement getTop() to all derived classes
• id factory for each node type

Created by: joanvallve

this implies the elimination of getTop() method also

Created by: joansola

I am actually thinking on leaving NodeLinked in. I am actually inclined to improve its design, although by now I'll leave it as it is. We can re-discuss it.

Created by: joansola

closing. Will reopen if we find it useful in the future.

Status changed to reopened

@acoromin , @joanvallve , @artivis

NodeLinked is removed in branch node-linked-removed

Please try to make merge tests from your branches to this branch. I suggest this:

git commit <my last developments in my branch>
git checkout node-linked-removed
git checkout -b test 
git merge <my branch>

see what conflicts come up. Possibly a lot, but hopefully not (I already merged master into node-linked-removed so we are not sooo far apart). Call me to see what to do.

Depending of the results of your testing, I will take an action or another to merge node-linked-removed into master. And will come back with further instructions.

After the testing, if you do not want to keep the test branch, you can delete it:

git checkout <my branch>
git branch -D test

Following #99 (closed) :

git commit <my last developments in my branch>
git fetch <gitlab remote>
git checkout -b my_branch_on_top_node_linked_remove
git rebase node-linked-removed

Again conflicts might come up. In case things are getting too messy :

git rebase --abort

@jsola I'll wait for the merge on master before taking the appropriate action for !80 (merged) !81 (closed) !82 (merged) !83 (merged)

Mentioned in issue #100 (closed)

...the node-linked-removed branch isn't there anymore :(

This has been merged to master. I sent a couple of emails with this info a couple of days ago but they did not go through. I hope this hasn't created much trouble.

Status changed to closed