[lkml]   [2016]   [Jul]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch] crypto: sha256-mb - cleanup a || vs | typo
    [ rare comment rant. I think I'll do this once, and then ignore the discussion ]

    On Fri, Jul 8, 2016 at 9:45 AM, Herbert Xu <> wrote:
    > Nack. As I said the commenting style in the crypto API is the
    > same as the network stack. So unless we decide to change both
    > please stick to the current style.

    Can we please get rid of the brain-damaged stupid networking comment
    syntax style, PLEASE?

    If the networking people cannot handle the pure awesomeness that is a
    balanced and symmetric traditional multi-line C style comments, then
    instead of the disgusting unbalanced crap that you guys use now,
    please just go all the way to the C++ mode.

    In other words, these three models are good:

    /* This is a comment *./

    * This is also a comment, but it can now be cleanly
    * split over multiple lines

    // This can be a single line. Or many. Your choice.

    and they are all obviously visually balanced. Sometimes you want (b)
    even for a single line, if you want the white-space to make it stand
    out more, but you can obviously do that with (c) too, by just
    surrounding it with two empty (comment) lines.

    The (c) form is particularly good for things like enum or structure
    member comments at the end of code, where you might want to align
    things up, but the ending comment marker ends up being visually pretty
    distracting (and lining _that_ up is too much make-believe work).

    There's also another acceptablr traditional multi-line style that
    you'll find in some places, but it's not the common kernel style:

    /* This is an alternate multi-line format
    that isn't horrible, but not kernel style */

    Note how all the above comment styles have a certain visual symmatry
    and balance.

    But no, the networking code picked *none* of the above sane formats.
    Instead, it picked these two models that are just half-arsed

    /* This is disgusting drug-induced
    * crap, and should die

    /* This is also very nasty
    * and visually unbalanced */

    Please. The networking code actually has the *worst* possible comment
    style. You can literally find that (no-no-no) style, which is just
    really horribly disgusting and worse than the otherwise fairly similar
    (d) in pretty much every way.

    I'm not even going to start talking about the people who prefer to
    "box in" their comments, and line up both ends and have fancy boxes of
    stars around the whole thing. I'm sure that looks really nice if you
    are out of your mind on LSD, and have nothing better to do than to
    worry about the right alignment of the asterisks.

    I'd be happy to start moving the whole kernel over to the C++ style,
    it's been many many years since we had compatibility issues and we are
    all used to it by now, even if we weren't all fans originally.

    I really don't understand why the networking people think that their
    particularly ugly styles are fine. They are the most visually
    unbalanced version of _all_ the common comment styles, and have no
    actual advantages.

    So just get rid of the (no-no) and (no-no-no) forms. Not in one big
    go, but as people touch the code, just fix that mess up.


     \ /
      Last update: 2016-07-08 20:01    [W:0.025 / U:18.108 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site