Bug 16676 – Apply GTK coding style to WebKit Gtk public headers

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Michael Natterer]

Summary says it all :)

[Michael Natterer]

Created attachment 18191 [details]
Apply GTK coding style

[Darin Adler]

Comment on attachment 18191 [details]
Apply GTK coding style

Seems smart to apply GTK coding style to the headers. But seems strange to do it inside the files. I'll let the GTK WebKit hackers make the call, though.

[Alp Toker]

Comment on attachment 18191 [details]
Apply GTK coding style

Mitch,

I've been thinking about this one the last couple of days.

There are two reasons why I don't think we should do this (yet):

(1) The patch makes indentation dependent on the length of various types names and variables names that we're planning to change pretty soon. So it would make future patches much larger than necessary. We want to keep patches as small as possible to ease reviewing and to ease merging, since several developers currently maintain their own branches and push regularly.

(2) The Apple guys often help out in maintaining this code, which is very valuable to the GTK+ port. I think they'd be less likely to casually fix our build if they had to mess about with whitespace in surrounding unrelated lines.

The GTK+ coding style is more aesthetic but I don't think it's worthwhile given these factors.

It may make sense to apply the header changes after the API has stabilised.

It probably makes sense to apply the part of the patch to GtkLauncher's main.c right now. I can apply a reduced version of the patch to do that if you like.

[Michael Natterer]

Sure, please go ahead and apply the patch to the launcher please, better
that than nothing :)

However, I don't really agree on the "we want to keep patches as small
as possible" argument. If you apply that policy, the coding stlye will
always be slightly messed up. Also, there is never a moment where you
can "safely" change a project's coding style. People always have
patches and branches.

I vote for better now than later as the API is guaranteed to grow,
and if it's in the right style now people are at least shown what
the right style is, rather than having their commits changed at
some later point and sitting on even larger patches to a bigger
code base which don't apply any longer.

[Alp Toker]

Thanks for your comments Mitch -- I think we should keep this debate open. There are strong arguments in each direction from both camps, one experienced in WebKit/C++ and the other in GTK+/C.

Our project sits at the borderline and the current coding style is a compromise that's worked quite well. As the project grows I'd be happy to re-evaluate this decision.

[Christian Dywan]

Created attachment 21564 [details]
Apply gtk+ style to headers

This patch applies Gtk+ coding style to public webkit headers. It doesn't touch any implementation or private files.

[Darin Adler]

Comment on attachment 21564 [details]
Apply gtk+ style to headers

rs=me

[Christian Dywan]

Comment on attachment 21564 [details]
Apply gtk+ style to headers

Committed in r34477.