Pacman - Contributing

All code should be indented with tabs. (Ignore the use of only spaces in this file.) A tab size of two spaces is used when calculating line widths, which should be a maximum of 80 characters. An EditorConfig file is used to set this project-wide default.

When opening new blocks such as while, if, or for, leave the opening brace on the same line as the beginning of the codeblock. The closing brace gets its own line (the only exception being else). Do not use extra spaces around the parentheses of the block. ALWAYS use opening and closing braces, even if it’s just a one-line block. This reduces future error when blocks are expanded beyond one line.

When declaring a new function, put the opening and closing braces on their own line. Also, when declaring a pointer, do not put a space between the asterisk and the variable name.

Comments should be ANSI-C89 compliant. That means no // Comment style; use only /* Comment */ style.

/* This is a comment */
   NOT
// This is a comment

Return statements should not be written like function calls.

return 0;
   NOT
return(0);

When using strcmp() (or any function that returns 0 on success) in a conditional statement, use != 0 or == 0 and not the negation (!) operator. It reads much cleaner for humans (using a negative to check for success is confusing) and the compiler will treat it correctly anyway.

if(strcmp(a, b) == 0)
   NOT
if(!strcmp(a, b))

Use spaces around almost all arithmetic, comparison and assignment operators and after all ,;: separators.

foobar[2 * size + 1] = function(a, 6);
   NOT
foobar[2*size+1]=function(a,6);

for(a = 0; a < n && n > 0; a++, n--) {}
   NOT
for(a=0;a<n&&n>0;a++,n--) {}

Declare all variables at the start of the block.