X-Git-Url: http://git.openbox.org/?p=dana%2Fopenbox.git;a=blobdiff_plain;f=HACKING;h=e450bba39e8f5a0a3b351b1e09bf1197c1d8e914;hp=3433ad67f608c15451ae84d9ffbc41b79df94931;hb=fa0ae17adbc8f73b707c33836d37841e81b9303a;hpb=bb649ac996a8dddb315b3283d547a35c26a24afc diff --git a/HACKING b/HACKING index 3433ad6..e450bba 100644 --- a/HACKING +++ b/HACKING @@ -1,19 +1,13 @@ -Woop, a HACKING document for openbox at last! - dirs: - kernel - core of the WM + openbox - core of the WM render - librender, rendering routines for the WM and for apps parser - libparser, for parsing config files Beware the Client.transient_for. It can be set to a !NULL value of TRAN_GROUP, which is not a valid pointer. You must ALWAYS check for TRAN_GROUP before -following transient_for. When TRAN_GROUP is found, Client.group will always -be !NULL. Some smart action should be taken using all members of the group in -this case. - Smart action idea: - Skip over members of the group that are also transients of the group - (have Client.transient_for set to TRAN_GROUP). These windows are not - ancestors and using them will also end up causing infinite loops! +following transient_for. However if it is transient for the group, this +excludes other windows whom are transient for the group, and windows which +are children of the window (infinite loops would result)! When using coordinates/sizes of windows, make sure you use the right area. The Client.area rect is the reference point and size of the *CLIENT* window. This @@ -33,8 +27,7 @@ not all, guidelines: lines * most other constructs should have braces on the same line as the statement + * else appears on a new line, just like an if * when in doubt look at the rest of the source * vim users can use "set expandtab tabstop=4 shiftwidth=4 softtabstop=4" for some of this - -