The Source for Java Technology Collaboration


Home | Changes | Index | Search | Go

JavaBean

Notes/API for Internal Review 25 July 2006

The following API and feedback were from the first internal review for JavaBean held 25 July 2006. If you have any feedback either on the format of this page, or on the content of the review, please send it to this forum posting: http://forums.java.net/jive/thread.jspa?threadID=17215&tstart=0

Reference API as HTML:

Feedback from Internal Review (25 July 2006):

  • JavaBean should have protected fire methods instead of public
  • Maybe rename JavaBean to AbstractBean

More thoughts

  • Need to have VetoablePropertyChange support as well
  • Should also optionally support ensuring that property change events are fired on the EDT

Notes/API for Internal Review 10 August 2006

Just a quick email internal review for JavaBean. The API can be found here JavaBean. Notable differences: feedback from last time was applied, and methods to support Vetoable notification was added.

Feedback from Internal Review (10 August 2006):

  • the "has" methods should be protected

Feedback from External Review (http://forums.java.net/jive/thread.jspa?messageID=145801):

  • Desire to split JavaBean into a JavaBean interface and AbstractJavaBean? implementation

Decisions

  • Leave the API generally as it is (add final in a few more places)
  • Change the name to AbstractBean?, and move to org.jdesktop.beans (matching java.beans)
  • Since an interface can be added at any time, no interface will be added at this time (see thread for details).
  • Adding a new constructor that takes a PropertyChangeSupport? and VetoableChangeSupport? so you can swap out the delegate, if you so choose

Topic JavaBean . { Edit | Ref-By | Printable | Diffs r5 < r4 < r3 < r2 < r1 | More }
 XML java.net RSS

Revision r5 - 11 Sep 2006 - 22:20:26 - Main.rbair
Parents: WebHome > SwingLabs > SwingLabsAPIReview