Update my email address

Fix some compiler warnings for ShowMenu

Some fixes found by coverity for things used before a null check

In the second case, we actually check the wrong variable.

Initialize some stuff to 0 before taking their address

Move assignment to before break, hopefully this doesn't break stuff

Fix an if statement getting moved into a loop it shouldn't have

Found by coverity code analysis.

Use PANGO_UNDERLINE_SINGLE for rendering shortcuts

The pango docs say to use _LOW for accelerators, but that causes the line
to be drawn beneath the ink extents which for us may be outside the area
that gets displayed and we end up with no line at all. We also don't want
to extend every menu entry with enough space to fit this possible line.

Fix compile failure when X_HAVE_UTF8_STRING is not defined

Use GString for appending strings

It can take a very long time to print a _NET_WM_ICON property for firefox with the previous code, several minutes. Fixes bug #6042 and also implements the change for append_string.

Fix use after free in menu_entry_frame_free

Fix copy paste typo in swap_byte_order

Update CHANGELOG with changes so far

Make desktop actions noninteractive

Remove freshmeat related stuff from release script, the site is dead

Clip client rect to monitor bounds while picking a corner for resizing

This way you can actually do everything to it when it's offscreen,
not just resize the one edge that's poking out

Change some while loops to for loops in config.c

Allow specifying several mousebinds in one go, like for keybinds

Also fixes minor indent error

Make RaiseLower work for the dock as well

As a side effect, _NET_RESTACK_REQUEST now also allows specifying the dock as the sibling.

Add keyboard/rebindOnMappingNotify option

I have to switch to a qwerty layout to play steam games, and every time
I switch back and forth, we lose some random bindings so I also had to
reconfigure Openbox every time, now I don't.

Pass the GravityPoint as const* instead of by value

Let the menu placement code know if the position was user specified

This way, it can avoid running the code that moves it to a good position
relative to a mouse that isn't there

Move common gravity application to screen.c

Resolve gravity coords in menuframe so it can use the widths

Add <position> tag to ShowMenu action

Fix negative relative coordinate placement

double space

Fix thinko in if.c, *o in Stop can never be pointing at the right place.

Use a file static boolean instead.

Fix typo in variable name

Avoid initializing a VLA of 0 length

Declare variables at start of blocks in place.c

Break loop if is_true was set to false in If

Don't segfault when there's no target in If

Some fixes in french translation

Cache results of offset/shift functions so we don't do 6 function calls per pixel when reducing color depth

Fix gcc warnings in screen.c

Added missing break statement (Bug 6043)

Add a strict option to the ToggleShowDesktop action

When the strict option is used, normal windows are not able to show themselves
while showing the desktop.

Portuguese update for openbox

Allow FillToEdge to grow when all its edges are blocked.

This change points out how useless the return value from client_find_resize_directional()
was, so it also removes that.

When all edges are blocked, the FillToEdge action will try to grow all four edges agin
without blocking any of them on their current edge. This more closely matches the
behaviour of the GrowToEdge action.

Clean up the FillToEdge action implementation

This extend the client_find_resize_directional() method to support
two growing modes, and return a bool for whether it was able to grow/shrink.

The client_find_resize_directional() method now takes an enum instead
of a bool, with two growing modes. The old mode which always tries to
grow, the a new mode that will only grow if the client's edge is not
already at a grow stopping point (ie against the edge of another window).

Created a 'GrowToFill' action. (Bug 3356)

Use the BSEARCH() macro in overlap placement

Currently the code rolls its own binary search, but now that we have
a well-tested binary search implementation in obt/ we can make use
of that.

Fix bugs and add unit tests for BSEARCH()

Also adds running unit tests to the release/go script so we can be
sure that all tests pass before we make a release.

Fix nits for overlap placement center option

Add the old <center> option for the placement policy. (Bug 5946)

Original commit messages:
. Reformat to move closer to house style
. Add center on top of leat overlap place algo
. Add sentinel value to edge arrays
. Use a Size instead of a Rect for a centering field
. Fix off by one bug
. Need to declare dx and dy
. Pass length of edge array instead of recomputing
. Fix missing open-brace in config.c
. Address the more trivial subset of danakj comments
. Revert "Remove now-unused config_place_center option."
  This reverts commit 5e282dae08be3b900e0337efa0fae8f3ffa92cd7.
. Remove reliance on sentinel value when scanning edge arrays
. Avoid need to initialize Size structure by removing it :)
. Clean up field expansion code somewhat
. Compress code further by using a structure for common args
. Fix search for next grid point
. Squeeze it even more by not using Size at all

Fix gcc warnings

Fix a typo causing submenuShowDelay to be ignored (Bug 6019)

Add window type to If as well

This one required some extra helpers so is separate commit

Allow matching class, name and role in If action

Add ForEach action which is like If but runs on all clients

Also adds a Stop action that lets you stop running, in case you only
want to run actions on the first match.

Remove the else-on-newline rule from HACKING

Add queries to If actions

This allows the If action to run queries against a client other than
the target of the actions being run, for example to check state on
the focused window while performing actions on another window during
focus cycling.

The syntax looks like

<action name="If">
  <query target="default">
    <title>FooBar</title>
    <maximized>yes</maximized>
  </query>
  <query target="focus">
    <desktop>3</desktop>
  </query>
  <then>
   <action name="NextDesktop"/>
  </then>
</action>

The above checks the client window that the actions will run on to
verify that its title is "FooBar" and that it is maximized. If that
is true, it also checks that the currently focused client window is
on desktop 3. If that is true also, then it runs the NextDesktop
action.

The target="" option can be set to "default" which uses the client
window that the actions will run on, or it can be "focus" which uses
the client window that is currently focused.

The <query> tag is optional, and the conditions inside the query can
be placed directly inside the If <action> tag, as they were before
this change. In that case, a default <query> tag is assumed with
target="default" which matches the previous behaviour.

Multiple <query> tags can be present, and they must all be true in
order to run the actions in <then>. If any one is false, the actions
in <else> will be run instead.

Update HACKING for brace rules

The rules in HACKING for braces are kind of unclear, so add some
more rules and provide examples too.

Add a note to add the version to the bug tracker for new releases.

Also important to update IRC and keep hydrated.

Update po files for 3.5.2

Add CHANGELOG for 3.5.2

Update version in configure.ac to 3.5.2

No library interfaces were changed since 3.5.1.

Maximized undecored windows have incorrect top border. (Bug 5996)

If the window has no border since the theme has bwidth=0, or because the
window is truly undecorated via MWM hints, then we should not allocate space
for the frame of the window which will just be empty pixels.

window.inactive.label.text.color should default to white.

Currently it defaults to itself, which is NULL, and causes a crash.

The spec http://openbox.org/wiki/Help:Themes#window.inactive.label.text.color
claims that the default is white.

In 98b02c6b60bbde2a5db026b3ee3e6e6dc44d1a92 we had this property being read
twice, once with the NULL backup and once with white. It seems that we kept
the wrong one, so just switching this to white now.

Fix a write out of bounds in splitvertical gradients (Bug 3612)

If the gradient has height 1, then y1sz is 0. We don't want to use the
first color and move the data pointer, since this will move it past the
end of the array.

We use congfigure.ac not configure.am

Fix some compiler warnings

Void function should actually be declared void. Not-reached switch
condition should still return something.

Update line numbers in po files

Change "plain" to "exact" for If's <title>

Update CHANGELOG

Add matching which monitor the client is on in the If action (Bug 5426)

Use enum value for monitor in MoveToCenter

Let the If action match window titles with GRegex

Use a helper function to make if.c a bit shorter

Add activedesktop to If

Dock used showDelay for both hiding and showing (Bug 5811)

Allow specifying only one of width and height in per-app settings size

Fixes bug 5819.

Fix use after free in error message

Add some comments in MoveResizeTo code

Remove fuzzy and outdated translations, add plural expressions.

Clean up the .po files for this release. Plural expressions are needed in
all files that have plurals. And replace fuzzy (incorrect) translations
with empty ones instead.

Make nplurals=2 in Turkish translation since there are plurals translations

The plural should be a c expression that says if n is plural or not.

Remove po_PL from LINGUAS as that doesn't exist, should have be pl.po

Fix format specifications in Interlingua translation

3.5.1 changelog

New Polish translation was an update to existing one.

Move the po_PL.po file to pl.po as it should have been named.

Remove Imlib2 mention from po files

Remove deprecated and unneeded g_type_init call.

Remove unneeded obrender/button.h

Update version to 3.5.1.

obrender broke backward compatibility by changing the layout of the
theme structure.

obt broke backward compatibility by changing the method signature of
some keyboard methods.

Both libraries added some new methods as well.

Move shortcuts to latin characters in Greek translation

The _ keyboard shortcut in menus only works on latin characters.

Move shortcuts to latin characters in Hebrew translation

The _ keyboard shortcut in menus only works on latin characters.

Updated Italian translation

Updated Serbian translation

Updated Spanish translation

Updated Czech translation

Update Hungarian translation

Add Afrikaans translation

Add Romanian translation

Updated Turkish translation

Updated German translation

Updated Arabic translation

Add Belarusian translation

Add Interlingua translation

Add Hebrew translation

Updated Dutch translation