Merge branch 'backport' into work
[openbox.git] / HACKING
blobe450bba39e8f5a0a3b351b1e09bf1197c1d8e914
1 dirs:
2         openbox - core of the WM
3         render - librender, rendering routines for the WM and for apps
4         parser - libparser, for parsing config files
6 Beware the Client.transient_for. It can be set to a !NULL value of TRAN_GROUP,
7 which is not a valid pointer. You must ALWAYS check for TRAN_GROUP before
8 following transient_for. However if it is transient for the group, this
9 excludes other windows whom are transient for the group, and windows which
10 are children of the window (infinite loops would result)!
12 When using coordinates/sizes of windows, make sure you use the right area. The
13 Client.area rect is the reference point and size of the *CLIENT* window. This
14 value is not what you see in any shape or form, and gravity is applied to it to
15 translate it into what you see. The Client.frame.area is the actual position
16 and size of the entire frame. This is usually the value you want to use, unless
17 you are in client.c (probably) and adjusting/using the position or size from
18 the client's perspective.
20 Indentation
21 -----------
22 For openbox, we aim to have consistent coding style. Some, but surely
23 not all, guidelines:
24  * use 4 space indents
25  * tabs should not appear in source files
26  * functions should have the opening and closing braces on their own
27    lines
28  * most other constructs should have braces on the same line as the
29    statement
30  * else appears on a new line, just like an if
31  * when in doubt look at the rest of the source
32  * vim users can use "set expandtab tabstop=4 shiftwidth=4
33    softtabstop=4" for some of this