Code cleanup including:

- some deprecated fixes
- fixed some try-catch statements
- removed unnecessary casts
- renamed member shadowing locals
- removed unnecessary else statements
- commented out unused variables and dead code.

1 files changed, 2 insertions, 4 deletions

diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner/src/org/eclipse/jst/pagedesigner/actions/container/ContainerActionGroup.java b/jsf/plugins/org.eclipse.jst.pagedesigner/src/org/eclipse/jst/pagedesigner/actions/container/ContainerActionGroup.java
index 24b84e40e..807cad294 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner/src/org/eclipse/jst/pagedesigner/actions/container/ContainerActionGroup.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner/src/org/eclipse/jst/pagedesigner/actions/container/ContainerActionGroup.java
@@ -88,12 +88,10 @@ public class ContainerActionGroup extends ActionGroup {
 				if (position.getOffset() == 0
 						|| position.getContainerNode() instanceof Text) {
 					return position.getContainerPart();
-				} else {
-					return position.getSiblingEditPart(false);
 				}
-			} else {
-				return RangeUtil.findCommonAncestor(range);
+                return position.getSiblingEditPart(false);
 			}
+            return RangeUtil.findCommonAncestor(range);
 		} else {
 			return null;
 		}