[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4d8e3fd30801101435l228a0d0ydd0174e4073400e6@mail.gmail.com>
Date: Thu, 10 Jan 2008 23:35:39 +0100
From: "Paolo Ciarrocchi" <paolo.ciarrocchi@...il.com>
To: "Daniel Walker" <dwalker@...sta.com>
Cc: linux-kernel@...r.kernel.org
Subject: Re: [x86] list of style offenders
On Jan 10, 2008 9:03 PM, Daniel Walker <dwalker@...sta.com> wrote:
>
>
> I was playing with checkpatch.pl ;) .. I made a list of style offenders
> inside the arch/x86/ sub-directory. I thought it might be helpful to
> prioritize any style clean ups that people might want to do for x86 ..
> OTOH some of these might show imperfections in checkpatch.pl .. Either
> way I thought it was interesting to review ..
>
> These are sorted by errors, then warnings.. I only included the first
> hundred files which includes about the top %30 of the list .. The
> kprobes_64.c error rate is a little miss-leading because several errors
> are found on a single line ..
I just pulled
git://git.kernel.org/pub/scm/linux/kernel/git/x86/linux-2.6-x86 mm
> ./arch/x86/math-emu/fpu_trig.c
> total: 649 errors, 11 warnings, 1845 lines checked
On that tree checkpatch reports:
total: 5 errors, 54 warnings, 1643 lines checked
> ./arch/x86/kernel/kprobes_64.c
> total: 528 errors, 9 warnings, 749 lines checked
total: 0 errors, 0 warnings, 1 lines checked
> ./arch/x86/math-emu/reg_ld_str.c
> total: 392 errors, 17 warnings, 1375 lines checked
total: 8 errors, 39 warnings, 1224 lines checked
> ./arch/x86/kernel/kprobes_32.c
> total: 271 errors, 8 warnings, 756 lines checked
total: 1 errors, 0 warnings, 0 lines checked
> ./arch/x86/math-emu/errors.c
> total: 214 errors, 28 warnings, 739 lines checked
total: 26 errors, 36 warnings, 689 lines checked
So it looks like a lot of errors are fixed in the x86 git tree.
Regards,
--
Paolo
http://paolo.ciarrocchi.googlepages.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists