login
Header Space

 
 

Re: [PATCH] 0/3 checkpatch updates, new checkfiles script

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Erez Zadok <ezk@...>
Cc: Ingo Molnar <mingo@...>, <rdunlap@...>, <jschopp@...>, <linux-kernel@...>
Date: Thursday, October 11, 2007 - 9:47 am

On Sun, Oct 07, 2007 at 03:05:47PM -0400, Erez Zadok wrote:

Yes, this support seems to be wholy broken, as a non emacs user I had
failed to test it correctly as I added the --file option.  Bad Andy.

In testing it I note we are emitting wholy the wrong line number, and
the filename was off as you fixed up in your patch.


As I understand things this is called emacs mode because the emacs
buffer mode expects the filename:line:message format with the long error
to follow.  It seems pretty emacs specific.  So for now I'll leave it
named that.  If people convince me its --compiler-format or something we
can add that as an alias later.

I have hopefully sorted the main problems with it and will push out an
update for testing.

-apw
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH] 0/3 checkpatch updates, new checkfiles script, Andy Whitcroft, (Thu Oct 11, 9:47 am)
speck-geostationary