Status: New
Owner: ----

New issue 198 by heue...@gmail.com: Remove property code constants, add firePropertyChange(String, T, T) methods
http://code.google.com/p/piccolo2d/issues/detail?id=198

Remove all public PROPERTY_CODE_XXX and PROPERTY_XXX String constants, and refactor

firePropertyChange(int, String, Object, Object)

methods to

firePropertyChange(String, T, T)

where T is Object, int, boolean as in java.beans.PropertyChangeSupport.


I don't believe the performance reasons hinted at in the javadoc

http://www.piccolo2d.org/doc/piccolo2d.java/release-1.3/core/apidocs/edu/umd/cs/piccolo/PNode.html#fireChildPropertyChange(java.beans.PropertyChangeEvent,%20int)

are worth polluting the API with all these constant values, and convention in Swing programming is to use String literals for firePropertyChange, not public String constants.

--
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en

Reply via email to