[ https://issues.apache.org/jira/browse/PIVOT-1011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16336574#comment-16336574 ]
Roger Whitcomb commented on PIVOT-1011: --------------------------------------- Lots more: Sending core\src\org\apache\pivot\json\JSONSerializer.java Sending core\src\org\apache\pivot\json\JSONSerializerListener.java Sending core\src\org\apache\pivot\serialization\CSVSerializer.java Sending core\src\org\apache\pivot\serialization\CSVSerializerListener.java Sending core\src\org\apache\pivot\xml\Element.java Sending core\src\org\apache\pivot\xml\ElementListener.java Sending core\src\org\apache\pivot\xml\Node.java Sending core\src\org\apache\pivot\xml\NodeListener.java Sending core\src\org\apache\pivot\xml\XMLSerializer.java Sending core\src\org\apache\pivot\xml\XMLSerializerListener.java Sending demos\src\org\apache\pivot\demos\million\LargeData.java Sending tests\src\org\apache\pivot\tests\CardPaneTest.java Sending tutorials\src\org\apache\pivot\tutorials\navigation\CardPanes.java Sending wtk\src\org\apache\pivot\wtk\Alert.java Sending wtk\src\org\apache\pivot\wtk\AlertListener.java Sending wtk\src\org\apache\pivot\wtk\CardPane.java Sending wtk\src\org\apache\pivot\wtk\CardPaneListener.java Sending wtk\src\org\apache\pivot\wtk\Container.java Sending wtk\src\org\apache\pivot\wtk\ContainerListener.java Sending wtk\src\org\apache\pivot\wtk\ContainerMouseListener.java Sending wtk\src\org\apache\pivot\wtk\DesktopApplicationContext.java Sending wtk\src\org\apache\pivot\wtk\FileBrowser.java Sending wtk\src\org\apache\pivot\wtk\FileBrowserListener.java Sending wtk\src\org\apache\pivot\wtk\Viewport.java Sending wtk\src\org\apache\pivot\wtk\ViewportListener.java Sending wtk\src\org\apache\pivot\wtk\content\ListViewItemEditor.java Sending wtk\src\org\apache\pivot\wtk\content\TableViewRowEditor.java Sending wtk\src\org\apache\pivot\wtk\content\TreeViewNodeEditor.java Sending wtk\src\org\apache\pivot\wtk\media\Image.java Sending wtk\src\org\apache\pivot\wtk\media\ImageListener.java Sending wtk\src\org\apache\pivot\wtk\media\Movie.java Sending wtk\src\org\apache\pivot\wtk\media\MovieListener.java Sending wtk\src\org\apache\pivot\wtk\skin\CalendarButtonSkin.java Sending wtk\src\org\apache\pivot\wtk\skin\ColorChooserButtonSkin.java Sending wtk\src\org\apache\pivot\wtk\skin\ListButtonSkin.java Sending wtk\src\org\apache\pivot\wtk\skin\MenuBarItemSkin.java Sending wtk\src\org\apache\pivot\wtk\skin\MenuButtonSkin.java Sending wtk\src\org\apache\pivot\wtk\skin\MenuItemSkin.java Sending wtk\src\org\apache\pivot\wtk\skin\MovieViewSkin.java Sending wtk\src\org\apache\pivot\wtk\text\Block.java Sending wtk\src\org\apache\pivot\wtk\text\BlockListener.java Sending wtk\src\org\apache\pivot\wtk\text\BulletedList.java Sending wtk\src\org\apache\pivot\wtk\text\BulletedListListener.java Sending wtk\src\org\apache\pivot\wtk\text\Element.java Sending wtk\src\org\apache\pivot\wtk\text\ElementListener.java Sending wtk\src\org\apache\pivot\wtk\text\NumberedList.java Sending wtk\src\org\apache\pivot\wtk\text\NumberedListListener.java Sending wtk\src\org\apache\pivot\wtk\text\TextNode.java Sending wtk\src\org\apache\pivot\wtk\text\TextNodeListener.java Sending wtk-terra\src\org\apache\pivot\wtk\skin\terra\TerraDialogSkin.java Sending wtk-terra\src\org\apache\pivot\wtk\skin\terra\TerraFileBrowserSheetSkin.java Sending wtk-terra\src\org\apache\pivot\wtk\skin\terra\TerraMenuPopupSkin.java Sending wtk-terra\src\org\apache\pivot\wtk\skin\terra\TerraSuggestionPopupSkin.java Transmitting file data .....................................................done Committing transaction... Committed revision 1822066. > Move ListenerList implementations of interfaces into the interface itself > ------------------------------------------------------------------------- > > Key: PIVOT-1011 > URL: https://issues.apache.org/jira/browse/PIVOT-1011 > Project: Pivot > Issue Type: Improvement > Components: core-collections, core-util, web, wtk, wtk-terra > Environment: All > Reporter: Roger Whitcomb > Assignee: Roger Whitcomb > Priority: Minor > Attachments: 1011.diffs > > > A universal paradigm in Pivot is to have a "listener" interface for a class > or data structure that is used to notify listeners of changes in the > class/data. There is then an "Adapter" static class in the interface file > that implements the interface with default implementations. Then there is a > very separate enclosed static class that implements the "ListenerList" > interface of that listener interface. And usually (or always) this "listener > list" class is defined/used only in the class that needs to notify the > listeners. However, this class must be very parallel to not only the > interface itself, but also the "Adapter" class, and yet it is in a different > place. > So, it seems somewhat reasonable to move all these "listener list" classes > into the interfaces themselves, so all three related things are located in > the same file. A preliminary POC of this concept was done with "Query.java", > and "QueryListener.java" and it looks good. > This doesn't seem to require changes to client code, because the accessor > methods only refer to "ListenerList<....>" and not to the listener list class > itself (in order to be more general, of course), but which helps us to hide > the implementing class away inside the interface. > I will attach the diff of the POC, to hopefully make this more clear. It may > seem a somewhat nebulous concept, but the idea is to keep "like things" > together for clarity. -- This message was sent by Atlassian JIRA (v7.6.3#76005)