Comment 0 for bug 1252490

Revision history for this message
Pupu Toivonen (scolphoy) wrote : Alignment issues with uncrustify

The Uncrustify Code Beautifier tool systematically aligns variable names, although it might not be reasonable in all places.

 struct sockaddr_in6 hipfw_addr;
-int n = 0;
+int n = 0;

There is the variable align_var_def_thres in Uncrustify config file, which can be used to set an upper bound for how many spaces can be used for aligning.

Also there is a known bug in Uncrustify (http://sourceforge.net/p/uncrustify/bugs/522/) that causes it to mistake a special case of variable definitions (a static pointer to an array of function pointers) for a function call, and apply rules related to function calls instead of rules related to variable definitions.

-static int (*action_handler[])(struct hip_common *, int action,
+static int(*action_handler[]) (struct hip_common *, int action,
                                const char *opt[], int optc, int send_only) = {