[PATCH 1.12] A coding style for the server
Daniel Stone
daniel at fooishbar.org
Wed Jan 18 20:52:40 PST 2012
Hi,
On 19 January 2012 06:42, Chase Douglas <chase.douglas at canonical.com> wrote:
> Part of the problem of a coding style is the ongoing maintenance. The
> kernel handles this through the check_patch script. This gives rise to
> the following questions:
>
> 1. Should we just go with linux style, so we can use check_patch without
> any modifications?
No - the main problem is that the kernel uses tabs with ts=8 rather
than four-space indents. While I'm not opposed to it on principle, it
only works for them because they're so aggressive about keeping
functions small and using early return/break-of-control rather than
deeply nested ifs. Our code would be utterly, utterly unreadable if
formatted to ts=8 while retaining the wrapping at 80 characters.
> 2. If not, will we fork check_patch or make our own?
>
> 3. Who will enforce style? I believe if we have a script we can add a
> commit hook to the git server, but that may be too heavy-handed.
Yeah - maybe it's just something we should try to catch during the
review process, rather than enforce?
> 4. If no one will enforce style, do we want to periodically run indent
> to fix things. Maybe once per cycle after the merge window closes (as
> you suggest here, but I begged for mercy against this time :)?
Heh, sure. :)
Cheers,
Daniel
More information about the xorg-devel
mailing list