291-swap.t: make tests fail less (#2964)

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.

Merge pull request #2960 from orestisf1993/next

Fix possible errors related to free()

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.

Merge pull request #2902 from orestisf1993/swap-for-fullscreen

Make swap work with fullscreen windows

Make swap work with fullscreen windows

Swap works like normal with fullscreen windows but swaps the fullscreen
mode of the 2 containers after it's done.

Fixes #2811

Add SWAP util macro

Merge pull request #2887 from orestisf1993/next

Allow assign to workspace by number

Merge pull request #2950 from CyberShadow/pull-20170914-153533

Fix erratic behavior with single container child jumping outputs

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.

Use numeric comparison in sort for $displaynum (#2952)

Eg in the case of sockets:
- X0
- X99
- X100

X99 would be picked instead of X100.

Allow assign to workspace by number

Makes "assign [<criteria>] workspace number <number>" work in the same
manner as "move to workspace number <number>" instead of assigning the
window to a workspace named "number <number>".

config.spec is modified to expect a 'number' string and an extra
argument is used in cfg_assign.

For workspaces that don't exist yet, workspace_get is used as a
fallback. This also allows the user to assign to "<number> <workspace>"
eg "2: work" and the full name will be used if workspace number 2
doesn't exist yet.

Fixes #2590.

Fix typo in con_parent_with_orientation description

Add -name argument for Xephyr (#2937)

This way you can assign the test windows to an empty workspace to avoid
interacting with them (when xvfb-run is not an option):
assign [instance="i3test"] workspace testing

Merge pull request #2945 from stapelberg/i3-config

simplify tests which use launch_with_config precisely once

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

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

fixes #2944

travis: install xrandr in Docker container for t/533-randr15.t (#2949)

Merge pull request #2942 from stapelberg/append

Improve append_layout’s invalid JSON handling

Add .editorconfig (see editorconfig.org) (#2943)

This file uses the same settings which we currently have in our vim modeline,
but can be picked up by many different editors without having to add and
maintain editor-specific modelines in all of our source files.

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

AnyEvent-I3: use Carp for easier debugging (includes stacktraces)

Bugfix: free incomplete containers when JSON parsing fails

related to #2755

Merge pull request #2935 from orestisf1993/issue-2934

Make sure that parse_config input terminates with '\0'

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

Merge pull request #2920 from CyberShadow/monitor-output-names

Consider RandR 1.5's monitors' output names in addition to monitor names

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

Merge pull request #2928 from CyberShadow/pull-20170911-130032

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

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.

Merge pull request #2926 from stapelberg/unflake-tests

Unflake tests

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

Revert "Use OVER operator for drawing text (#2908)" (#2925)

This reverts commit 16160462a30f186f5b72bb551ba2188670d4e45c.

Merge pull request #2923 from stapelberg/tests-split

Testsuite: avoid starting new i3 instances where possible

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

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

Kill windows between tests

i3test: add kill_all_windows convenience function

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

fixes #2914

Add README.md (#2910)

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).

travis: remove clang-analyze (#2917)

• The output currently contains a large number of false-positives and — AFAICT —
  no actual issues.

• Upstream shows little interest in addressing the long-standing issues with the
  TAILQ macros, so the false-positive situation probably won’t change soon:
  https://bugs.llvm.org/show_bug.cgi?id=18222

Currently, we largely spend travis CPU cycles on this, for no additional value.

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.

Use OVER operator for drawing text (#2908)

For opaque text, SOURCE is not any different from OVER. However, when
drawing color glyphs (which consist of RGBA pixels instead of strokes)
SOURCE's handling of alpha is not what we want.

I stumbled across this because cairo 1.15.8 seems to clear the surface
before drawing color emoji if the operator is SOURCE, deleting every-
thing drawn before. Arguably, the area outside the glyph bounds should
not be touched, but even if this is a cairo bug the problem of alpha
within the glyph remains.

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

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

fixes #2905

Merge pull request #2911 from i3/asan

i3bar: ensure get_buffer does not leak memory

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

Merge pull request #2901 from orestisf1993/issue-2900

Set marks to NULL after freeing

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.

Merge pull request #2899 from orestisf1993/patch-1

Fix typo in 265-swap.t

Fix typo in 265-swap.t

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)

Merge pull request #2891 from hwangcc23/init-sigaction

Properly initialize sigaction struct

Merge pull request #2875 from botovq/next

Avoid use of uninitialized in init_dpi_end

debian: update changelog

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)

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().

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