Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 09 Apr 2015 18:14:10 +0200
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: Coding Style

On 2015-04-09 17:58, Solar Designer wrote:
> On Thu, Apr 09, 2015 at 08:58:42PM +0800, Kai Zhao wrote:
>> Even though
>> Kernel style states that "Never break user-visible strings such as printk
>> messages even it exceeds column 80" , the checkpatch.pl can't find this
>> problem, since the situation maybe complex. I think it would be better to
>> manually review these strings.
> 
> FWIW, while I understand the rationale behind this rule, I dislike it.

Is this about grepping for strings you saw on screen? I'm sure I hold
the world record in doing that with JtR, and I really never saw this as
a problem. Sure, I've had to re-phrase my grep at times but that's not a
big deal. If it is a problem, this is too:

#define ALGORITHM_NAME                  "MD5 " MD5_ALGORITHM_NAME

as is this:

        fprintf(stderr, "Waiting for %d child%s to terminate\n",
            waiting_for, waiting_for == 1 ? "" : "ren");

Rewriting them so people can grep for the actual strings printed, would
be plain silly.

> It looks like I'm in the minority, so I won't object to this rule in
> jumbo, but I'm not eager to introduce it in core.

I'm not too eager to introduce it in Jumbo either.

magnum


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.