On 31/08/16 15:41, Chris Dollin wrote:
On 28/08/16 11:47, Andy Seaborne wrote:
So that was a while ago :-)
Indeed.
To clear up, would this seem like a good plan:
1/ Remove "// extends Polymorphic "
2/ Change the javadoc. No sure what the wording should be (Chris -
suggestion?)
but something like
"""
This class adds the {@link Polymorphic} functions for a Node
within this {@link Model} based on a supplied {@link Personality}.
"""
Andy
There's some code that can be stripped out too, eg I believe
getPersonality() is never used elsewhere. I'll have a look at it
now I have the requestSubCancel() action under control.
EnhGraph.getPersonality is used by EnhNode.getPersonality
I think the best route forward is to separate cleaning the comments from
any code changes.
Andy
Chris
