You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@jena.apache.org by Andy Seaborne <an...@apache.org> on 2016/09/01 12:25:47 UTC

Re: EnhGraph polymorphism

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
>