== Code Style == * Maximum Line Width is 100 Characters. The reason is not about people with low-res screens but rather sticking to 100 columns prevents you from easily nesting more than one level of if statements or other code blocks. * Use 4 Space Tabs with whitespace fillers to Indent. Never use tabs. * No Trailing Whitespace * Follow the GNU-Style guidelines. We won't go through all of them here. Please use the same indentation as other developers. Follow the style used in GNU Midnight Commander: indent --gnu-style --format-first-column-comments --indent-level4 --brace-indent0 --line-length100 --no-tabs --blank-lines-after-procedures or in short notation: indent -gnu -fc1 -i4 -bli0 -nut -bap -l100 == Programming Tips == === Comments === Comments should always use the standard C syntax. C++ style comments are not currently allowed. The lines before a comment should be empty. If the comment directly belongs to the following code, there should be no empty line after the comment, except if the comment contains a summary of multiple following code blocks. '''This is right:''' {{{ /* * This is a multi line comment, * Delete '\n' char. * Note that edit_delete() will not corrupt anything if called while * cursor position is EOF. */ (void) edit_delete (edit); /* This is a one line comment. Allocate additional memory. */ mem = (char *) malloc (memneed); /** * @brief This is a doxygen comment. * * This is a more detailed explanation of * this simple function. * * @param[in] param1 The parameter value of the function. * * @param[out] result1 The result value of the function. * * @return 0 on success and -1 on error. */ int example (int param1, int *result1); }}} '''This is wrong:''' {{{ //This is a one line comment. /*This is a one line comment.*/ /* This is a multi line comment, with some more words...*/ }}} === Code blocks (if, for, case...) === Always follow an 'if' keyword with a space but don't include additional spaces following or preceding the parentheses in the conditional. '''This is right:''' {{{ if (i == 0) }}} '''This is wrong:''' {{{ if ( i == 0 ) if (0 == i) }}} Always insert a space between the name and left parentheses when invoking functions. '''This is right:''' {{{ do_example (int param1, int *result1); }}} '''This is wrong:''' {{{ do_example(int param1, int *result1); }}} Braces for code blocks used by '''for, if, switch, while, do..while''', etc. should begin on the next line after the statement keyword and end on a line of their own. Functions are different and the beginning left brace should be located in the first column on the next line. If the beginning statement has to be broken across lines due to length, the beginning brace should be on a line of its own. '''This is right:''' {{{ if (xterm_flag && xterm_title) { path = strip_home_and_password (current_panel->cwd); ... } for (j = 0; j < 10; j++) { for (i = 0; str_options[i].opt_name != NULL; i++) g_free (*str_options[i].opt_addr); } }}} '''This is wrong:''' {{{ if (xterm_flag && xterm_title) { path = strip_home_and_password (current_panel->cwd); ... } if (xterm_flag && xterm_title) { path = strip_home_and_password (current_panel->cwd); } for (k = 0; k < 10; k++) for (j = 0; j < 10; j++) for (i = 0; str_options[i].opt_name != NULL; i++) g_free (*str_options[i].opt_addr); }}} Use "goto" only when necessary. "goto"s are evil, but they can greatly enhance readability and reduce memory leaks when used as the single exit point from a function. '''This is right:''' {{{ { if (link_type == LINK_HARDLINK) { src = g_strdup_printf (_("Link %s to:"), str_trunc (fname, 46)); dest = input_expand_dialog (_("Link"), src, MC_HISTORY_FM_LINK, ""); if (!dest || !*dest) goto cleanup; ... ... } ... ... cleanup: g_free (src); g_free (dest); } }}} Use your best judgement and choose the more readable option. '''This is right:''' {{{ bytes = read (fd, &routine.pointer, sizeof (routine)); if (bytes == -1 || (size_t) bytes < (sizeof (routine))) }}} '''This is wrong:''' {{{ if (bytes = read (fd, &routine.pointer, sizeof (routine)) == -1 || (size_t) bytes < (sizeof (routine))) if (read (from_parent_fd, &i, sizeof (int)) != sizeof (int)) return NULL; }}} typical structure of *.c file {{{ /*** global variables ****************************************************************************/ }}} {{{ /*** file scope macro definitions ****************************************************************/ }}} {{{ /*** file scope type declarations ****************************************************************/ }}} {{{ /*** file scope variables ************************************************************************/ }}} {{{ /*** file scope functions ************************************************************************/ /* --------------------------------------------------------------------------------------------- */ }}} {{{ /* --------------------------------------------------------------------------------------------- */ /*** public functions ****************************************************************************/ /* --------------------------------------------------------------------------------------------- */ }}}