This document specifically relates to software which is or has been created for the Apertis project. It is important that any code added to an existing project utilises the coding conventions as used by that project, maintaining consistency across that projects codebase.
Coding conventions is a nebulous topic, covering code formatting and whitespace, function and variable naming, namespacing, use of common GLib coding patterns, and other things. Since C is quite flexible, this document mostly consists of a series of patterns (which it’s recommended code follows) and anti-patterns (which it’s recommended code does not follow). Any approaches to coding which are not covered by a pattern or anti-pattern are completely valid.
Guidelines which are specific to GLib are included on this page; guidelines specific to other APIs are covered on their respective pages.
- Use the GLib coding style, with vim modelines.
- Consistently namespace files, functions and types.
- Always design code to be modular, encapsulated and loosely coupled.
- Especially by keeping object member variables inside the object’s private structure.
- Code defensively by adding pre- and post-conditions assertions to all public functions.
- Report all user errors (and no programmer errors) using GError.
- Use appropriate container types for sets of items.
- Document all constant values used in the code.
- Use standard GLib patterns for defining asynchronous methods.
- Do not call any blocking, synchronous functions.
- Do not run blocking operations in separate threads; use asynchronous calls instead.
- Prefer enumerated types over booleans whenever there is the potential for ambiguity between true and false.
- Ensure GObject properties have no side-effects.
- Treat resources as heap-allocated memory and do not leak them.
Using a consistent code formatting style eases maintenance of code, by meaning contributors only have to learn one coding style for all modules, rather than one per module.
Each C and H file should have a vim-style modeline, which lets the programmer’s editor know how code in the file should be formatted. This helps keep the coding style consistent as the files evolve. The following modeline should be put as the very first line of the file, immediately before the copyright comment:
For more information about the copyright comment, see Applying Licensing.
If a file or module does not conform to the code formatting style and needs to be reindented, the following command will do most of the work — but it can go wrong, and the file must be checked manually afterwards:
$ indent -gnu -hnl -nbbo -bbb -sob -bad -nut /path/to/file
To apply this to all C and H files in a module:
$ git ls-files '*.[ch]' | \ $ xargs indent -gnu -hnl -nbbo -bbb -sob -bad -nut
Alternatively, if you have a recent enough version of Clang (>3.5):
$ git ls-files '*.[ch]' | \ $ xargs clang-format -i -style=file
.clang-format file (added to git) in the same directory, containing:
Consistent and complete namespacing of symbols (functions and types) and files is important for two key reasons:
- Establishing a convention which means developers have to learn fewer symbol names to use the library — they can guess them reliably instead.
- Ensuring symbols from two projects do not conflict if included in the same file.
The second point is important — imagine what would happen if every project
exported a function called
create_object(). The headers defining them could
not be included in the same file, and even if that were overcome, the programmer
would not know which project each function comes from. Namespacing eliminates
these problems by using a unique, consistent prefix for every symbol and
filename in a project, grouping symbols into their projects and separating them
The conventions below should be used for namespacing all symbols. They are the same as used in other GLib-based projects, so should be familiar to a lot of developers:
- Functions should use
- Structures, types and objects should use
- Macros and #defines should use
- All symbols should be prefixed with a short (2–4 characters) version of the namespace.
- All methods of an object should also be prefixed with the object name.
Additionally, public headers should be included from a subdirectory, effectively
namespacing the header files. For example, instead of
#include <abc.h>, a
project should allow its users to use
For example, for a project called ‘Walbottle’, the short namespace ‘Wbl’ would be chosen. If it has a ‘schema’ object and a ‘writer’ object, it would install headers:
(The use of
$API_MAJOR above is for
For the schema object, the following symbols would be exported (amongst others), following GObject conventions:
Modularity, encapsulation and loose coupling are core computer science concepts which are necessary for development of maintainable systems. Tightly coupled systems require large amounts of effort to change, due to each change affecting a multitude of other, seemingly unrelated pieces of code. Even for smaller projects, good modularity is highly recommended, as these systems may grow to be larger, and refactoring for modularity takes a lot of effort.
Assuming the general concepts of modularity, encapsulation and loose coupling are well known, here are some guidelines for implementing them which are specific to GLib and GObject APIs:
- The private structure for a GObject should not be in any header files (whether private or public). It should be in the C file defining the object, as should all code which implements that structure and mutates it.
- libtool convenience libraries should be used freely to allow internal code
to be used by multiple public libraries or binaries. However, libtool
convenience libraries must not be installed on the system. Use
Makefile.amto declare a convenience library; not
- Restrict the symbols exported by public libraries by using
my_library_LDFLAGS = -export-symbols my-library.symbols, where
my-library.symbolsis a text file listing the names of the functions to export, one per line. This prevents internal or private functions from being exported, which would break encapsulation. See Exposing and Hiding Symbols.
- Do not put any members (e.g. storage for object state or properties) in a
public GObject structure — they should all be encapsulated in a private
structure declared using
- Do not use static variables inside files or functions to preserve function state between calls to it. Instead, store the state in an object (e.g. the object the function is a method of) as a private member variable (in the object’s private structure). Using static variables means the state is shared between all instances of the object, which is almost always undesirable, and leads to confusing behaviour.
Pre- and post-condition assertions
An important part of secure coding is ensuring that incorrect data does not propagate far through code — the further some malicious input can propagate, the more code it sees, and the greater potential there is for an exploit to be possible.
A standard way of preventing the propagation of invalid data is to check all inputs to, and outputs from, all publicly visible functions in a library or module. There are two levels of checking:
- Assertions: Check for programmer errors and abort the program on failure.
- Validation: Check for invalid input and return an error gracefully on failure.
Validation is a complex topic, and is handled using
GErrors. The remainder of this section discusses pre- and
post-condition assertions, which are purely for catching programmer errors. A
programmer error is where a function is called in a way which is documented as
disallowed. For example, if
NULL is passed to a parameter which is documented
as requiring a non-
NULL value to be passed; or if a negative value is passed
to a function which requires a positive value. Programmer errors can happen on
output too — for example, returning
NULL when it is not documented to, or not
setting a GError output when it fails.
Adding pre- and post-condition assertions to code is as much about ensuring the
behaviour of each function is correctly and completely documented as it is about
adding the assertions themselves. All assertions should be documented,
preferably by using the relevant
The pre-conditions should check each parameter at the start of the function,
before any other code is executed (even retrieving the private data structure
from a GObject, for example, since the GObject pointer could be
post-conditions should check the return value and any output parameters at the
end of the function — this requires a single return statement and use of
to merge other control paths into it. See
for an example.
A fuller example is given in this writeup of post-conditions.
Any kind of runtime failure (anything which is not a
programmer error) must be handled by
GError** parameter in the function, and setting a useful and
relevant GError describing the failure, before returning from the function.
Programmer errors must not be handled using GError: use assertions,
pre-conditions or post-conditions instead.
GError should be used in preference to a simple return code, as it can convey more information, and is also supported by all GLib tools. For example, introspecting an API with GObject introspection will automatically detect all GError parameters so that they can be converted to exceptions in other languages.
Printing warnings to the console must not be done in library code: use a GError, and the calling code can propagate it further upwards, decide to handle it, or decide to print it to the console. Ideally, the only code which prints to the console will be top-level application code, and not library code.
Any function call which can take a
GError**, should take such a parameter,
and the returned GError should be handled appropriately. There are very few
situations where ignoring a potential error by passing
NULL to a
parameter is acceptable.
The GLib API documentation contains a full tutorial for using GError.
GLib provides several container types for sets of data:
It has been common practice in the past to use GList in all situations where a sequence or set of data needs to be stored. This is inadvisable — in most situations, a GPtrArray should be used instead. It has lower memory overhead (a third to a half of an equivalent list), better cache locality, and the same or lower algorithmic complexity for all common operations. The only typical situation where a GList may be more appropriate is when dealing with ordered data, which requires expensive insertions at arbitrary indexes in the array.
If linked lists are used, be careful to keep the complexity of operations on
them low, using standard CS complexity analysis. Any operation which uses
is almost certainly wrong. For example, iteration over a GList should be
implemented using the linking pointers, rather than a incrementing index:
Using an incrementing index instead results in an exponential decrease in performance (O(2×N^2) rather than O(N)):
The performance penalty comes from
which both traverse the list (O(N)) to perform their operations.
Implementing the above with a GPtrArray has the same complexity as the first (correct) GList implementation, but better cache locality and lower memory consumption, so will perform better for large numbers of elements:
Do not use constant values in code without documenting them. These values can be known as ‘magic’ values, because it is not clear how they were chosen, what they depend on, or when they need to be updated.
Magic values should be:
- defined as macros using
#define, rather than being copied to every usage site;
- all defined in an easy-to-find-location, such as the top of the source code file; and
- documented, including information about how they were chosen, and what that choice depended on.
One situation where magic values are used incorrectly is to circumvent the type system. For example, a magic string value which indicates a special state for a string variable. Magic values should not be used for this, as the software state could then be corrupted if user input includes that string (for example). Instead, a separate variable should be used to track the special state. Use the type system to do this work for you — magic values should never be used as a basic dynamic typing system.
Long-running blocking operations should not be run such that they block the UI in a graphical application. This happens when one iteration of the UI’s main loop takes significantly longer than the frame refresh rate, so the UI is not refreshed when the user expects it to be. Interactivity reduces and animations stutter. In extreme cases, the UI can freeze entirely until a blocking operation completes. This should be avoided at all costs.
Similarly, in non-graphical applications that respond to network requests or D-Bus inter-process communication, blocking the main loop prevents the next request from being handled.
There are two possible approaches for preventing the main loop being blocked:
- Running blocking operations asynchronously in the main thread, using polled I/O.
- Running blocking operations in separate threads, with the main loop in the main thread.
The second approach (see Threading typically leads to
complex locking and synchronisation between threads, and introduces many bugs.
The recommended approach in GLib applications is to use asynchronous operations,
Asynchronous operations must be implemented everywhere for this approach to
work: any use of a blocking, synchronous operation will effectively make all
calling functions blocking and synchronous too.
Key principles for using them:
- Never call synchronous methods: always use the
- Never use threads for blocking operations if an asynchronous alternative exists.
- Always wait for an asynchronous operation to complete (i.e. for its
GAsyncReadyCallbackto be invoked) before starting operations which depend on it.
- Never use a timeout (
g_timeout_add()) to wait until an asynchronous operation ‘should’ complete. The time taken by an operation is unpredictable, and can be affected by other applications, kernel scheduling decisions, and various other system processes which cannot be predicted.
- Never use a timeout (
Enumerated types and booleans
In many cases, enumerated types should be used instead of booleans:
- Booleans are not self-documenting in the same way as enums are. When reading code it can be easy to misunderstand the sense of the boolean and get things the wrong way round.
- They are not extensible. If a new state is added to a property in future, the boolean would have to be replaced — if an enum is used, a new value simply has to be added to it.
This is documented well in the article Use Enums Not Booleans.
Properties on GObjects are a key feature of GLib-based object orientation. Properties should be used to expose state variables of the object. A guiding principle for the design of properties is that (in pseudo-code):
my_object in exactly the same state as it was originally.
Specifically, properties should not act as parameterless methods, triggering
state transitions or other side-effects.
As well as memory leaks, it is possible to leak resources such as GLib timeouts, open file descriptors or connected GObject signal handlers. Any such resources should be treated using the same principles as allocated memory.
For example, the source ID returned by
must always be stored and removed (using
when the owning object is finalised. This is because it is very rare that we can
guarantee the object will live longer than the timeout period — and if the
object is finalised, the timeout left uncancelled, and then the timeout
triggers, the program will typically crash due to accessing the object’s memory
after it’s been freed.
Similarly for signal connections, the signal handler ID returned by
should always be saved and explicitly disconnected
unless the object being connected is guaranteed to live longer than the object
being connected to (the one which emits the signal):
Other resources which can be leaked, plus the functions acquiring and releasing them (this list is non-exhaustive):