Cleanup warnings, now that the compiler understands $NON-NLS-1$ in role files.
4 remaining warnings are bogus:
 - discouraged access in CompilationThreadWatcher (our bad)
 - missing entry in build.properties (PDE's bad)
4 files changed