[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:0.100 / U:0.336 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site