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

docs markup fixes

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.

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

support the window event, tag 0.15

fix doc error: missing opening brace (Thanks bitonic)

0.14: add support for the mode event

update Changes file

bump version to 0.13

Implement the GET_VERSION request (with a fallback to i3 --version)