lkml.org 
[lkml]   [2019]   [Sep]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [Ksummit-discuss] [PATCH v2 3/3] libnvdimm, MAINTAINERS: Maintainer Entry Profile
    On Wed, Sep 11, 2019 at 3:11 PM Jens Axboe <axboe@kernel.dk> wrote:
    >
    > On 9/11/19 12:43 PM, Dan Carpenter wrote:
    > > On Wed, Sep 11, 2019 at 08:48:59AM -0700, Dan Williams wrote:
    > >> +Coding Style Addendum
    > >> +---------------------
    > >> +libnvdimm expects multi-line statements to be double indented. I.e.
    > >> +
    > >> + if (x...
    > >> + && ...y) {
    > >
    > > That looks horrible and it causes a checkpatch warning. :( Why not
    > > do it the same way that everyone else does it.
    > >
    > > if (blah_blah_x && <-- && has to be on the first line for checkpatch
    > > blah_blah_y) { <-- [tab][space][space][space][space]blah
    > >
    > > Now all the conditions are aligned visually which makes it readable.
    > > They aren't aligned with the indent block so it's easy to tell the
    > > inside from the if condition.
    > >
    > > I kind of hate all this extra documentation because now everyone thinks
    > > they can invent new hoops to jump through.
    >
    > FWIW, I completely agree with Dan (Carpenter) here. I absolutely
    > dislike having these kinds of files, and with subsystems imposing weird
    > restrictions on style (like the quoted example, yuck).
    >
    > Additionally, it would seem saner to standardize rules around when
    > code is expected to hit the maintainers hands for kernel releases. Both
    > yours and Martins deals with that, there really shouldn't be the need
    > to have this specified in detail per sub-system.

    So this is *the* point of the exercise.

    I picked up this indentation habit a long while back in response to
    review feedback on a patch to a subsystem that formatted code this
    way. At the time CodingStyle did not contradict the maintainer's
    preference, so I adopted it across the board.

    Now I come to find that CodingStyle has settled on clang-format (in
    the last 15 months) as the new standard which is a much better answer
    to me than a manually specified style open to interpretation. I'll
    take a look at getting libnvdimm converted over.

    If we can take that further and standardize all of the places where
    contributors see variations across subsystems on the fundamental
    questions of style, timing, follow-up, and unit test invocation the
    Maintainer Entry Profile can be superseded with common guidelines.

    Otherwise, yes I completely agree with you that a "Maintainer Entry
    Profile" is a sad comment on the state of what contributors need to
    navigate, but that's today's reality that needs to be addressed.

    \
     
     \ /
      Last update: 2019-09-12 09:43    [W:6.079 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site