WIP: config_parser file sitting in my tree

Add "visible" filter for windows on current desktop.

Fix error printing when parsing and save the last error.

Errors would have a line number relative to the string of text being parsed,
but if this is in a larger file, then it's useless. So add a callback that
can modify the line number for parsing things nested in a larger document.

This gives proper errors for parsing actions in the xml config files.

Also, change how errors are given back when creating a new action. Instead of
printing it directly, return an error string so the parser can show what line
the invalid action was found on.

Add obt_xml_file_path() and obt_xml_node_string_raw() and don't return text in child nodes.

Add an action for toggling debug output.

- Rename "Debug" action to "Print" for displaying text.
- Add "Debug" action that sets/toggles showing debug messages. It has the
  standard set:<foo> argument where <foo> is on/off/toggle.

read the per-app position x/y values from the right xml node

Add SkipTaskbar/SkipPager actions (bug #3334)

Fix a crash on shutdown/reconfigure.  Clean up the registered action filters properly.

Frees the list and sets the pointer to NULL for empty list.

Remove complicated stuff from config_value.c that we're not using at the moment.

Add functions to set the value of an xml node

Add a function to save a cache file (it makes the directory as needed etc).

add obt_xml_load_cache_file() to load a file in ~/.cache/<domain>/<file>

Make the ConfigValue list type into a gchar** string list instead.

1. I don't want to have to deal with nested lists cuz they just don't need to
   happen (now).

Add obt_xml_path_get_list() that returns a list of xml nodes that match the given path.

<foo>
  <b>hi</b>
  <b>i'm</b>
  <b>a</b>
  <b>list</a>
</foo>

It would return all the b nodes above for the path "foo/b".

don't create empty xml nodes when there is no default value given for them. NULL != ""

Turn the Foo/ToggleFoo/UnFoo actions into a single Foo action with option "set" which is "on", "off", or "toggle"

Use ObConfigValue to parse gravity coords, remove parse special functions from config.[ch]

Rename obt_xml_find_node() to obt_xml_find_sibling(). Add obt_xml_tree_get_node() and xml_tree_delete_node()

Fix interactive actions with the new action running code.

It was running the interactive action init code when running an interactive
action that is already running.

Rename ObActionValue to ObConfigValue.  This holds one of three types.

1. A string
2. A list of other ObConfigValues
3. A list of actions (ObActionList).

The string type can be returned in various ways such as int, enum, fraction, gravity coordiate.

Provide a ways to save the ObConfigValue into a pointer.  This lets you keep
the value contained withing the ObConfigValue in a variable of its basic type.

register filters on startup

TODO:
1. write filters for everything in actions/if.c
2. rm actions/if.c
3. make action(s) to change config options?
   includes key and mouse binding
4. rm -rf *xml*?

parse filters and actionlists inside an action list correctly

Pass a client set to all actions, and make focus cycling use a client set.

Provide operations to get the size of a ObClientSet, and get the clients in it. Also bug fixes.

Add functions to ObClientSet to run a function on each client in the set, and add constness.
Anything that doesnt change/destroy the given set takes a const ObClientSet*.

Generate a fake leave event on actions that move windows (used to generate another enter event?)

Rename the fields in ObActionListRun for clarity

Add action_list_run.c/h with action_list_run() and struct ObActionListRun.

This function runs an action list, taking over from the function that used to
live in action.c.  Now the action.c function will simply run a single action.

The actions and filters now take a const ObActionListRun* which describes
the user event which triggered the actions/filters to run.

The action_list_run() function can also run filters, yay.  However...

TODO:
1. the filters' client sets are not passed to actions yet. do this.
2. make action(s) to change config options
   includes key and mouse binding
3. rm -rf *xml*

Add a means to duplicate ObClientSet

and fix destroy notifications.
- set the destroy notification when making a set with a single window, not
  when making an empty set.
- always add/remove the destroy notification when creating/destroying the hash
  table

Add the minus and contains() operations to ObClientSet, and allow creating a set of any single client.

Fix expand/reduce in the full or empty set cases.

Make empty ObClientSet objects not hold a hash table.

This saves creating one every time we make an empty set (boolean filter
  false result)

make a special "ALL" ObClientSet which contains all clients.

This saves copying all the clients into the set.

IMPORTANT: This client set will test as a "true" boolean regardless of if there
  are clients present or not.
  - We did this to allow for boolean filters.  A boolean filter returns the
    "ALL" set for its true value, and an empty set for its false value.

add the "all" and "target" filters

Pass a user data to the client_set_reduce/expand function which is given to the
callback.

This lets us pass the filter's context in to its test function along with the
  client window, so it can make an intelligent decision about adding/removing
  the client from the current set.

Stick an ObClientSetReduce/ExpandFunc directly in the filter definition.

Saves a redirection to the filter just to go to the client_set_reduce/expand
  function.

Rename actions/all.[ch] and filters/all.[ch] to _all.[ch]

There will actually be an "all" filter so this clashed.

Actions say what kind of filter they would like by default (one window or all)

make empty sets still a ObClientSet* structure and not a NULL.

so we can differentient between a missing set and an empty set.

Use ObClientSet objects in the filter system, so filters work on these sets.

Add ObClientSet data type.  It holds a set of clients !

The sets can be modified by union and intersection, and can have clients
added or removed with a test callback function.
They use a GHashTable underneath.

Keep track of the currently targetted client for user events in event.c/h

Add filters/all.c which will run startup on all the filters to register them

Add a generic filter system similar to the action system (not connected to anything yet)

Rename ObActions* to ObAction* and remove more 3.4-compat action stuff that was missed.

rm some unused fn defns

rm the actions_parse* functions which parsed xml actions

rm some unused struct defns cruft

Parse ObActionsList*s from the config file (out of the xml for now).

- config.c: use the new actions parser
- actions.c: provide the actions parser a way to create a new action (takes
    a hash table of options for it)
- actions/*.c: take a hash table of options instead of an xml subtree
- *: us ObActionsList* instead of a GList* of ObActionAct*s.

TODO:
1. make filters
  openbox/filters/*.c
  openbox/filters.c as interface to them
  store filters in parsed filter tests instead of key/value pairs
2. make action(s) to change config options
  includes key and mouse binding
3. rm -rf *xml*

Add a parser for the action command language.

The parser generates list of actions (complete with filters).  The parser isn't
called anywhere yet, and uses the incomplete actions_act_new() function.

TODO:
The filters now are just text string key:value pairs. But they should
  become intelligent objects like actions are.
The actions need to change their option parsing from XML to the new key:value
  pair lists.

Link X into libobt (Fixes compile on debian).

Ignore enters on windows that are raised over focused window while being mapped.

Use the KeyCode to directly find the modifier mask. (bug #5173)

We were finding the KeySym first, and then converting back to a modifier mask.
But KeySym on a key's release can differ from on its press, and we don't need
them to determine the modmask from the keycode.

[setxkbmap -option "grp:shifts_toggle"] turns Shift_L into XK_ISO_Prev_Group on
key release, and Shift_R into XK_ISO_Next_Group.

check RECT_INTERSECTS_RECT before using RECT_SET_INTERSECTION

Don't make desktop switching interactive when no mods used. (bug #5203)

Make mod state passing more consistent, and always give actions the full state
instead of stripping sometimes. (They ended up expecting it stripped always).

Pick the monitor most relevant to a rectangle more cleverly.

When monitors overlap (this happens with cloning), we were choosing a monitor
to associate with a window, for maximization for example, somewhat arbitrarily.

Now we have a more clever algorithm that considers the configured primary
monitor first, and that does not prefer monitors based on their sizes, but only
only how much of the window is in the monitor, excluding parts that were claimed
by another monitor already.

Select the menu item under the mouse when the mouse is moved (bug #5237)

Make ResizeRelative resize in terms of pixels for windows with resize increments (bug #5228)

If the resize is less than the window's resize increment then we bump it up
to ensure a resize does at least happen.

Make the inactive osd text color fall back to the old osd property, and lastly inherit from the unfocused window color.

Allow an xml tag to be unregistered from the xml parser.

Fix release scripts: test en .po files for correct version. correctly find previous version tag in git and display it.

Fix prompt cleanup (was not going through the list correctly when freeing items)

Create some Openbox release scripts:

release/bugs: Prints a list of bugs that are mentioned in git commits for a
  git revision, since previous release.
  - Very useful for updating the CHANGELOG file!

release/go: Tests a git revision for correct compilation, and prepares files
  for release.
  - Makes the tarball
  - Makes a GPG signature for the tarball
  - Tags the release
  - Spits out URLs to edit and gives the changelog for copy/paste.

release/email: Sends an email to the Openbox mailing list with the changelog
  and details about the release.  Call this with the same parameters used for
  running release/go once it is finished, and the files are uploaded, etc.
  - Also emails mikachu re freshmeat.net

Allow client destroy notify functions to be removed based on the data associated with them.

Properly build path names for TryExec

add menu_sort_entries() which sorts all entries in an ObMenu

this function sorts each group of entries that appear together between two
consecutive separators (or ends of the list)

Add another BSEARCH function that lets you search through an array of objects.

If you give it a value x, the macro lets you pull a value out of each object
that you want to compare to x.

skip empty strings when splitting paths in the environment

hide the menus before shutting them down on reconfigure

Fix for when the menu's update function returns FALSE and no menu is shown.

The ObMenuFrame was not being freed, and the child pointer was being left
pointing to the frame that was not visible.

Quiet build outbut on by default

Fix window width/height going less than 1 on some platforms (bug #5246)

ignore .rej files

Fix maximized windows and such after screen resizings

Hear screen resizes when they don't change the root window (cloning)

Check the return value for launching the startup command.

Copy/paste error?

Fixes for gnome-session 3.

We run by default with a panel (default gnome-panel) and then run Openbox
without any panel if one cannot be found.

- Adds a fallback session for if a panel (default gnome-panel) not found.
- Removes notifications as a requirement.  Seems to be a legacy thing judging
  from the ubuntu .session files.  Notifications are being provided by
  notify-osd on modern systems, and you can't check for its presence in
  gnome-session (gnome-classic.session always fails because it looks for it, so
  ubuntu falls back to gnome-fallback.session).

Remove unused variable

respect the active/mouse options for monitor placement, and use ObMonitorPlace for per-app too

Allow per-app settings to pick a monitor for a window without choosing a position in it.

Add "active" and "primary" options to the <monitor> placement option for per-app settings (bug #5180)

make undecorated windows place according to their undecorated state (bug #5179)

splits client_setup_decor_and_functions() into 3 functions.

1. add client_setup_default_decor_and_functions()
- called from client_get_all() to get the maximum decor/functions that will
  be available for use by the client.
2. add client_setup_decor_undecorated()
- sets up the client's undecorarted decor if the flag is set by the per-app
  settings or session state.
- we do this before setting up the frame so the frame reflects the window as it
  should be when getting placed.
3. client_setup_decor_and_functions()
- calls the above 2 to perform the same functions as before.
- added to client_apply_startup_state() so that we can ensure it was run fully
  at least once in the mapping process, since it is not called in
  client_get_all() anymore.

it would seem this cannot return NULL now

It will get the primary monitor if there is not monitor under the pointer. But
assert so it's clear something went wrong if this does happen. Note that there
was previously no check for the return value even though the comment claimed
there should be.

Link libobt into libobrender, as it uses things from libobt (bug #5150)

libobrender uses ObtPaths stuff, so it needs to link in libobt.

Propagate _NET_WM_WINDOW_OPACITY to the frame window (bug #5132)

Strip leading/trailing whitespace when parsing colors and other strings (bug #4937)

free undecorated border colors

Specific border width/color for undecorated active/inactive windows (bug #4889)

The attached patch adds three theme options with backward compatible defaults:

undecorated.border.width
  defaults to border.width

window.undecorated.active.border.color
  defaults to window.active.border.color

window.undecorated.inactive.border.color
  defaults to window.inactive.border.color

Fix crash when keyboard map changes and no keybindings exist.

Don't assume old tree is non-null and start rebinding it.

Change default doubleclick timeout to 500ms and keep track of where last click was

Bug #5152 - "mouse double-click time is too low by default - 200ms"

We only use the doubleclick in one place in the default configuration,
for doubleclicking titlebars to maximize windows, so any negative impact
of increasing the timeout should be minimal, especially with the
addition of requiring the two clicks to be in the same place.

Doubleclicks are hardcoded to occur within 8 pixels for now, it doesn't
seem worth it to add a config until someone complains. A possibility is
using the drag threshold, but some people have that set very low so it
could be hard to doubleclick then.

Parse all the actions in a single menu item

Don't remove openbox-gnome.session on make clean - it's not generated.

Make openbox-gnome-session compatible with gnome3 with gnome-session 3.0+ support.

Yay they did something nice and made it easier for other window managers to be used.  How unexpected and pleasant.

Don't show prompts inside the message handler to prevent recursion

Showing prompts causes messages to be created which causes the glib message
handler to abort().  Save the messages and show them when done all other
processing for the current event.

bug #5253 kde4 is out now, use kde4 menus by default.

Startup scripts should export "XDG_MENU_PREFIX=kde-4-" but they export
"XDG_MENU_PREFIX=kde-".

Make "no" a valid choice for the dialog option in cyclewindows.c.

It is the same as "none" for backward compatibility (it used to be a boolean).

Update copyright notice in --version output

Fix typo in configure.ac, noticed by hwoarang

Consistency with helper/modal windows and multiple desktops.

When focusing a window, bring its helper windows.  This happened before when
switching its desktop, but if you have 2 windows on different desktops and
switch between them, the helpers should come to both when they are being used.

When a new helper/modal window maps on a different desktop and you are using
the application, then the helper/modal window is moved to the current desktop
to be with the application you are using.

Fix focus stealing for changing desktops/Use timestamp for user input events

Change the logic for when to allow stealing focus across desktops.
- It was possible to call event_time_after() with a CurrentTime in the old code.
- It would disallow a user requested change which is crazy.
- It would change desktops on you when a new window appeared but this is
  generally not desirable.

event_source_time() is supposed to give the time which the user made things
happen. we leave it at 0 for user-input events right now which means stuff like
changing desktop doesn't save any timestamp at all.  we should use the
timestamp from x for user-generated events.