Merge pull request #2301 from EvilPudding/next

Minor optimization

Leaving the last byte in Colorpixel.hex NULL

Added explicit assignment of last byte to the null character, for
appearence's sake.

Hex in struct Colorpixel.hex to contiguous memory

No reason for hex not to be of a constant size, and
no reason to introduce a cache miss by allocating it
separated from the rest of the structure.

Only add NumLock fallback where necessary.

Previously, we always discarded the NumLock bit when looking up key
bindings for key press events, and we always grabbed every keycode with
and without the NumLock modifier.

With this commit, the NumLock bit is no longer discarded: since the
previous commit 3bd5e6e5c81b448f9f0ac84454671a871fbfea66 we can
correctly look up key bindings with/without the NumLock bit, as both
variants are stored in |keycodes_head|.

Further, before adding the NumLock fallback (resulting in grabbing the
keycode with the NumLock modifier), we now check whether the key has the
same meaning when NumLock is enabled. This correctly distinguishes the
KP_End vs. KP_1 case, i.e. one can now use the following key bindings:

    # No longer accidentally triggered when pressing KP_1.
    bindsym KP_End nop KP_End

    # Properly distinguished now:
    bindsym KP_End nop KP_End
    bindsym Mod2+KP_1 nop KP_1

fixes #2346

refactoring: store modifiers alongside translated keycodes

This is a no-op refactoring in terms of functionality.

related to #2346

move xkb_current_group check into own function

This is a no-op refactoring.

remove superfluous check for input_type

This condition is already enforced in grab_all_keys().

Update manpages (#2404)

Add uxterm to terminal list (#2397)

Don't change border style if BS_NORMAL is requested in motif hints (#2386)

Fixes #2385

Handle _MOTIF_WM_HINTS changes (#2384)

Introduced a new GET_BINDING_MODES message type and reply. (#2376)

This type dumps all currently configured binding modes.

fixes #2375

Merge pull request #2350 from madroach/OpenBSD

Fix building and testing on OpenBSD

Don't use pthread on OpenBSD

since OpenBSD pthread does not support pthread_condattr_setpshared().
This patch could also stay in the OpenBSD ports tree or depend on a
configure test macro rather than defined __OpenBSD__.

Always use socket activation

it neither depends on systemd nor on any linuxism.

Fix testsuite on OpenBSD

OpenBSD perl forks for glob().
Therefore attach SIGCHLD handler as late as possible.

Use 64 bit time_t

Explicitly terminate fallback in set_from_resource. (#2366)

Added qterminal to i3-sensible-terminal (#2367)

Fix outdated documentation (#2362)

travis: ubuntu: install libxcb-xrm from our repository

see #2363

travis: remove libxcb-xrm-dev from build-deps before dpkg-buildpackage

travis: ubuntu: install libtool

travis: ubuntu: also install autotools-dev and automake

travis: ubuntu: install autoconf (for autoreconf)

travis: ubuntu: install ca-certificates to make git clone work

travis: ubuntu: remove libxcb-xrm-dev from Build-Deps

It’s installed separately.

This should fix the failing travis build.

Don't trigger binding on window border click unless --border is given. (#2349)

Prior to this commit a binding specifying only --whole-window would
trigger even when clicking on the window border. However, this should
only happen if --border is specified.

fixes #2348

Split list of command / config directives in tests. (#2345)

This makes adding new directives less prone to merge conflicts.

Smart option added to hide_edge_borders config param (#2191) (#2191)

Use case:

* When managing multiple terminals in a workspace, the borders makes it easier
to know where the focus is, but when there is only one it's obvious where the
focus is.

* When there's only a web browser for example, the borders are actually counter-
productive since it makes clicking a side scrollbar or a tab a bit harder (if I
smash my cursor to the side or the top of the workspace, I have to move it in
the other direction by just a few pixels to be able to grab it)

Behaviour:

* No borders when there's a single window in a workspace
* Borders when there are multiple windows in a workspace

fixes #2188

Introduce support for specifying variables from X resources. (#2286)

This patch introduces a new 'set_from_resource' config directive which
allows defining a variable by retrieving its value from the X resource
database. This avoids having to configure a color scheme in multiple
files. The directive takes an additional fallback value which is used
in case the resource cannot be found or during config validation where
no X connection is available.

Furthermore, this patch includes the following changes:
- If the same variable is defined twice, we now properly overwrite the
  value of the assignment rather than inserting two variable definitions
  with the same key.
- We now depend on xcb-util-xrm to query the resource.
- Increase the buffer size for variable / resource assignments.

fixes #2130

Properly close disabled outputs restored during a restart. (#2337)

If an output is disabled during a restart, for example because a binding
such as

    bindsym $mod+Shift+r exec "xrandr --auto", restart

is used, it can happen that we first write the layout to disk and only
then receive the RandR change events. This leads to a situation where
the restored tree will contain these outputs, but the restarted i3
process will not receive the RandR events, thus the internal output in i3
is marked disabled.

This patch finds these cases after a restart and force-disables the
affected outputs.

fixes #2326

Do not count '\' in comment as line continuation (#2181)

fixes #2176

Use the last known timestamp when calling xcb_set_input_focus(). (#2332)

In most cases this won't make a difference, but consider the following
scenario:

* Some application acquires input focus on its own (which the window manager
  is not involved in, so it will "always" work).
* Before the FocusIn / FocusOut events are processed by i3, the
  workflow to push X changes is run, resulting in xcb_set_input_focus()
  being called.

Using XCB_CURRENT_TIME, this means i3 will "win" and override the focus,
even though it simply wasn't aware (yet) that focus had already been taken
over by someone else. When processing the FocusIn / FocusOut events, i3
assumes that the focus is already set which results in a broken focus
state as i3 assumes some container has the input focus when it doesn't.

With the new behavior, i3's attempt to set the input focus will fail if
the FocusIn / FocusOut events have not yet processed. Once they are processed,
the focus state will be automatically corrected.

fixes #2322

Remove redundant depth argument from x_con_init(). (#2323)

Remove dead code guarded with "#if 0 … #endif" (#2338)

This code has been neutralized for many years now and served no purpose
other than cluttering up the code. We obviously don't need it and it's
out of date anyway.

If there's ever any reason to restore (parts of) it, we have git for
that. But we don't need to keep commented out code around.

travis: snapshot git version before creating dist tarball

fixes #2334

Remove title indentation in nested containers. (#2330)

This commit removes the title indentation in certain nested container
situations. The behavior was considered broken as it didn't seem to
make sense and it was unclear why this was introduced.

Given that nobody on i3-discuss@ complained about removing this logic
within two weeks, we are now removing it to make the title rendering
code a little less complex.

fixes #2247

i3-input: Proper position in non-standard cases. (#2313)

This commit fixes two issues:
* We detect the EWMH support window from the root window. If this window
  currently has the input focus, we ignore this. We do this because this
  window is not a window the user is aware of and positioning relative to
  it makes no sense.
* We also detect whether the current input focus is in an i3-frame window.
  This can happen, e.g., when selecting a parent (split) container. Since
  frame windows are direct children of the root window, we must not
  translate its coordinates or we get weird results and i3-input ends up
  off-screen (see #2312). For all other windows, including those without
  any WM_CLASS, we proceed as before.

fixes #2312

Ensure config variables match on longest-length (#2306)

fixes #2235

Focus a newly managed container only if it doesn't use the globally active input model. (#2317)

fixes #1784

Update "mark" to "marks" in i3-save-tree. (#2308)

When allowing multiple marks on a container, we renamed the "mark"
field to "marks". This breaks i3-save-tree which will now filter out
the marks on a window because it doesn't match anymore. This commit
fixes that issue.

Thanks to /u/xenomachina for hinting to this issue.

Remove skipped tests. (#2305)

We simply remove these tests since they are sufficiently covered
by other testcases.

relates to #2289

Merge pull request #2295 from Airblader/bug-2247-2

Fix rendering of marks (and smaller fixed)

Use logical_px() and fix indentation.

relates to #2247

Redraw right-hand side border correctly.

Since we don't actually stop drawing a title (or marks) with the
distance of the border width on the right side, but instead with
a hard-coded two pixel distance, we can only redraw that much of
the border without potentiall cutting off text.

relates to #2247

Only redraw right-hand side border if a non-pango font is used.

relates to #2247

Cast con id as uintptr_t (#2298)

Use `uintptr_t` to cast the con id to int instead of `long int`. This
type is guaranteed to hold the pointer as an int regardless of platform.

fixes #2283

Correctly count the number of windows for no_focus. (#2296)

Previously we counted the number of (direct) children of the workspace to
decide whether no_focus should be applied or not. However, this doesn't
work correctly if there's a single container with multiple windows on the
workspace.

This patch correctly counts all windows on the workspace.

fixes #2292

Only grab the mouse buttons that need to be grabbed. (#2290)

This is a followup to #2049. While we had fixed that bug by only grabbing
buttons 4 and 5 if there is a whole-window binding for that button, this
did not consider buttons higher than 5 as found on many mice.

Therefore, we now ditch the special handling for scrollwheel buttons and
instead do the same for all buttons higher than 3.

fixes #2271

Merge pull request #2287 from layus/fix-punned-pointer

Fix punned-pointer warning from gcc

Avoid hazardous casting.

Remove "dereferencing type-punned pointer" warning

Fix the remaining warning discussed in #1538.
This is obviously a false positive from gcc.

Update i3bar LICENSE to conform to other LICENSE files. (#2284)

This change of the LICENSE has been approved by Axel Wagner in #2281.

Merge pull request #2278 from jolange/version-extraction

Version extraction

travis: clean old docs before building with asciidoc git config

cleanup-bintray: keep the most recent, not delete the most recent

travis: build debian packages and documentation

The resulting packages are pushed to Debian repositories hosted on
bintray.com.

This is the first step to move away from our custom buildbot setup (see
https://i3wm.org/docs/buildbot.html for details on that) towards
infrastructure which is more standard (travis) and in the open.

use git version information if possible, otherwise use information
from (I3_)VERSION files (for tarballs)

dist tarball uses existing (I3_)VERSION file

auto-creating (I3_)VERSION for each release and post-release version files
with 'non-git' suffix

fixes #1993

initial (manual) version files check-in

Remove unmaintained CHANGELOG file from i3bar

Make fullscreen windows open on the output which is indicated by their geometry

With this change, multi-monitor presentations (e.g. as implemented by
LibreOffice Impress) work out of the box. Previously, one had to move
the presentation windows to the right outputs oneself.

Fix spelling mistakes

Point out the default ipc message type in i3-msg.man

debian: remove menu file, add lintian-overrides

*.xsession.desktop: add Keywords=

See also https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=693918 and
https://wiki.gnome.org/Initiatives/GnomeGoals/DesktopFileKeywords

Merge pull request #2243 from Xarthisius/pango_support

Add missing MAX macros, ensure that i3 can be compiled without pango

Merge pull request #2266 from cedricbu/dmenu-death-by-failed-open

i3-dmenu-desktop: do not die on failed open

i3-dmenu-desktop: do not die on failed open

Convert ifdef CAIRO_SUPPORT to if statements to keep consistency with common.mk

Add missing MAX macros, ensure that i3 can be compiled without pango

Merge pull request #2264 from ixjlyons/i3bar-protocol-typos

Fix typos in documentation.

Fix typos in documentation.

Merge pull request #2245 from Osse/next

Update information on where to find #i3 on IRC

Update information on where to find #i3 on IRC

Merge pull request #2244 from jubalh/desktopf

Make i3.applications.desktop file more complete

Make i3.applications.desktop file more complete

Add GenericName to i3.applications.desktop file.

travis: skip login+push for pull requests

release.sh: clone from canonical location

This ensures the repository is up-to-date (in all branches!), hopefully
preventing merge mistakes such as the one fixed with
https://github.com/i3/i3/commit/0e29101ae5f3753626ba447b83a9b8bd490de37e

fix i3 4.12 merge issue in src/commands.c (Thanks Airblader)

release.sh: update i3-announce mailing list address

update release.sh for the 4.12 release

debian: update changelog

Update debian/changelog

Merge branch 'release-4.12'

Merge branch 'next' into master

release i3 4.12

debian/watch: verify signature, use https

debian/control: bump standards-version to 3.9.7

debian/compat: bump debhelper compat level to 9

debian: update copyright to machine-readable copyright

Merge pull request #2239 from wodny/xcb-32bit-properties

window_update_motif_hints(): uint32_t for fields, fixes #2238

window_update_motif_hints(): uint32_t for fields, fixes #2238

xcb_get_property_value() returns 32-bit property fields

Merge pull request #2233 from Airblader/bug-2228

Revert to default binding mode before reloading the config.

Merge pull request #2232 from Airblader/bug-2229

Determine focused workspace correctly when moving workspace to output.

Merge pull request #2231 from Airblader/feature-2223

Introduce I3_FLOATING_WINDOW.

Revert to default binding mode before reloading the config.

If a user reloads the config while in some binding mode, the binding mode
will revert to the default, but no event will ever be fired, causing a
broken i3bar mode display.

This patch explicitly reverts to the default binding mode before reloading
the config. We reload rather than switch to the binding mode after having
reloaded the config because there's no guarantee that mode will even still
exist.

fixes #2228

Determine focused workspace correctly when moving workspace to output.

This patch correctly determines the previously focused workspace on the
target output when moving a workspace to another output. Before, we used
nodes_head for this, which will not actually return the previously focused
workspace, but just the first workspace on that output. Hence, we now use
focus_head instead.

This bug was introduced all the way back in 1e143fea when the feature of
moving workspaces to another output was first implemented.

fixes #2229