tests: unflake tests by not starting i3bar

tests: remove the (broken) exit_gracefully check

I previously tried to fix the check, but could only come up with a fix which
required removing our module pre-loading, which makes the tests considerably
more expensive. Instead, let’s just remove the check.

tests: use i3_config arg instead of precisely one launch_with_config

This way, tests are shorter, and i3test’s invocation of launch_with_config
parallelizes work better, using dont_block => 1.

i3test::XTEST: don’t “use i3test” to avoid clobbering state

Before this commit, the $i3_autostart variable was accidentally overridden.

Reorder tests to not use the same number (#2947)

Distinct numbers make re-running individual tests easier by helping with
tab-completion.

Completeness verified using:
% for i in $(seq 0 600)
do
  files=$(ls testcases/t/$(printf "%03d" $i)-*.t 2>&- | wc -l)
  [ "$files" != "0" ] && [ "$files" != "1" ] && echo "clash: $i"
done

Kill windows between tests

i3test: add kill_all_windows convenience function

529-net-wm-desktop: avoid timeout, avoid restarts, split

This shaves off almost half a second of the wall-clock time (from 5.9s to 5.6s).

165-for_window: merge config and re-use i3 instance, split remainder

This reduces total test wall-clock time by 1.5s (from 7.5s down to 5.9s).

t/265-swap: don’t start new i3 instances with the same config

$config is never touched after being initially set up.
Not restarting i3 between each test case reduces the runtime of this test by an
order of magnitude.

ipc: document how to detect i3’s byte order in memory-safe languages (#2961)

related to issue #2958

Bugfix: avert crash by fixing focus when creating output containers (#2966)

fixes #2854

Bugfix: don’t invalidate layout upon invalid 'layout toggle' params (#2965)

fixes #2903

docs/ipc: "urgent": complete the list of container types (#2967)

Thanks chressie!

Migrate tooling to ExtUtils::MakeMaker (#2963)

ipc: rename COMMAND to RUN_COMMAND for consistency (#2956)

All other message types are verbs, only our first-ever message COMMAND wasn’t.

While we’re here, also change the message type dictionary into a table with
clickable links to the corresponding reply type.

Authors of downstream IPC libraries are encouraged to keep the old name around
so as to not break existing code, but mark it as deprecated.

Fix use of err after it is freed

Fix wrong call to free

To confirm, assign n to a constant value and try to use the
append_layout command.

Without the change i3 crashes.

Prevent freeing of uninitialized pointer

> variable 'buf' is used uninitialized whenever 'if' condition is true

Note: freeing a NULL pointer is fine.

ipc: tree reply: document focus, nodes and floating_nodes (#2955)

These haven’t ever changed, but were only included in the example, not in the
list, so people might not have realized that these are safe for use.

Fix erratic behavior with single container child jumping outputs

This fixes a regression introduced in commit
4e88c10564ca5366c2578908f62ec56625a26718: when attempting to move the
single child of a container in the direction of another output, i3
would move the window to the output, despite the window not being at
the edge of its output, instead of moving it to its parent container.

The bug occurred because the check for moving containers across
outputs with non-default workspace layouts (issue #1603) did not
actually verify that the moved window lies at the edge of the
workspace, despite what its comment said.

Fixes issue #2466.

Fix typo in con_parent_with_orientation description

tests: unflake t/257-keypress-group1-fallback.t (#2946)

fixes #2944

validate JSON before loading

This commit also introduces slurp() which reads a file in its entirety. Using
this function instead of doing IO in the functions in load_layout.c again and
again makes the code cleaner (fixing at least two memory leaks) and avoids
re-reading the same file 3 times.

related to #2755

Bugfix: free incomplete containers when JSON parsing fails

related to #2755

scalloc parse_config input to make sure it terminates with '\0'

Otherwise strchr() can crash for files that don't end with '\n' because
it won't find a null char to terminate at.

Fixes #2934

Don't insert newline at end of config with launch_with_config

docs/userguide: Document that i3 can accept RandR output names

533-randr15.t: Add test for bar output name canonicalization

533-randr15.t: Stop hard-coding the output name

Refactor away all mentions of DP3.

533-randr15.t: Add a fake output connected to the fake monitor

Add an output ID to the simulated RRGetMonitors reply, then add a
simulated RRGetOutputInfo reply describing the added output.

testcases/lib: Add inject_randr15_outputinfo argument

Allow tests to specify a file name for inject_randr15's
--getoutputinfo_reply command-line parameter.

inject_randr1.5: Intercept X11 error responses in addition to replies

Allow clients to send garbage to the server, then intercept the
server's error response and substitute it with the supplied simulated
reply data.

inject_randr1.5: Add RRGetOutputInfo reply injection

Add a --getoutputinfo_reply switch to indicate a filename containing
the RRGetOutputInfo reply data to inject.

inject_randr1.5: Refactor reading and storing reply buffer to a struct

Allows easier introduction of additional reply buffers in upcoming
changes.

ipc: Canonicalize output names in bar configuration

Convert the output names specified in the "output" and "tray_output"
fields in bar blocks in i3's configuration to the referred output's
primary name. This allows specifying names other than the primary
output's name in the given fields without changing the IPC protocol.

randr: Look up alternative output names when searching outputs

Update get_output_by_name to look at all additional names added by the
change in the previous commit, not just the primary one.

randr: Register monitors' output names as additional i3 output names

In addition to the name of the monitor itself (which is still used as
the i3 output's primary name), register RandR output names associated
with the RandR monitor as alternative i3 output names.

Store output names as a linked list

Currently, only one name is ever added, and only the first name is
ever accessed; actually using the capability to store and access
multiple names comes in the following commits.

Introduce output_primary_name function

Currently simply returns output->name, but this will make it easier to
change how output names are stored in the following commits.

Also replace reading output->name with invocations of
output_primary_name. Code which writes output->name is unchanged. Done
using a mostly mechanical replacement of output->name to
output_primary_name(output).

testsuite: install Module::Install so that AnyEvent-I3/Makefile.PL works (#2940)

As per https://perlmaven.com/cant-locate-inc-module-install-in-inc, the inc/
directory should not be under version control.

fixes #2914

Fix userguide bug (#2932)

Fixes #2931

docs/hacking-howto: Update section topology

- Promote the "How to build?" sub-section to a top-level
  section ("Building i3")

- Convert the "Introduction" sub-section as the intro to the remaining
  contents of the "Using git / sending patches" section

- Keep "Which branch to use?" as a level-3 sub-section, thus making it
  a sub-section of what used to be the "Introduction" sub-section.

docs/hacking-howto: Promote "How to build?" sub-section

Move the "How to build?" sub-section to the top of its parent section.

docs/hacking-howto: Promote "Using git / sending patches" section

Move the contents of the "Using git / sending patches" section to the
top of the document.

tests: run 533-randr15.t at the very end

The test runs `xrandr setmonitor`, which will otherwise affect any test
scheduled after 533-randr15.t, causing flakyness in t/217-NET_CURRENT_DESKTOP.t
for example.

tests: unflake t/263-edge-borders.t

tests: re-seed random number generator in workers

docs/testsuite: Correct Xephyr package name on Arch Linux (#2913)

The package is called `xorg-server-xephyr`, not `xorg-xserver-xephyr`.

testcases/Makefile.PL: tell MakeMaker this is a pure-Perl distribution (#2922)

fixes #2914

Add files generated by make check in AnyEvent-I3/ to .gitignore (#2915)

Include AnyEvent-I3 directory in dist tarballs (#2916)

fixes #2905

travis: downgrade temporarily due to asan issue

fixes #2912

i3bar: ensure get_buffer does not leak memory

This fixes an AddressSanitizer warning which recently popped up.

related to #2907

Set marks to NULL after freeing

realloc() was being called on an already freed pointer.

Fixes #2900

Improve 267-regress-mark-restart.t

Another window with a mark is needed for issue #2900.

Check if con_id exists in cmd_swap (#2898)

Also adds some testcases for swap using con_id.

Fixes #2895

Invert condition to log debug message in correct situation (#2896)

Properly initialize sigaction struct

The code in handle_signal() wasn't clearing the struct sigaction before passing it to sigaction().
This meant that we would block a random set of signals while executing the default handler, or jump to the uninitialized __sa_sigaction__ (instead of sa_handler).
Initialize properly as we do in setup_signal_handler().

Avoid use of uninitialized in init_dpi_end

If conn == NULL or display == NULL, init_dpi() jumps to init_dpi_end
before (declaring and) initializing resource. In init_dpi_end, there
is a free(resource) call conditionally on resource != NULL, so this
may lead to a bogus free. Found by clang -Wsometimes-uninitialized.

Update debian/changelog

Merge branch 'next' into master

Merge branch 'release-4.14'

Set non-git version to 4.14-non-git.

release i3 4.14

Revert "i3-nagbar: add button flag to execute action with /bin/sh directly" (#2893)

Respect focus_on_window_activation for ConfigureRequests (#2889)

fixes #2873

Merge pull request #2888 from stapelberg/leak

Fix memory leak

Update doc for dependency Module::Install (#2877)

After moving to AnyEvent-I3, a new testsuite dependency is introduced: Module::Install.
Update the doc for this.
See the issue #2876.

do leak check before exiting

related to #2541

Fix memleak: free regex when parsing fails

fixes #2541

Merge pull request #2882 from jolange/dev

docs markup fixes

docs markup fixes

Merge pull request #2868 from stapelberg/mouse

t/264-keypress-numlock: add mouse binding test

t/264-keypress-numlock: add mouse binding test

fixes #2523

Respect dont_warp flag when moving containers (#2867)

fixes #2681
fixes #2592

i3bar: only restart child when command changed (#2866)

this is a follow-up to
https://github.com/i3/i3/commit/98f202dd1b2782d11a713513f5dcca2f52daab73

fixes #2689

Focus windows upon ConfigureWindow with stack-mode=Above (#2865)

fixes #2708
fixes #2745

Merge pull request #2864 from stapelberg/fixconfig

Improve error messages for incorrect config lines starting with “set”

t/201-config-parser: update expected token list

parser: only skip set[\s], not set.*

fixes #2564

start nagbar when encountering invalid set statements

related to #2564

Skip lines consisting only of “set”

(Re-)initialize optional fields to empty strings

Bugfix: check bounds before accessing memory

This fixes the following issue when having an error early in the config file:

==1562==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x6220000180ff at pc 0x55c837edb1d3 bp 0x7ffee7534650 sp 0x7ffee7534648
READ of size 1 at 0x6220000180ff thread T0
    #0 0x55c837edb1d2 in start_of_line ../../i3/src/config_parser.c:238
    #1 0x55c837edc96f in parse_config ../../i3/src/config_parser.c:493
    #2 0x55c837edf527 in parse_file ../../i3/src/config_parser.c:1091
    #3 0x55c837ecf14b in parse_configuration ../../i3/src/config.c:65
    #4 0x55c837ed1ef4 in load_configuration ../../i3/src/config.c:230
    #5 0x55c837f0a8d0 in main ../../i3/src/main.c:539
    #6 0x7fb63ae042b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #7 0x55c837e95eb9 in _start (/home/michael/i3/build/i3+0x4beb9)

0x6220000180ff is located 1 bytes to the left of 5165-byte region [0x622000018100,0x62200001952d)
allocated by thread T0 here:
    #0 0x7fb63e590cf8 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.3+0xc1cf8)
    #1 0x55c837f59aa6 in smalloc ../../i3/libi3/safewrappers.c:24
    #2 0x55c837edef45 in parse_file ../../i3/src/config_parser.c:1029
    #3 0x55c837ecf14b in parse_configuration ../../i3/src/config.c:65
    #4 0x55c837ed1ef4 in load_configuration ../../i3/src/config.c:230
    #5 0x55c837f0a8d0 in main ../../i3/src/main.c:539
    #6 0x7fb63ae042b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)

Merge pull request #2863 from stapelberg/fixfreeze

i3bar: Bugfix: avoid freeze after VisibilityNotify

i3bar: Bugfix: avoid freeze after VisibilityNotify

fixes #2790

Bugfix: consider inactive monitors when querying (#2862)

fixes #2815
fixes #2594

Merge pull request #2861 from stapelberg/ipcconfig

Introduce the GET_CONFIG IPC request

Bump AnyEvent-I3 to 0.18

Introduce the GET_CONFIG IPC request

This introduces memory usage by one copy of the config file, which is an
acceptable trade-off for being able to easily revert data loss.
The default config is 6KB, user configs will be in the same ballpark.

fixes #2856

Makefile.am: fix anyevent-i3.stamp dependencies

Merge pull request #2860 from stapelberg/anyevent-i3

Merge the AnyEvent-I3 repository into i3/AnyEvent-I3

Merge pull request #2859 from stapelberg/fixtests

Fix tests

testcases: remove external AnyEvent::I3 dependency

Move to AnyEvent-I3

tag 0.17

use lib '.' for Perl 5.25.11+

see also https://rt.cpan.org/Ticket/Display.html?id=120943

Add the shutdown event (#2)

The shutdown event is triggered when the ipc shuts down because of
either a restart or when i3 exits.

support the barconfig_update and binding event, tag 0.16