Remove static string literals from visitors

This pull request removes all static string literals that would ordinarily be duped for no reason from visitor methods.

I've also added a test case to ensure that no static string literals are ever put in visitor method bodies in future.

cc @fxn