Author: rwhitcomb Date: Tue Apr 10 20:27:29 2018 New Revision: 1828856 URL: http://svn.apache.org/viewvc?rev=1828856&view=rev Log: PIVOT-1032: Reduce the number of blank lines and unnecessary method implementations to reduce the file size of TerraTreeViewSkin.java.
Modified: pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTreeViewSkin.java Modified: pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTreeViewSkin.java URL: http://svn.apache.org/viewvc/pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTreeViewSkin.java?rev=1828856&r1=1828855&r2=1828856&view=diff ============================================================================== --- pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTreeViewSkin.java (original) +++ pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTreeViewSkin.java Tue Apr 10 20:27:29 2018 @@ -1164,7 +1164,6 @@ public class TerraTreeViewSkin extends C assert (path != null) : "Path is null"; NodeInfo result = null; - int n = path.getLength(); if (n == 0) { @@ -1267,14 +1266,12 @@ public class TerraTreeViewSkin extends C while (nodes.getLength() > 0) { NodeInfo nodeInfo = nodes.get(0); nodes.remove(0, 1); - visibleNodes.insert(nodeInfo, insertIndex++); // If we encounter an expanded branch, we add that branch's // children to our list of nodes that are to become visible if (nodeInfo instanceof BranchInfo) { BranchInfo branchInfo = (BranchInfo) nodeInfo; - if (branchInfo.isExpanded()) { branchInfo.loadChildren(); for (int i = 0, n = branchInfo.children.getLength(); i < n; i++) { @@ -1308,7 +1305,6 @@ public class TerraTreeViewSkin extends C || (branchIndex >= 0 && parentBranchInfo.isExpanded())) { NodeInfo nodeInfo = parentBranchInfo.children.get(index); - int insertIndex = branchIndex + index + 1; if (index > 0) { @@ -1517,7 +1513,6 @@ public class TerraTreeViewSkin extends C if (showBranchControls && nodeInfo instanceof BranchInfo && x >= baseNodeX && x < baseNodeX + indent) { BranchInfo branchInfo = (BranchInfo) nodeInfo; - treeView.setBranchExpanded(path, !branchInfo.isExpanded()); consumed = true; } @@ -1586,7 +1581,6 @@ public class TerraTreeViewSkin extends C if (!consumed) { TreeView treeView = (TreeView) getComponent(); - NodeInfo nodeInfo = getNodeInfoAt(y); if (nodeInfo != null && !nodeInfo.isDisabled()) { @@ -1610,16 +1604,13 @@ public class TerraTreeViewSkin extends C } else { if (selectPath != null && count == 1 && button == Mouse.Button.LEFT) { TreeView.NodeEditor nodeEditor = treeView.getNodeEditor(); - if (nodeEditor != null) { if (nodeEditor.isEditing()) { nodeEditor.endEdit(true); } - nodeEditor.beginEdit(treeView, selectPath); } } - selectPath = null; } } @@ -1693,7 +1684,6 @@ public class TerraTreeViewSkin extends C } consumed = true; } - break; } @@ -1727,7 +1717,6 @@ public class TerraTreeViewSkin extends C } consumed = true; } - break; } @@ -1738,19 +1727,15 @@ public class TerraTreeViewSkin extends C if (paths != null && paths.getLength() > 0) { Path path = paths.get(paths.getLength() - 1); NodeInfo nodeInfo = getNodeInfoAt(path); - if (nodeInfo instanceof BranchInfo) { BranchInfo branchInfo = (BranchInfo) nodeInfo; - if (branchInfo.isExpanded()) { treeView.collapseBranch(branchInfo.getPath()); } } - consumed = true; } } - break; } @@ -1761,19 +1746,15 @@ public class TerraTreeViewSkin extends C if (paths != null && paths.getLength() > 0) { Path path = paths.get(paths.getLength() - 1); NodeInfo nodeInfo = getNodeInfoAt(path); - if (nodeInfo instanceof BranchInfo) { BranchInfo branchInfo = (BranchInfo) nodeInfo; - if (!branchInfo.isExpanded()) { treeView.expandBranch(branchInfo.getPath()); } } - consumed = true; } } - break; } @@ -1835,14 +1816,12 @@ public class TerraTreeViewSkin extends C @Override public void enabledChanged(Component component) { super.enabledChanged(component); - repaintComponent(); } @Override public void focusedChanged(Component component, Component obverseComponent) { super.focusedChanged(component, obverseComponent); - repaintComponent(); } @@ -1929,15 +1908,9 @@ public class TerraTreeViewSkin extends C } @Override - public void nodeEditorChanged(TreeView treeView, TreeView.NodeEditor previousNodeEditor) { - // No-op - } - - @Override public void selectModeChanged(TreeView treeView, TreeView.SelectMode previousSelectMode) { // The selection has implicitly been cleared clearFields(NodeInfo.SELECTED_MASK); - repaintComponent(); } @@ -1945,7 +1918,6 @@ public class TerraTreeViewSkin extends C public void checkmarksEnabledChanged(TreeView treeView) { // The check state of all nodes has implicitly been cleared clearFields(NodeInfo.CHECK_STATE_MASK); - invalidateComponent(); } @@ -2194,14 +2166,12 @@ public class TerraTreeViewSkin extends C validateSelection = true; } - // Repaint the node repaintNode(nodeInfo); } @Override public void selectedPathRemoved(TreeView treeView, Path path) { NodeInfo nodeInfo = getNodeInfoAt(path); - nodeInfo.setSelected(false); repaintNode(nodeInfo); } @@ -2219,7 +2189,6 @@ public class TerraTreeViewSkin extends C // Un-select the previous selected paths for (int i = 0, n = previousSelectedPaths.getLength(); i < n; i++) { NodeInfo previousSelectedNode = getNodeInfoAt(previousSelectedPaths.get(i)); - previousSelectedNode.setSelected(false); repaintNode(previousSelectedNode); } @@ -2229,15 +2198,10 @@ public class TerraTreeViewSkin extends C // Select the current selected paths for (int i = 0, n = selectedPaths.getLength(); i < n; i++) { NodeInfo selectedNode = getNodeInfoAt(selectedPaths.get(i)); - selectedNode.setSelected(true); repaintNode(selectedNode); } } } - @Override - public void selectedNodeChanged(TreeView treeView, Object previousSelectedNode) { - // No-op - } }