Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 9 Apr 2015 23:02:38 +0800
From: Kai Zhao <>
Subject: Re: Coding Style

> Can you please name plain text attachments *.txt?

Get it. I will the next time.

> This one definitely is a false positive:
> ERROR: space prohibited before open square bracket '['
> #50: FILE: john_core/src/BF_std.c:50:
> +#define INDICES                                [BF_N]

Yes, this maybe the difference between john and kernel. Kernel only permits
such cases:

1. with a type on the left -- int [] a;
2. at the beginning of a line for slice initialisers -- [0...10] = 5,
3. inside a curly brace -- = { [0...10] = 5 }

It is notable that the errors generated by maybe not really
errors and generates lots of errors for current kernel which
still there.

> And why is it that only line 50 triggers this message, but not the
> subsequent lines or similar lines in DES_bs_b.c?

The attachment is not the original output which is 10.6MB for john jumbo and
130KB for john core. Since there are many errors and warnings, I select
some typical problems and put them in the attachment. Sorry for not making
it clear.



Content of type "text/html" skipped

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.