Remove potential CCE in PatternMatchingForInstanceofFixCore (eclipse-jdt#2212)

- for eclipse-jdt#2208

Author: jjohnstn

org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/fix/PatternMatchingForInstanceofFixCore.java

@@ -114,9 +114,6 @@ public boolean visit(final InstanceofExpression visited) {
 				boolean isPositiveCaseToAnalyze= true;
 				ASTNode currentNode= visited;
 
-				if (visited.getLocationInParent() == ConditionalExpression.EXPRESSION_PROPERTY) {
-
-				}
 				while (currentNode.getParent() != null
 						&& (!(currentNode.getParent() instanceof IfStatement)
 								|| currentNode.getLocationInParent() != IfStatement.EXPRESSION_PROPERTY)
@@ -173,9 +170,7 @@ public boolean visit(final InstanceofExpression visited) {
 						fResult.add(collector.build());
 						return false;
 					}
-				} else {
-					IfStatement ifStatement= (IfStatement) currentNode.getParent();
-
+				} else if (currentNode.getParent() instanceof IfStatement ifStatement) {
 					ResultCollector collector= new ResultCollector(visited);
 					if (isPositiveCaseToAnalyze) {
 						collector.collect(ifStatement.getThenStatement());
@@ -457,6 +452,7 @@ public PatternMatchingForInstanceofFixOperation(final InstanceofExpression nodeT
 			this.expressionToMove= expressionToMove;
 		}
 
+		@SuppressWarnings("deprecation")
 		@Override
 		public void rewriteAST(final CompilationUnitRewrite cuRewrite, final LinkedProposalModelCore linkedModel) throws CoreException {
 			ASTRewrite rewrite= cuRewrite.getASTRewrite();