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)

add a more involved example to the SYNOPSIS

bump version to 0.12

remove relative directories from $ENV{PATH} (for taint mode)

Otherwise, the module will die when you use it with PATH=$PATH:. (as is
the case on the OpenBSD cpan testers).

bump version to 0.11

taint mode fix for FreeBSD

bump version to 0.10

update copyright and URL/email

use i3 --get-socketpath by default for determining the socket path

This was introduced in i3 v4.1 (released 2011-11-11, so should be
widespread enough by now).

Revert "implement the GET_LOG_MARKERS request" (no longer available)

This reverts commit 7e2ed06447af5e3b49af69dd24e2a5dec373ad9c.

Bugfix: Also delete callbacks which are triggered due to an error

implement the GET_LOG_MARKERS request

Requires i3 version 966c654112561b21fca076a8e967033510da9981 or later.

Bump version to 0.09

Add TYPE_GET_MARKS and accompanying sugar method

Fix documentation for get_marks (s/tree/marks)

Bump version to 0.08

Implement support for the TYPE_GET_MARKS request, add ->get_marks sugar method

Bugfix: The synopsis mentioned ->workspaces, but it’s ->get_workspaces

introduce get_tree request (tree branch only)

Add check to Makefile to abort in a Windows environment (neither i3 nor unix sockets available)

Hopefully, this will stop CPAN Testers emails about failing tests on windows.

bump version

use getpwuid() to resolve ~ in socket paths instead of glob()

This fixes a warning about using a tainted variable (broke the
tests with newer perl versions).

use new default ipc-socket path, glob() path, bump version

Bump version for new CPAN upload, fix MANIFEST

bump version to 0.02, update MANIFEST

add testcase for sugar methods

Implement _error callback

Add sugar methods for easier usage