This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Check style of mark up and comment


>>>>> "Yao" == Yao Qi <yao@codesourcery.com> writes:

Yao> This patch adds two new hooks to check markup and comment style.
Yao> Missing marker up and incorrect comment style are the most common
Yao> mistakes found in review.  It is useful to check them when writing
Yao> code.

Thanks.

Yao> +   '(("\\<warning[ ]*\\(\([^_]\\)" 1 font-lock-warning-face t))))

Should 'error' be there as well?

Yao> +  ;; A space shoud follow "/*", otherwise report a warning.

Typo: "should".

This patch is ok with the typo fix.  It is also ok if you want to add
"error", up to you.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]