Declaring this as static is arguably a little suspect, although to be fair I'm not entirely sure why to _not_ make it static other than I'm always suspicious of static methods. I think the larger question is why this logic belongs here, and what the relationship is between `TileEntitySign` and `PlayerConnection`. Given that they essentially share this logic, there should presumably be some meaningful relationship between them; otherwise you just created a dependency between them that didn't need to be there. Are they truly related, or should there be a third place/class where this logic leaves that each of these classes should then depend on?