Good practice to put 'public' on interface methods, even though it's not necessary? Argument for: if you put 'public' you can just copy/paste the line into an implementing class, which would otherwise be an error. Argument against: it's the default for interfaces (but not for classes that you might paste it into). A suitable decision probably should be recorded on the EGit contributor code guidelines.