Openwall GNU/*/Linux - a small security-enhanced Linux distro for servers
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 8 Apr 2015 17:29:30 +0800
From: Kai Zhao <loverszhao@...il.com>
To: john-dev@...ts.openwall.com
Subject: Re: Coding Style

> 1) it should be possible to convert code to it automatically,
>   a) to convert existing code (e.g. all jumbo);
>   b) to convert new code before committing it to the repo;

I have done some research and experiment on the automatic tools. Here are
the tools:

1. Commercial

    - Jindent
    - Polystyle
    - Pretty Printer
    - ScrEnc
    - SQCBW

2. Free

    - GNU Indent
    - Artistic Style
    - UniversalIndentGUI
    - Uncrustify
    - GreatCode

However, I think we'd better not use these tools. There are two reasons.
Firstly, linux kernel which our coding style very similar to does not use
it.
Secondly, the tools is arbitrary. E.g., if we set the max column 80, but
there are some cases which we may exceed column 80.

I think kernel does very well on the coding style. They use the script
checkpatch.pl to check the coding style, and add it to the git pre-commit
hooks, so each patch will be checked before commit.

The script generates errors and warnings. It depends on the committers to
judge whether change their code.


Thanks,

Kai

[ CONTENT OF TYPE text/html SKIPPED ]

Powered by blists - more mailing lists

Your e-mail address:

Powered by Openwall GNU/*/Linux - Powered by OpenVZ