IMO 
return priorities != null ? priorities : Collections.emptySet();
is more readable, but that's subjective of course