Merge branch 'backport' into work

fix a crash in the SessionLogout action

Merge branch 'backport' into work

Respect $XDG_CONFIG_HOME in openbox-session

Reload motif wm hints when property changes

Google's chrome does this when you toggle window decorations.

Based on patch in #4250 by Daniel Erat.

Add all the action names used in 3.4 so configs don't break

Merge branch 'backport' into work

Conflicts:
openbox/actions/desktop.c
openbox/client.c
openbox/event.c
openbox/extensions.c
openbox/popup.c
openbox/screen.c
parser/parse.c

Update slovak translation.

Remove newline from startupnotify description

It doesn't look good in some places, e.g. WnckTasklist (gnome-panel)

[ Also update translations -- Mikael ]

Don't replace ~ when we shouldn't

Bump glib req to 2.14 to get GRegex, now we only replace ~ with your
homedir in an action if it is (at the start of the string or preceded
by whitespace) and (at the end of the string or followed by (whitespace
or a slash)). (?:^|(?<=[ \t]))~(?=[/ \t$])

Fixes bug #4033.

Another debug message typo

Allow escaping _ in menu labels by putting __

Currently you can't mark anything that comes after the __ with _ to make
that a shortcut.

Don't trust xinerama not to return NULL

We check that xinerama is active already, but someone got a NULL here.

Update ukrainian translation and rename to uk.po

Don't draw borders in fullscreen mode when keepBorder is on

Fix interpretation of struts with multiple screens

According to the WM Specification, the left, top, right, and bottom
fields are to be declared relative to the overall X screen dimensions,
not the monitor dimensions.

The example given in the spec (v1.3 or 1.4draft2) is: "Another example
is a panel on a screen using the Xinerama extension. Assume that the
set up uses two monitors, one running at 1280x1024 and the other to the
right running at 1024x768, with the top edge of the two physical displays
aligned. If the panel wants to fill the entire bottom edge of the smaller
display with a panel 50 pixels tall, it should set a bottom strut of 306,
with bottom_start_x of 1280, and bottom_end_x of 2303. Note that the strut
is relative to the screen edge, and not the edge of the xinerama monitor."

In my case, I have a 1680x1050 monitor to the left of a 1920x1200 monitor
aligned at the top. I then have a gnome-panel along the bottom edge of
the 1680x1050 monitor with a height of 24 pixels.

xprop reports the following partial strut: _NET_WM_STRUT_PARTIAL(CARDINAL)
= 0, 0, 0, 175, 0, 0, 0, 0, 0, 0, 0, 1679 which is correct according to
the spec. Gnome-panel is reserving the 150 pixels along the bottom that
aren't visible on the screen plus the 25 it requests for itself.

However, maximizing a window on this monitor leaves a gap of exactly 150
pixels between the bottom edge of the maximized window and the top edge
of the panel.

Also, when the 1680x1050 monitor is the primary monitor (id=1) then the
_NET_WORKAREA property on the root window is also off by 150px for the
same reason.

This patch fixes the two issues I mentioned for exterior monitor edges.
It doesn't attempt to account for "interior" monitor edges (i.e. a 'left'
strut on monitor A when monitor B is directly to the left of monitor A)
because it's not possible to do so with the current strut specification
(see http://mail.gnome.org/archives/wm-spec-list/2004-March/msg00004.html
for a discussion on this limitation)

This could be avoided by having the partial strut atom contain a xinerama
screen ID that the strut applies to, but unfortunately the discussion
all those years ago never got anywhere.

[ quoted from bug #3792 ]

Fix a typo in debug message

Show desktop switch popup on every monitor

This should be a satisfactory fix for #3694 I hope.

Give the popup to the timer remove so it does the right one

Only the chroot popup uses delayed mapping so this is just a latent bug.

Fix memory corruption when y2sz is 0.

data is incremented one too many times when y2sz is zero, leading to
memory corruption.

[ also changed % 2 to & 1 -- Mikael ]

Add "prev" and "next" as possible targets for moveto/resizeto actions.

One of the Debian users asked if it's possible to send a window to other
monitor when using xinerama, especially useful of you have 2 monitors and want
to toggle a window to the other one. I wrote a patch that implements next and
prev to also make that work for 3 or more workspaces.

Only open the default font once, then ref it.

As suggested in #3622, we don't need to open the default font for every
place that wasn't specified in the theme. Solved a bit differently than
the patch given there.

client_validate should return FALSE only for UnmapNotifies that will cause the window to become unmanaged

Show window role in the debug message for name/class too.

Fix for #3715, app settings applied too late.

This caused problems for placing windows with decor turned off, the
placement code thought they had it on.

Add support for 24bit

Make at least the default configs validate

More things need to be case insensitive and whatnot.

Update and reactivate croatian translation.

Add latvian translation.

Avoid anonymous unions

Some compilers, like sun studio and clang don't support them.

Update italian translation.

Close openbox.log when restarting

Merge branch 'backport' into work

Conflicts:
openbox/config.c

Fix per-app monitor setting

A couple of things were wrong, the parser added 1 to the value despite
expecting the user to give values in the range of 1 to
screen_num_monitors, rc.xml documented the values to start from 0 and
finally the monitor value wasn't copied over at all when matching the
client.

Merge branch 'backport' into work

Conflicts:
openbox/event.c
openbox/frame.c

Add Plural-Forms header to .po files that need it.

Handle InputShape type shaping too.

Many composite apps use this to pass through clicks where they are transparent.

Typos in error message from gdm-control

install-sh should be executable

Apparently on solaris, you can't invoke -x scripts via "sh install-sh".

Merge branch 'backport' into work

Only update the focus cycle indicator when already visible in frame.c to avoid overriding bar=no

Another problem pointed out by clang

Using format specifiers without arguments in printf is still bad.

openbox/debug.c:105:18: warning: format string is not a string literal (potentially insecure)
    fprintf(out, log_domain);
                 ^~~~~~~~~~
openbox/debug.c:107:18: warning: format string is not a string literal (potentially insecure)
    fprintf(out, level);
                 ^~~~~
openbox/debug.c:109:18: warning: format string is not a string literal (potentially insecure)
    fprintf(out, message);
                 ^~~~~~~

Merge branch 'backport' into work

Conflicts:
openbox/event.c

Use ngettext for %d desktop(s).

This poses a small problem. We currently let translators reorder this
string, but ngettext only takes one numeric argument. This means that you
can either get correct pluralization or the order you want, but not both.

I fixed up the languages I understand at a very basic level, but the
rest will need translator assistance.

Fix a problem pointed out by clang

openbox/client.c:269:10: warning: incompatible pointer types passing 'gint *', expected 'guint32 *'
    if (!OBT_PROP_GET32(self->window, NET_WM_USER_TIME, CARDINAL, &user_time))
         ^~~~~~~~~~~~~~

Don't move windows to current desktop on net_active_window if the event didn't come from the user.

Instead, just activate the flashy thinger.

Using format specifiers without arguments in printf is bad.

Merge branch 'backport' into work

Bring helper windows when following a window.

_NET_WM_USER_TIME=0 means don't focus the new window.

Merge branch 'backport' into work

Conflicts:
openbox/client.c
openbox/prop.c
openbox/prop.h

No \n on ob_debug in this branch.

The code to replace $wid/$pid died on $foo.

It failed to increment a variable after finding a $ but no pid or wid,
which led to a fun infinite loop. Also don't check if c[1] is w if we
already established it was p.

Show name/class when mapping in debug mode.

Revert "make windows stay on screen when they..."

This reverts commit 73adc6480d59fc5dccf9fe19ed5234c9582b328c.
I know I had a good reason for reverting this commit, I don't know what
that reason was though.

Correct a 64-bit bug in event_time_after

The code assumed the timestamps had the same domain as the type Xlib
uses for them, which is almost never the case with Xlib. Change all
involved variables to guint32.

give the compiler hints about what can be kept in registers so that it can optimize the gradient code better with all the pointers going on

Make sure the _MOTIF_WM_INFO atom exists.

Without this, urxvt won't use motif hints for borderless mode.

Update Basque translation.

0 is the first monitor here, not 1.

Add danish translation.

Properly escape strings before writing XML.

Openbox doesn't support _NET_WM_USER_TIME_WINDOW atm.

Update catalan translation.

Add a translators note about not translating the word "SessionLogout" in an error message.

Add turkish translation.

Update italian translation.

Update spanish translation.

Add Lithuanian translation.

Don't translate action names.

Update russian translation.

Merge branch 'backport' into work

Conflicts:

openbox/config.c
openbox/keyboard.c
openbox/moveresize.c

Fix typo introduced a few commits ago that made resizing windows hard.

Don't need to explicitly check for xft.

Don't use GrabModeSync, it hangs the xserver on chains sometimes.

More or less a revert of 5bfa48dcffec3b00d119d851cb2427a43f80aa7b
"Grab key bindings with GrabModeSync"
See also bug #3582.

Show the resize popup right away.

Previously, it would wait for a resize step before showing it, when resizing
windows with resize increments that were bigger than the moveresize threshold.

When icons had a very small aspect ratio, we crashed, don't.

Move "if"s out of the loop and rewrite the loop to use parse_find_node() to search for "file".

Center the moveresize popup even when shaded.

make windows stay on screen when they are resized due to their normal hints changing

remove some verbose debug prints

Update dutch translation.

Update finnish translation.

Update hungarian translation.

Update czech translation.

Update portuguese translation.

Update german translation.

Update weejun translation.

Make alt-tab icon size configurable.

Update simplified chinese translation.

Update dutch translation.

Update vietnamese translation.

fix a bug from when $window changed to $wid in the execute action

Merge branch 'backport' into work

Conflicts:

obt/keyboard.c
openbox/actions/cyclewindows.c
openbox/actions/directionalwindows.c
openbox/frame.c
openbox/openbox.c

Fix a typo in --help output (master only).

Remove double newlines.

Don't run actions in focus cycling actions backwards either (as in 664b0b4e943ba4d410168817dcff953804b44253).

Don't draw titlebar buttons on top of eachother.

Merge branch 'backport' into work

Conflicts:

openbox/client.c

don't set variables twice