[lkml]   [2000]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] 2.3.41 - cleanup file_operations structs
    James Manning <> writes:
    > FWIW, I chose the gcc syntax since it was already very prevelant
    > in the kernel... my patch simply finishes conversion for file_ops :)
    > 1) I'd hate adding a #define to each file or create another .h dep
    > 2) it's a simple perl script to make this change later on if it
    > proves to be desirable.

    Fair enough. Given the similarity between the gcc initializer syntax
    and the C labelled statement syntax, I don't think the Perl script
    would look simple if I had to write it. But many people are better
    Perl coders than I.

    > As it stands, gcc is still our baseline compiler so I'd rather
    > keep things compatible with that my design, not by #ifdef.

    I agree that the macro approach is ugly, it just seems wiser to think
    these things through before we have lots of designated initializers in
    the kernel, rather than after, and it wasn't clear from your messages
    that you had considered this issue.

    > While I
    > personally like the C99 syntax better, I think it's important to at
    > least get named initilizers in place first, and we can either change
    > or abstract out the interface later on. First thing's first :)

    Yup, generally I'm in favour of the patch.

    David Wragg

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:56    [W:3.300 / U:0.012 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site