Fix potential memory leak

xkb_state_new uses calloc and may fail in a rare case, which would cause a memory leak.
Note that xkb_state_unref checks if the parameter given is not null (!state) before freeing.
Calls to xkb_state_new have been grouped to remove code duplication.

Signed-off-by: Elouan Martinet <exa@elou.world>

Merge pull request #3191 from orestisf1993/userguide

Fix userguide link

Fix userguide link

[[move_to_outputs]] doesn't work currently:
https://i3wm.org/docs/userguide.html#move_to_outputs
This does:
https://i3wm.org/docs/userguide.html#_moving_containers_workspaces_to_randr_outputs

This combines a 'BlockId Element' with an 'anchor'. Both should work
now.

Merge pull request #3189 from orestisf1993/run_assignments

run_assignments: check for A_COMMAND early

Merge pull request #3188 from orestisf1993/free_ran_assignments

Free ran_assignments

Merge pull request #3187 from orestisf1993/assign-memleak

Fix memleak: FREE(assign->dest.output)

run_assignments: check for A_COMMAND early

Free ran_assignments

When we run 'reload' all the assignments are freed:
https://github.com/i3/i3/blob/e3e09119bf994ea3f5222441832952a8dd352941/src/config.c#L99-L109

Assignments are saved to each window after they are executed:
https://github.com/i3/i3/blob/e3e09119bf994ea3f5222441832952a8dd352941/src/assignments.c#L41-L46

This means that the pointers stored in window->ran_assignments are
invalid (shouldn't be dangerous currently but could lead to a segfault
if the code is modified) after a 'reload'.

Fix memleak: FREE(assign->dest.output)

Merge pull request #3186 from orestisf1993/i3-msg

i3-msg: only print input + errorposition if they exist

i3-msg: only print input + errorposition if they exist

Before:
$ i3-msg floating disable, move window to position 100 px 100 px
ERROR: Your command: (null)
ERROR:               (null)
ERROR: Cannot change position of a window/container because it is not floating.
[{"success":true},{"success":false,"error":"Cannot change position of a window/container because it is not floating."}]

After:
$ i3-msg floating disable, move window to position 100 px 100 px
ERROR: Cannot change position of a window/container because it is not floating.
[{"success":true},{"success":false,"error":"Cannot change position of a window/container because it is not floating."}]

Merge pull request #3180 from chrisduerr/next

Add alacritty to `i3-sensible-terminal`

Add alacritty to i3-sensible-terminal

Merge pull request #2975 from orestisf1993/move-con_focus

Fix move's focus bugs

Merge pull request #3177 from orestisf1993/trailing

Remove trailing whitespace from Perl scripts

Remove trailing whitespace from Perl scripts

Don't call con_focus in tree_move

Fixes:
- Issue where moving an urgent (unfocused) window resets it's urgency
hint.
- Moving an unfocused container to a new parent should not move it to
the top of the focus stack.

Correct insert_con_into's focus handling

Change from always putting con on the head of the new parent. Important
for moving unfocused containers.

Merge pull request #3176 from orestisf1993/dump-asy-marks

dump-asy.pl: Add marks

Merge pull request #3175 from orestisf1993/dump-asy

dump-asy.pl: Add options

dump-asy.pl: Add marks

dump-asy.pl: Add --save option

dump-asy.pl: Add --gv option

dump-asy.pl: Add POD usage

Merge pull request #3172 from klorax/patch-1

Docs [#3164]: Clarification about X resource value

Docs [#3164]: Clarification about X resource value

Clarification about X resource value: they are loaded verbatim and must therefore be in the format that i3 uses.
Solves #3164.

Merge pull request #3167 from hwangcc23/fix-3163

Add strip_workspace_name

Add strip_workspace_name

See the issue #3163 (https://github.com/i3/i3/issues/3163).

Add strip_workspace_name to strip off the workspace name.

Refocus focused window for FOCUS_IN events on the root window. (#3097)

This deals with (admittedly somewhat misbehaving) clients which
use XSetInputFocus to take focus, but then don't properly restore
focus. This has been observed with TK apps, but also, e.g., Steam.

fixes #2722
fixes #3096

Merge pull request #3166 from Hritik14/next

Fixed typo

Merge pull request #3168 from stapelberg/release-fixes

Release fixes

update release.sh after release

release: new-enough dput no longer needs an explicit file name

release: also build a Debian source-only upload

release: disable git’s rename protection (prevented merging)

debian: update changelog

Merge branch 'release-4.15'

Set non-git version to 4.15-non-git.

release i3 4.15

Fixed typo

Merge pull request #3165 from stapelberg/nagbar

Bugfix: ignore ConfigureNotify with width == 0 || height == 0

Bugfix: ignore ConfigureNotify with width == 0 || height == 0

fixes #3132

Merge pull request #3087 from orestisf1993/focus_order

 Fix focus order issues when encapsulating workspaces

Fix focus order issues when encapsulating workspaces

See new tests for an explanation of the problem.

Introduce *focus_order functions

Merge pull request #3153 from orestisf1993/issue-2990

Don't raise floating windows when workspace is shown

Don't raise floating windows when workspace is shown

From comment:
https://github.com/i3/i3/issues/2990#issuecomment-368345169

To easily reproduce:
1. Open 2 floating windows
2. Focus (with `focus_follows_mouse`) the one behind
3. Move the mouse to the other workspace
4. Move the mouse inside the previous workspace (without it even
touching a window)

Makefile: include contrib/ in dist tarballs (#3152)

unbreaks the debian package build

Merge pull request #3147 from walker0643/next

dump-asy: add prerequisites check; fix empty container names warnings

Merge pull request #3150 from stapelberg/examples

debian: install contrib/ scripts as examples (Thanks anarcat)

Merge pull request #3151 from stapelberg/gzip

debian: explicitly use gzip compression

debian: explicitly use gzip compression

fixes #3146

debian: install contrib/ scripts as examples (Thanks anarcat)

contrib/dump-asy.pl: add prerequisites check and fix warnings about empty container names

Merge pull request #3145 from DebianWall/gvim

Added gVim to i3-sensible-editor

Merge pull request #3144 from DebianWall/guaketilda

Added guake and tilda.

Added gVim to i3-sensible-editor

Added guake and tilda.

Merge pull request #3138 from clawoflight/default-border

[WIP] Add default_{,floating_}border

Merge pull request #3142 from jcrben/patch-1

Remove hardcoded 2013 date from user guide

Remove hardcoded 2013 date

The front page and docs landing page emphasize documentation, but the user guide looks like it hasn't been updated since 2013. It seems from the history of the file that this number is misleading. Let me know if I'm missing anything.

Merge pull request #3135 from op8867555/update_ewmh_atoms

Update EWMH atoms when closing inactive workspace

Add default_{,floating_}border to config

userguide: document default_{,floating_}border

201-config-parser.t: add new tokens to $expected_all_tokens

201-config-parser-t: fix whitespace

closes i3/i3#2702

Update EWMH atoms when closing a workspace container

fix #3126

Merge pull request #3131 from EdwardBetts/patch-1

Correct spelling a mistake.

Correct spelling a mistake.

Merge pull request #3080 from orestisf1993/cmd_move_con_to_workspace

 Reduce repetition in cmd_move_con_to_workspace*

Merge pull request #3102 from jolange/fix3071

tiling resize: remove minimum size (was 5%)

Merge pull request #3104 from jolange/testcase_warnings

Testcase warnings

Merge pull request #2998 from orestisf1993/issue-2990

Don't raise floating windows when focused because of focus_follows_mouse

Don't raise floating windows when focused because of focus_follows_mouse

Fixes #2990.

Implement 'resize set <width> ppt <height> ppt' command for tiling windows (#3036)

Send success response for nop. (#3113)

fixes #3112

Improve resize_find_tiling_participants() and simplify cmd_resize_tiling_width_height() (#3111)

Merge pull request #3110 from DikiCook/config

Add comment $mod+r in back to normal when resize window

config: add comment $mod+r in back to normal when resize window

Merge pull request #3101 from tpraxl/feature/workspace-variables

Add workspace vars to support DRY when customizing

Add testcases for focus_follows_mouse

cmd_move_con_to_workspace_number: rename workspace->ws

For consistency with other cmd_move_con_to_workspace* functions.

Reduce repetition in cmd_move_con_to_workspace*

Merge pull request #3105 from clonejo/feat/pango-markup

Docs: state that pango markup in i3bar requires the use of a pango font.

Docs: state that pango markup requires the use of a pango font.

fixing uninitialized warnings in testcases

fixing redeclaration warnings in testcases

tiling resize: remove minimum size (was 5%)

fixes #3071

Shorten comment for workspace variables

The old comments gave the impression that you had to define workspace names upfront, which is not true. This also keeps the overall config as brief as possible.

Merge pull request #3100 from jolange/fix3086

free last_motion_notify before returning

free last_motion_notify before returning

fixes #3086

Add workspace vars to support DRY when customizing

Relabeling the workspaces required the user to update the labels at least in two places of the i3 config file (switch to workspace + move focused container to workspace) to keep a consistent state.

This got even worse when the user started to reference the workspaces elsewhere, e.g. for assigning containers to specific workspaces.

This change supports the DRY principle for users. Relabeling the workspaces is now merely a matter of changing the respective variable ($ws1, $ws2, … $ws10).

Merge pull request #3095 from pallavagarwal07/next

Add support for relative coordinates in i3bar click events

Add relative coordinates in JSON for i3bar click events (fixes #2767)

Add support for relative coordinates in i3bar click events

Rename {x,y}_rel to relative_{x,y}

Update i3bar-protocol doc to mention the added fields in click events

Merge pull request #2969 from orestisf1993/issue-1819

Make 'focus' disable blocking fullscreen windows

Merge pull request #3079 from orestisf1993/generate-command-parser

 generate-command-parser.pl: remove trailing characters

Make 'focus' disable blocking fullscreen windows

The problem here is that con_fullscreen_permits_focusing() does not
check if there is a blocking fullscreen container in the workspace that
the container to be focused belongs. This makes it possible to focus a
container behind a fullscreen window if it's in an unfocused workspace.

This commit introduces a change in the 'focus' command behaviour. When
focusing a container blocked by a fullscreen container, either CF_OUTPUT
or CF_GLOBAL, the blocking container loses its fullscreen mode and the
target container is focused like normal.

This should not affect directional focus commands: left, right, up,
down, parent, child.

Fixes issue #1819.

userguide: mention <criteria> in focus

Move is_num_fullscreen to Test.pm

Add error reply to cmd_focus_window_mode

Use con_has_parent in con_fullscreen_permits_focusing

generate-command-parser.pl: remove trailing whitespace

generate-command-parser.pl: remove trailing comma