Bulk Operation

JBIDE-1636 Review org.jboss.tools.jst.web.tomcat package classes and remove unused

JBIDE-1636 Activate PMD rule ReturnEmptyArrayRatherThanNull: For any method that returns an array, it's a better behavior to return an empty array rather than a null reference.

JBIDE-1636 Activate PMD rule PositionLiteralsFirstInComparisons: Position literals first in String comparisons - that way if the String is null you won't get a NullPointerException, it'll just return false.

JBIDE-1636 Activate PMD rule SimplifyConditional: No need to check for null before an instanceof; the instanceof keyword returns false when given a null argument.

JBIDE-1636 Activate PMD rule UseCollectionIsEmpty: The isEmpty() method on java.util.Collection is provided to see if a collection has any elements. Comparing the value of size() to 0 merely duplicates existing behavior.

JBIDE-1636 Check if PMD has rule to check that methods with Collection based return type never returns null

JBIDE-1636 org.jboss.tools.common.text.ext.hyperlink.AbstractHyperlink review

JBIDE-1636 Code review for org.jboss.tools.common.editor