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

fix the check for if a relative is focused on map

when a window pops up a child, don't avoid focusing it because you were working in its parent window before this.  that's probably what made the window appear in the first place

ignore the gdm-control binary

Don't leak auto-scaled RGBA textures.

Update traditional chinese translation.

Update brazilian portuguese translation.

Update japanese translation.

add some debug prints in the session code

make --debug-focus and --debug-session only show their appropriate messages so you can filter easier

Merge branch 'backport' into work

Conflicts:

openbox/session.c

add xfce-settings-show to the default menu

don't save our state for SmSaveGlobal session saves, just Local ones

use SmSaveGlobal to log out.  this will make apps save their documents or whatever so that you don't lose things, but it won't take a snapshot of the current session

Add new strings to the very outdated hr.po file (which is not activated).

Update french translation and fix a typo in actions/session.c

Change ja and pt_BR back to 3.4.6.1 as they now have untranslated strings again.

Update swedish translation.

Was a bit inconsistent with capitalizing titles.

Merge branch 'backport' into work

change the logout button to same capitilization as the title, and update translatable strings

be consistent with logout vs log out use

add the new translatable strings

Update swedish translation.

show the kill dialog if something doesn't respond for 6 seconds instead of 3

Merge branch 'backport' into work

Conflicts:

openbox/actions/execute.c
openbox/event.c
openbox/openbox.c
openbox/openbox.h

add a title to the session logout confirm prompt

allow prompts to have titles specified.  show a prompt when there are syntax errors in the xml config files.

show an error message when parsing syntax errors are encountered in the openbox config files

add a cleanup callback to the prompt interface.  when the prompt's callback returns TRUE, then the cleanup function is called. likewise when the prompt system is shutdown (openbox is exiting), then the cleanup function is also called.  it should unref/destroy the prompt and any memory associated with it

the #ifdef was backwards for using SM or not

add a RECONFIGURING state, and enter that when doing reconfiguring

glib already prints a verbose error message when execute fails.  don't add a second level of useless verbosity to that

use g_debug for debug messages

show prompts when printing error messages

Merge branch 'backport' into work

make code to show a prompt when you just want to display and message and not do anything when the user presses the button

make the SessionLogout action always available but it shows an error if you use it and Openbox wasn't built with SM support

Merge branch 'backport' into work

Conflicts:

openbox/actions/desktop.c
openbox/actions/if.c

Update portuguese translation.

Update japanese translation with the last couple of strings.

don't steal focus from windows you are typing in, even if the new window is a relative

display the NotShowIn stuff correctly