User Tools

Site Tools


design:coding_style_guide

This is an old revision of the document!


Coding style guide

This style guide is meant to be loosely adhered to when writing libraries for the Wonderful ecosystem.

Note that this is very much a draft - I'm trying to codify what I learned from the past few years… please remain patient.

Naming

As everyone downstream will end up using the hardware libraries' API, consistency in naming is among the most important things to pay attention to. Formatting can be fixed for free; naming changes are expensive.

Global identifiers

Regarding cases:

  • Global C function names are written with snake_case. This allows easily distinguishing them from function names in more modern languages on top of the C API, which tend to use PascalCase or camelCase.
  • Global C/C++ macro definitions are written with UPPER_SNAKE_CASE.
    • One exception is for macros which serve the role of functions - the line between a static inline function and a macro can sometimes be very thin.

Regarding names:

  • Global identifiers should be prefixed with the name of the library.
    • For example, if you're working on libws, a global function name should be ws_function(void), not function(void).
    • Similarly, for libwsx, the name is wsx_function(void); even if it also targets the ws platform, the library is different, so the prefix changes likewise.
  • Following from that, categorizing functions should be done prefix-first, but beyond the category of the function, an operation should be written like a sentence.
    • For example, ws_screen_fill_tiles is appropriate, not ws_screen_tiles_fill or ws_fill_screen_tiles.
  • Some macros can refer to a type; for readability and to distinguish it from function categories, these should be suffixed.
    • For example, an I/O port will be referred to as WS_DISPLAY_BORDER_PORT, and a mask for WS_DISPLAY_BORDER_COLOR© will be referred to as WS_DISPLAY_BORDER_COLOR_MASK.

Code formatting

void braces_on_the_same_line(int always) {
    if (single_line)
        no_braces_is_fine;
    
    if (multi_line) {
        braces_on_the_same_line(1);
        
        // ... but treat sizeof, etc. like a function.
        memcpy(&b, &a, sizeof(a));
    }
}

Commenting

Use Doxygen-compatible formatting for all user-facing documentation comments:

  • Javadoc-style /** .. */ blocks for documenting functions, types, macros and/or where larger comment blocks are warranted;
  • For one-line comments, such as on enumerated types, ///< can be used.

Standards adherence

For target code on most platforms, it is acceptable to depend on a feature even if it is only available in modern versions of GCC and Clang.

Special consideration may need to be taken for platforms where the GNU/LLVM compiler is not necessarily the best open source option, such as the 8086, SM83 or Z80.

Best practices

  • Where the size of a variable matters, use stdint.h and stdbool.h types (int16_t, uint32_t etc).
    • For 8-bit platforms, rely on these more often; int tends to be 16-bit on those platforms, which is slower to process than uint8_t.
  • For functions which return a “succeeded” flag, use bool. For functions which return an error code, use int and return a non-negative value on success or a negative value on failure.

References

design/coding_style_guide.1746343022.txt.gz · Last modified: 2025/05/04 07:17 by asie