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

Remove obsolete paragraph about the need for a second connection

Due to the event-based handling of incoming data from i3 we do not
suffer from this problem.

Use $self->message in subscribe

kill left-over variable

Use only 'import' of 'Exporter'

reformat perldoc paragraph

use constant instead of magic number

Get rid of _bytelength, use encode_utf8 and length instead. Correctly check for scalar

Bugfix: Handle message_type == 0

Add new constants

update MANIFEST correctly (use MANIFEST.SKIP instead of ignore.txt)

Update MANIFEST

Provide constants for message types

Expand testcase for the get_workspaces message (needs i3 to be running)

Return connection status in condvar in $i3->connect

kill boilerplate

prefix internal subs with _

Initial commit

t/171-config-migrate: escape braces for newer Perl versions

Prevent accidentally introduced fall-through

This was added in commit https://github.com/i3/i3/commit/e82e26a24d22e2f92cd90d9547c213493e94ea63

Merge pull request #2847 from tmerr/next

Document missing case of workspace event

Document missing case of workspace event

The docs are missing the "move" workspace event emitted from
workspace.c's workspace_move_to_output function.

Add missing newline to end of file

Merge pull request #2836 from acrisci/bug/sysconfdir-to-makefile

Move SYSCONFDIR definition to makefile

Merge pull request #2838 from acrisci/bug/colors-only-hex

Ensure colors are in hex

libi3: Add basic validation to hex color conversion

Make sure a given hex color is the expected length and begins with a hash in
draw_util_hex_to_color() to avoid memory errors.

fixes #2829

i3bar: change error block color to hex

Named colors are not supported by the i3bar protocol so give the error block
color in hex.

Move SYSCONFDIR definition to makefile

Official autoconf docs contraindicate the use of the $sysconfdir
variable in configure.ac. For reference:

https://www.gnu.org/software/autoconf/manual/autoconf-2.63/html_node/Installation-Directory-Variables.html

The macro SYSCONFDIR (ordinarilly set to "/etc") is required to find
a system configuration file installed with i3. In some build setups,
this may not be defined properly in when the configure script is
compiled.

Instead, define this variable in AM_CPPFLAGS as the documentation
indicates.

fixes #2832

Merge pull request #2831 from dvejmz/next

Add KDE konsole terminal fallback to i3-sensible-terminal

Add KDE konsole terminal fallback to i3-sensible-terminal

Merge pull request #2827 from akash-akya/next

Fix 'Mouse binding don't work'

Fix 'Mouse binding don't work'

Merge pull request #2820 from hwangcc23/fix-2804

i3-msg: Return an exit code upon missing -t arg

i3-msg: Return an exit code upon missing -t arg

If getopt() returns '?' (option with a missing argument), exit the program with an error code rather than continuing the execution.

Fix the issue #2804 (https://github.com/i3/i3/issues/2804).

Merge pull request #2808 from hwangcc23/fix-2802

Fix 'rename workspace to tosomething'

Fix 'rename workspace to tosomething'

This patch fixes the issue #2802 (https://github.com/i3/i3/issues/2802).

1). Revise the state machine for the 'rename workspace' command.
    These scenarios are considered:
    a). 'rename workspace to to bla'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO -> RENAME_WORKSPACE_LIKELY_TO_NEW_NAME
    b). 'rename workspace to tosomething'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO
    c). 'rename workspace to to'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO
    d). 'rename workspace to bla'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO
    e). 'rename workspace bla to foo'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_TO -> RENAME_WORKSPACE_TO_NEW_NAME

2). Add a test case in 117-workspace.t for the scenario b.

Merge pull request #2795 from Adaephon-GH/doc-config-load-order

Fix manpage on configuration lookup order

Merge pull request #2799 from hwangcc23/add_compilation_inst

Add compilation instructions to the hacking howto

Add compilation instructions to the hacking howto

Merge pull request #2796 from Armael/next

Add a link to the ocaml-i3ipc library

Add a link to the ocaml-i3ipc library

Fix manpage on configuration lookup order

The lookup order stated in the i3 manpage was is contratiction to the
actual lookup order, which was introduced with commit
https://github.com/i3/i3/commit/bfa12a581915d6a3de182fa6025fce108cac8eab.
Since that commit (6.5 years ago) the "traditional" paths took precedence
over XDG_CONFIG paths.

Merge pull request #2772 from loungecube/patch-1

prevent multiple menu items per .desktop entry

Merge pull request #2784 from stapelberg/numrelease

compare modifiers for equality, not subset

compare modifiers for equality, not subset (+test)

Subset comparison was introduced with the rather large commit
bf3cd41b5ddf1e757515ab5fbf811be56e5f69cc, but I now think we should use
equality.

In other words, the following key binding:

    bindsym Mod4+x nop Mod4+x

previously would have been triggered when pressing Mod3+Mod4+x.

Strictly speaking, this is a change of behavior, but it breaks none of our
tests, and using equality instead of subset comparison enables more use-cases.

fixes #2002

no-op change: store |button| in the correct data type

no-op change: move bind->release check into loop

We don’t have to negate the check anymore, making it more readable.

no-op change: don’t compare keycode for every modifier

Fix the i3 crash caused by mark + restart commands (#2779)

This patch fixes the issue #2511(https://github.com/i3/i3/issues/2511).

1). Memorize the marks, but only call con_mark once the container has finished parsing. (Credit: This is @Airblader's patch.)

2). Add a test case 267-regress-mark-restart.t for regression test to check if mark and restart command crash i3.