[PATCH] [clang-tidy] Add an option to replace unrecognized namespace ending comments.

Daniel Jasper djasper at google.com
Thu Mar 5 04:58:02 PST 2015


I think adding this option is a way of copping out of fixing this correctly.

For now, I'd be fine saying always to overwrite such a comment. Either a project, enforcing such comment, then they won't have a different comment in the same place or they don't. If clang-tidy complains about / overwrites one of your comments, you'll reinstate it at a better location prior to submit.

A next step might be to analyze whether a comment in that place is meant to close the namespace, e.g. by checking whether its content contains at least one of "namespace" or <namespace-name>.


http://reviews.llvm.org/D8078

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list