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

Add Polish translation

Add Greek translation

Sort po/LINGUAS alphabetically

Updated Lithuanian translation

Change the message about no icons in menus.

The message used to mention Imlib2, but now there are two libraries
that can be used to support icons. So make the message more generic
and show it only when both libraries are not present.

Add support for loading SVG icons using librsvg.

This adds a configure option --disable-librsvg, but defaults to
using the library if it is present during configure.

When enabled, Openbox will attempt to load svg image files using
the library, similar to how Imlib2 is used for other image
formats.

Since librsvg uses the libXml2 library, their errors end up in
the same global namespace as Openbox config file parsing. To
avoid this, we reset the libXml current error whenever we start
loading a file, and save the last error that occurred when we
are finished, by storing the error in the ObtXmlInst.

Add autopoint to README.GIT list of packages

Fix a typo that prevented setting client size for the height only (???)

Convert two // to /*

Constrain the moveresize popup to the monitor which has the client

Rename bounds to monitor in place_overlap and move a const

Fix least overlap to fall back to the chosen monitor, not (0,0).

We forgot to count the dock in LeastOverlap

Remove some evil tabs

Refactor the per-app parsing code into a helper function and reduce nesting by 2 levels

Make MoveResizeTo work on the dimensions of the frame, not the client

Allow application rules to control window size (Fix bug 4661)

Use the following in your per-app rules:

<size>
  <width>A</width>
  <height>B</height>
</size>

A and B can be integer values to specify a size in pixels. They can also be
percentages or fractions to be relative to the size of the monitor the window
is placed on.

Remove now-unused config_place_center option.

The new LeastOverlap placement doesn't use this option, and it kind of defeats
the purpose of the algorithm.

When an undecorated window is maximized give it a 1px border at the top (Fix bug 5506)

We want to keep some way for a user to re-decorate the window with the default
bindings. However with a large border width, the top border is much bigger than
it needs to be! So make it fixed at 1px large in this case.

Allow non-interactive focus cycling.

Adds an <interactive>bool</interactive> option to the NextWindow and
PreviousWindow actions. When it is false, the action is not interactive and will
immediately switch focus to whatever the next focus target is.

Removing the "interactive" flag from the focus_cycle() method, as it was unused
previously, and the new code does not make use of it either. In order to be
non-interactive it simply starts a focus_cycle then immediately ends it when the
action ends.

The "interactive" flag in focus_cycle() forced a linear cycling order which may
not be what you want, so the new method is preferrable anyhow.

Don't let autostart checking for background-setting tools end up in logs (Fix bug 5186)

Add a $pointer execute variable substitution and make $wip and $pid consistent (Fix bug 5758)

$wip and $pid would output values for the current target window if there was
one, but output nothing at all if there wasn't making the output difficult to
parse. Changed to make these always output a 0 if there is no target.

Added a $pointer variable that is replaced with the x and y coordiates of the
pointer, separated by a space.

Inspired by the work of Denis Kaganovich <mahatma@eu.by>.

Remove unused variable

Fix duplicate const warning

Fix some namings and xml formatting in rc.xml and rc.xsd

When a window is the focus target and it is removed from the focus order, reset the focus cycling (Fix bug 5410)

We were only doing this if a dialog was showing. But if it's the target, then
a dialog is not relevant, we should always recompute the focus cycling. This
avoids a crash when not using a dialog and closing a window during cycling.

Fix negative ResizeRelative values with terminal windows (Fix bug 5228)

Inspired by patches from Brian Mock <mock.brian@gmail.com>.

Get the window's title in the fake-managed case since we apply app rule matching (Fix bug 5277)

Print error message to stderr (Fix bug 5731)

As stdout is redirected to /dev/null by some DMs, we need to print error
messages to stderr.

Address a couple bugs in rc.xsd (Fix bug 5746)

Make .desktop files obey specs where possible (Fix bug 5737)

Allow windows created by execute actions to steal focus if the user isn't interacting with another window (Fix bug 5419).

When the execute action was run, we would say that the user had used the focused
at that time. Then when a new window popped up, we'd think the user was busy in
the current window and prevent the new one from steal focus.

Now the execute action does not update the "user interacted with the focused
window" timestamp anymore. So, if they aren't currently typing in some window
when they trigger an execute action, and the window appears, it will steal
focus.

Allow window matching based on the group leader's name and class (Fix bug 5721)

In Openbox 3.4 we used the group leader's WM_CLASS value when it was available.
This prevents windows in the group from overriding with a specific value,
however which is bad. More rationale can be found in
http://icculus.org/pipermail/openbox/2010-September/006790.html

Some applications (eg. Firefox) use command line flags to set the WM_CLASS
property on the group leader but do not change the property on the mapped
windows themselves. This makes matching these windows not possible in Openbox
3.5.

We resolve this by exposing the group's WM_CLASS values alongside the individual
window's values. We add _OB_APP_GROUP_NAME and _OB_APP_GROUP_CLASS properties
along with "groupname" and "classname" attributes for the rc.xml application
tag.

Allow loading of menu files outside of your XDG_CONFIG_HOME (Fix bug 5711)

When the given file name can not be found in your XDG_CONFIG_HOME, ie in
 ~/.config/openbox, then try the file name directly.

This means if you specify a menu file such as "/home/dana/helloworld.xml",
openbox will try, in order:
1) ~/.config/openbox/home/dana/helloworld.xml
2) /home/dana/helloworld.xml

And it will load the file you meant when it tries the second one.

Make LeastOverlap placment replace the behaviour of Smart and address style nits

LeastOverlap placement option (Fix bug 5385)

Adds a new placement algorithm that finds a place on the monitor that overlaps
the least amount of windows as possible.

Original patch by Ian Zimmerman <itz@buug.org>.
Port to Openbox 3.5 by David Vogt <dv@adfinis.c>.

Use the nearest monitor when the search query rect does not intersect any monitor (Fix bug 5500)

Previously we would try to find the primary monitor and use that when the search
was outside any monitor. However, if the primary monitor is chosen by the mouse
position and the mouse is not inside any monitor, we enter infinite recursion
trying to find the primary monitor.

The nearest monitor is a better metric anyhow, and this ensures
screen_find_monitor() is never recursive as it always returns a value without
depending on other screen.c methods.

Don't edge-warp with the cursor based on a monitor that does not contain the pointer (Fix bug 4992)

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
how much of the window is in the monitor, excluding parts that were claimed
by another monitor already.

Theme button-specific values should fallback to the button general values consistently (Fix bug 5443)

Prompt should not always use the currently focused button as its result (Fix bug 5518)

Update manpage to current version. (Fix bug 5444)

Don't strip leading whitespace from menu labels (Fix bug 4782)

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.

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

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

Make ResizeRelative resize in terms of pixels for windows with resize increments (Fix 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.

Properly build path names for obt_paths_try_exec

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.

Just put copyright starting year in README so we don't need to keep updating it. Make --version match README.

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

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. (Fix 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.

Don't make desktop switching interactive when no mods used. (Fix 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).

Fix crash on unexpected NET_WM_MOVERESIZE_CANCEL messages

gtk+ 3.4 apparently sends these randomly when you select text in a
GtkEntry. This also fixes bug #5460.

Update german translation

Fix bootstrapping, forgot to change WANT_AUTOMAKE to 1.11 in previous commit

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).