Não faz sentido o tratamento de um erro está no get, a linha colocada vai lançar um assertationException caso shiroRoleInstance seja nullo e o tratamento está sendo feito nos catch's. Essa mudança foi feita com o intuito de retirar esse clone : 
if(!shiroRoleInstance)
            return

E também organizar o código. As mudanças abaixo também foram feitas com essa intenção. A ferramenta foi o intelliJ como eu disse acima.