Convert the placement parsing section to the new config parser

Fix the check when adding a new variable to the config parser, and NULL pointers in the ConfigParser.

- was checking if it didn't exist instead of looking for a duplicate

Convert the focus parsing section to the new config parser

Add a callback type to the config parser for complex stuff like key bindings that aren't represented in a single variable.

Make ObConfigParser objects.

The ConfigParser object lets you register paths/variables with a specific
 object.  Then you can load a file with that object.  Different files may
 obviously need different variables.

wip: Add config_parser.c which will provide a nice means to specify config variables and where they are in the config file.

- Has functions to register variables of different types.
- When it reads from the file it stores it in an ObConfigValue, but
  puts the raw value directly into a variable of the right type so there
  is no redirection needed to access config values.

TODO:
- move the config file loading shit into config_parser.c
- have config.c just register the various config vars with the parser.
- some more complex things like Fonts will need more work.
- remove the "#if 0" stuff in config_parser.c
  - this is an example of the direction I'm trying to go in config.c eventually

Add ObConfigValue types for a keyboard and mouse button (with modifiers).

Make the copy process return FALSE if the value cannot be copied successfully.
Make the enum, key, and mouse functions return FALSE if the ObConfigValue
  isn't valid.

Make a structure to hold a keycode+modifers and mousebutton+modifiers, and use them for the config values.

Makes a complete value for these so it's one variable instead of two.

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.

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.

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*

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

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.

Add Inotify to the release script testing

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

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.

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

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.

Fix blocking in inotify reader.

If the inotify reader reads an event and it ends at the same place the read()
  ended, then we don't know that there is anything more available, so don't
  try read() again until after it has been poll()'d.

add GNOME environment to the menus for now

Make OnlyShowIn and NotShowIn work in .desktop file parsing

Properly build path names for TryExec

remove debug prints

Fix memory issues in the apps menu when reconfiguring.

All menu pointers must be given up and recreated every reconfigure. The menu
system destroys all menus on its own.

On reconfigure, make the apps menu's linkbase refresh.

Make the obt_watch functionality work without inotify (via manual refreshes).

Adds obt/watch_manual.c which is a filler for when watch_inotify.c can't be
  used.  Other watch_foo.c may also exist in the future I hope.
Adds obt/watch_interface.h which the inotify (and others) subsystem can use
  to call back notification to the main watch system that events have
  occured.
Keep track of all files being watched within the main watch system, so that if
  a directory disappears, we can report the files inside it being removed.
  This change moved a lot of the recursive functionality out from
  watch_inotify.c into the main watch.c, making it much more simple and should
  help make it much easier to add other watch_foo.c subsystems.

fix updates in the app menu from the linkbase.

it did not remove things correctly if the linkbase removed something while the
  menu was not shown.
it sorted the wrong list.
basically it was buggy. tested now with a duplicate .desktop id in 2 places,
  and the menu shows only one copy. updating it while the menu is not shown
  correctly updates the menu.  updating it while the menu is shown also
  correclty updates the menu.

keep only one version of each .desktop file "id" in the linkbase category lists.

don't follow pointers after freeing them

Show all the .desktop links available in the system in the "apps-menu" menu.

They don't do anything, cuz it's problematic to try create an Execute action
from in the code right now.

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)

Give info for changes in the update callback from ObtLinkBase

Also provide a function to access all ObtLink objects in a given category.

specify the encoding of filesystem paths, and include a comparator function

the comparator compares two ObtLink** objects by their names.  if the names are
equal it uses the path of the source as a tie-breaker.

use the filesystem's locale for opening a .desktop file

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.

Show the list of categories in the apps menus as submenus.

The categories are not based on what you actually have yet but will be soon.

Removed the "main category" notion from the links themselves, they just
publish a list of categories instead.  Moved this notion out to the Apps menu
itself.  This should make it easy to possibly to customize your set of
visible categories if we should like to do that sometime.

Store all links in the linkbase grouped by their main category.

don't free the linkbase on reconfigure

the menu does get freed, so don't free it with the menu.

skip empty strings when splitting paths in the environment

don't free lists in the linkbase hash table prematurely.

the values (GSList objects) are freed by the hash table's free function if a
new value is inserted.  if we change the list and reinsert it, then it would
cause the list we're inserting to get freed (partially).  not good.  so instead
we manually free them when we remove them/destroy the hash table.

WIP: Add an xdg applications menu (it doesn't have anything in it yet, but it does make a linkbase)

Avoid blocks on read() for inotify.

When we come into the function we don't know how much data is available to
read.  So just read a single event and return, so poll can figure out if there
is more for us or not.

hide the menus before shutting them down on reconfigure

Move the GSource attach out to the generic watch code, and avoid blocking reads

GLib seems to call the read function sometimes even tho the PollFD's revents
field is empty.  So don't read from the fd if this happens.

Add an optional callback to ObtLinkBase to hear about updates, and fix crash.

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.

Filter links in a ObtLinkbase by environments.

Store a set of active environments in a linkbase and only include links in the
linkbase that 1) want to be displayed, 2) pass TryExec, 3) match environment
requirements and restrictions.

add comments

add --disable option inotify

so we can test compiling without it present

add a comment

Linkbase adds all the .desktops in the system, and updates them as they change.

Fixes to .desktop parsing.
Add language/locale support for .desktop files.
Fixes to inotify watching, change what information is passed through the notify handler.
Add linkbase.h to the public obt headers.

Add linkbase which will keep track of available .desktop files for application launching etc.

Add/fix inotify support for watching filesystem changes.

obt/watch.h has the interface.
obt/watch.c has the generic watch code.
obt/watch_inotify.c has an inotify-specific backend, which could be replaced with another.

some missing ifdef HAVE_SYS_INOTIFY

add an inotify watcher for directories (doesnt work without inotify yet)

Fix a memleak introduced by 0dd91ba0e2382a8f6177f501f8a0b26e0daf26a8, some cleanups

be a bit more explicit about what is being copied from one texture to another

Separate theme options for osd prompt buttons.

I managed to keep backwards compatibility without really cluttering
the code so here is the patch
http://bugzilla.icculus.org/show_bug.cgi?id=4874 too.

Please keep in mind that this is my first piece of code for openbox
and that I'm not a die hard openbox user (yet), not to tell the patch
was not exhaustively tested. Anyway I think it's pretty much in a good
shape but any criticism will be welcome.

Basically the patch add the following theme options for controlling
buttons in osd prompts:

%%%% colors
%
% for the text inside the button
osd.button.unpressed.text.color
osd.button.pressed.text.color
osd.button.focused.text.color
%
% for the line art around the button
%  (if you don't wan't the box just make box.color = bg.color)
osd.button.pressed.box.color
osd.button.focused.box.color

%%%% textures
%
osd.button.unpressed.bg
osd.button.pressed.bg
osd.button.focused.bg

The buttons can be in three states:

 unpressed: neither clicked nor selected
 focused: selected but not clicked
 pressed: clicked (and of course selected)

I discarded the previous distinction between press and pfocus as in
fact it was only a formal distinction, in that both appearances
mimicked each other in every sense. It think that it was just
inherited from the way titlebar buttons are managed so I decided to
simplify it a bit.

All the options default in a way that preserves backwards compatibility:

osd.button.unpressed.text.color  -> osd.active.label.text.color
osd.button.pressed.text.color -> osd.active.label.text.color
osd.button.focused.text.color -> osd.active.label.text.color

osd.button.pressed.box.color ->  window.active.button.pressed.image.color
osd.button.focused.box.color -> window.active.button.hover.image.color

osd.button.unpressed.bg -> window.active.button.unpressed.bg
osd.button.pressed.bg -> window.active.button.pressed.bg
osd.button.focused.bg -> window.active.button.hover.bg

Notice that a good deal of locs where added to theme.c but in
compensation prompt.c is pretty much simpler now because the
appearances and textures are created while loading the theme.

Merge branch 'm4/master'

Revert these two commits temporarily to make the buttons merge easier

1e427a3358deeadf8abc326e714ee201fddc43d6
2288da0ae3b44c987687e3deeb7864fcb7b23bb9

Update portuguese translation

Update estonian translation

Add Plural-Forms header to fi.po and remove fuzzy mark.

Update polish translation.

Fix some rebase artifacts

add --disable options for x extensions

so we can test compiling without them present