Redundant else-fi branch. It does no really mattered if the column was instanceof AttributeCol52 - the else-if branch and the else branch both contain exactly same code.