lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20071026081752.GA1232@elte.hu> Date: Fri, 26 Oct 2007 10:17:52 +0200 From: Ingo Molnar <mingo@...e.hu> To: "H. Peter Anvin" <hpa@...or.com> Cc: linux-kernel@...r.kernel.org Subject: [patch] raid6: generate raid6tables.c with proper style Subject: raid6: generate raid6tables.c with proper style From: Ingo Molnar <mingo@...e.hu> nitpicking from the coding style police: generate a raid6tables.c that does not show up in my editor all red ;-) this also improves the code quality metric of drivers/md/ by a factor of 10: errors lines of code errors/KLOC drivers/md/ [old] 9349 45543 205.2 drivers/md/ [new] 1062 45551 23.3 while humans are not likely to read that file, lets just do this to make life easier for automated quality checking ... Signed-off-by: Ingo Molnar <mingo@...e.hu> --- drivers/md/mktables.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) Index: linux/drivers/md/mktables.c =================================================================== --- linux.orig/drivers/md/mktables.c +++ linux/drivers/md/mktables.c @@ -69,7 +69,9 @@ int main(int argc, char *argv[]) for ( j = 0 ; j < 256 ; j += 8 ) { printf("\t\t"); for ( k = 0 ; k < 8 ; k++ ) { - printf("0x%02x, ", gfmul(i,j+k)); + if (k) + printf(" "); + printf("0x%02x,", gfmul(i,j+k)); } printf("\n"); } @@ -86,7 +88,9 @@ int main(int argc, char *argv[]) printf("\t"); for ( j = 0 ; j < 8 ; j++ ) { exptbl[i+j] = v; - printf("0x%02x, ", v); + if (j) + printf(" "); + printf("0x%02x,", v); v = gfmul(v,2); if ( v == 1 ) v = 0; /* For entry 255, not a real entry */ } @@ -102,7 +106,9 @@ int main(int argc, char *argv[]) printf("\t"); for ( j = 0 ; j < 8 ; j++ ) { invtbl[i+j] = v = gfpow(i+j,254); - printf("0x%02x, ", v); + if (j) + printf(" "); + printf("0x%02x,", v); } printf("\n"); } @@ -115,7 +121,9 @@ int main(int argc, char *argv[]) for ( i = 0 ; i < 256 ; i += 8 ) { printf("\t"); for ( j = 0 ; j < 8 ; j++ ) { - printf("0x%02x, ", invtbl[exptbl[i+j]^1]); + if (j) + printf(" "); + printf("0x%02x,", invtbl[exptbl[i+j]^1]); } printf("\n"); } - 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