Re: [RFC] Balsa coding style



PING - any comment from the project admins would be appreciated, also if you think this is complete nonsense.

Am 20.02.16 17:59 schrieb(en) Albrecht Dreß:
Hi all,

the balsa 'HACKING' file states

<snip>
The coding style used in Balsa currently is a bit of a mess.
</snip>

which is actually true... ;-)  I use Eclipse as development platform, and could not find a setting which 
displays /all/ files properly.

The same document states that the indent command

        indent -kr -i4 -psl

shall be used which seems to limit the line length to 72.  This was actually suitable for a 1024x768 screen, 
but seems to be too limited these days IMHO.  Actually, the short line wrapping makes reading the code very 
difficult.

What do you think about extending the indent statement above to a line length of, say, 132, and fixing the 
tab width to e.g. 4, i.e.:

        indent -kr -i4 -l132 -lc132 -ts4 -psl

and re-indent all files in the repository?

An other approach would be some re-factoring, e.g. replace complex statements like

<snip>
        if (!
            (key =
             get_key_from_name(ctx, name, FALSE, accept_low_trust, parent,
                               error))) {
</snip>

by

<snip>
        key = get_key_from_name(ctx, name, FALSE, accept_low_trust, parent,
                                                        error);
        if (key == NULL) {
</snip>

but this implies the risk to introduce errors.

Opinions?

Cheers
Albrecht.

Attachment: pgpXfP6Nda3KY.pgp
Description: PGP signature



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]