[ 
https://issues.apache.org/jira/browse/JENA-557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14259406#comment-14259406
 ] 

Andy Seaborne commented on JENA-557:
------------------------------------

Reviewing SimpleEventManager, I see that it can be clearer up to give 
flexibility in the future.  

The constructor with Graph argument is now deprecated, and a new non-argument 
constructor added. The target of the graph notification is passed into the 
notify* operations and this is passed to the graph listener.  By discouraging 
the use of the filed for the graph at creation time (also now deprecated), this 
could be used across many graphs, or used as a superclass of such an event 
manager.

SDB aligned to the new design.


> SimpleEventManger no longer needs Graph object
> ----------------------------------------------
>
>                 Key: JENA-557
>                 URL: https://issues.apache.org/jira/browse/JENA-557
>             Project: Apache Jena
>          Issue Type: Bug
>            Reporter: Claude Warren
>            Priority: Minor
>             Fix For: Jena 2.12.2
>
>
> The Graph object passed in the SimpleEventManger constructor is not used and 
> is no longer used in any code that I can find.
> Suggest deprecating the current constructor and replacing with a new no-arg 
> constructor.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to