Remove redundant NULL check

copy has been used before this point - so it is too late to be concerned
about a NULL pointer now.

This is OK as sstrdup() calls err() on NULL return from the underlying
strdup() call.

Raised by cppcheck.

i3-nagbar: add option for button that runs commands without a terminal (#3258)

Fixes #2199.

Merge pull request #3441 from stapelberg/template

add specific GitHub issue templates

Merge pull request #3440 from stapelberg/disable

configure.ac: add conditionals for building docs/mans

add specific GitHub issue templates

I learnt about this from the GitHub blog:
https://blog.github.com/2018-05-02-issue-template-improvements/

resolve_tilde: strncpy + strlen is pointless (#3436)

strlen already assumes that the string is NULL-terminated.

Like in https://github.com/i3/i3status/pull/312 but for whatever reason
gcc didn't warn about this here.

userguide: Mention know issues for assign (#3434)

Fixes #3222
Fixes #3293
Related to #2060

configure.ac: add conditionals for building docs/mans

fixes #3378

Merge pull request #3435 from vivien/i3-msg/subscribe

i3-msg/subscribe

i3-msg: add support for SUBSCRIBE message type

If i3-msg is invoked with -t subscribe, it will wait for the first event
matching the given payload, before exiting.

For instance, get the number of the next focused workspace with:

    i3-msg -t subscribe '[ "workspace" ]' | jshon -e current -e num

Like inotifywait, the -m flag allows to wait indefinitely for events,
instead of exiting right after receiving the first one.

For example, continuously monitor the names of focused windows with:

    i3-msg -t subscribe -m '[ "window" ]' | jq .container.name

i3-msg: check reply in quiet mode

i3-msg currently exits right after sending the IPC message if the quiet
flag is set. This means that if an error occurred when issuing a
command, e.g. "i3-msg -q foobar", it gets silently ignored.

What we really want is to just skip printing but still check the reply.

At the same time, explicitly print the reply when we need to, instead of
using an exit label.

Merge pull request #3430 from Synray/aspect_ratio

[RFC] Correctly calculate max_aspect

Correctly calculate max_aspect

Merge pull request #3415 from orestisf1993/g_utf8_make_valid

Check g_utf8_make_valid availability

Merge pull request #3424 from orestisf1993/con_num_windows

con_num_windows: Count floating windows

con_num_windows: Count floating windows

Fixes #3423.

Merge pull request #3421 from Streetwalrus/iconic-state

Reject requests for WM_STATE_ICONIC

Reject requests for WM_STATE_ICONIC

For compatiblity reasons, Wine will request iconic state and cannot
ensure that the WM has agreed on it; immediately revert to normal to
avoid being stuck in a paused state.

Provide g_utf8_make_valid if not available

See #3415 for licensing discussion.

Fixes Airblader/i3#236

Fix typo: terminaison -> termination

Merge pull request #3420 from orestisf1993/sncontext==NULL-3419

i3-nagbar & i3-config-wizard: check sncontext != NULL

i3-nagbar & i3-config-wizard: check sncontext != NULL

From
https://github.com/freedesktop/startup-notification/blob/07237ff25d6171e1b548118442ddba4259a53ba5/libsn/sn-common.c#L87-L171
it appears that SnDisplay can't be NULL, so I skipped the check.

Fixes #3419

Merge pull request #3416 from orestisf1993/530-bug-2229.t

530-bug-229.t: Get rid of smartmatch

530-bug-229.t: Get rid of smartmatch

Merge pull request #3400 from Synray/next

Respect max size from WM_NORMAL_HINTS

Respect max size in WM_NORMAL_HINTS

Merge pull request #3408 from orestisf1993/__focused__-crash

match_matches_window: Check if focused->window exists

Merge pull request #3410 from orestisf1993/min_width

docs/i3bar-protocol: Mention skipping blocks with empty full_text

Merge pull request #3386 from orestisf1993/click-event

Fix "relative_x" and "width" of click events

docs/i3bar-protocol: Mention skipping blocks with empty full_text

Closes #3405.

Fix "relative_x" and "width" of click events

Now clicks begin at the start of the "actual" block, offsets and
separators don't trigger click events. The width property is now just
the width of the block, including border.

Fixes #3380.

match_matches_window: Check if focused->window exists

Fixes #3406.

Merge pull request #3409 from orestisf1993/sb_hoff_px

i3bar: Include sb_hoff_px only when needed

i3bar: Include sb_hoff_px only when needed

Fixes #3404.

Merge pull request #3403 from orestisf1993/issue-3402

_con_move_to_con: Move upwards only on CT_FLOATING_CON

_con_move_to_con: Move upwards only on CT_FLOATING_CON

If target is inside a floating container but not its direct child, the
move can be completed as is.

Fixes #3402.

Merge pull request #3401 from orestisf1993/dump-asy-floating

dump-asy.pl: Include floating_nodes

dump-asy.pl: Include floating_nodes

Merge pull request #3342 from orestisf1993/tree_close_internal

Simplify tree_close_internal

Merge pull request #3399 from orestisf1993/156-fullscreen-focus

Make t/156-fullscreen-focus.t easier to work with

Merge pull request #3398 from orestisf1993/i3-save-tree

i3-save-tree: Exclude unsupported "transient_for" property

Make t/156-fullscreen-focus.t easier to work with

- Add routine that will refocus the expected window on test failure.
Thus, failure on one test will not make others fail.
- Remove some redundant commands, prefer fresh_workspace for screen
changing.
- Kill previous windows between sections if the next section does not
depend on the previous layout.

i3-save-tree: Exclude unsupported "transient_for" property

Even if i3 supported matching windows through "transient_for", it
wouldn't be useful for the purpose of i3-save-tree.

Merge pull request #3395 from lousyd/next

clarify JSON standard non-compliance

Merge pull request #3396 from orestisf1993/commands

commands.c: Improve error replies

commands.c: Improve error replies

- Improve / add various error messages.
- Replace all `LOG(…); ysuccess(false);` with `yerror(…);`.
- switch_mode: Remove redundant "ERROR:" ELOG string.
- cmd_move_con_to_workspace*: Make sure that we don't try to move an
empty workspace to another workspace. This can be problematic when we
match a workspace using command criteria (eg marks) and the target is a
non-existing workspace. We create the new workspace but since nothing is
moved there, we are left with an empty workspace. See added testcase.

clarify JSON standard non-compliance

Modified section on the layout file's non-compliance with the JSON
standard. The section previously stated that having multiple top-level
JSON texts is non-compliant. This isn't the case. It's just that most
JSON parsers will treat that as if it is non-compliant.

Merge pull request #3394 from orestisf1993/validate-utf8

 libi3: validate UTF8 strings

libi3: validate UTF8 strings

Will validate container / window titles.

Fixes #3156.

tree_restore: Check croot

Related to #2414, #3156.

Merge pull request #3391 from orestisf1993/yajl-utf8

tree_append_json: Allow strings that are not valid UTF8

tree_append_json: Allow strings that are not valid UTF8

Fixes #3156.

I couldn't reproduce the problem in a "natural" way so I cheated:
1. Start i3 with gdb
2. Set breakpoing on tree_restore
3. Run, open window, i3-msg restart
5. Open the file in *path with a hex editor
6. Edit the "name" field of the window and insert bytes that are not
valid UTF8
7. Continue

After parsing fails, all nodes including croot are incomplete, meaning
they have to be deleted. We can't recover in any reasonable way so we
have to allow non-UTF8 characters to avoid this situation altogether.

tree_append_json: don't focus freed container

Merge pull request #3389 from orestisf1993/resize-ensure-1px

Ensure containers have a size of at least 1px after resize

Merge pull request #3390 from orestisf1993/resize-tile

resize set for tiling: default to px when not specified

Merge pull request #3388 from orestisf1993/cmd_rename_workspace

cmd_rename_workspace: correct order of events

resize set for tiling: default to px when not specified

Ensure containers have a size of at least 1px after resize

Fixes #2226.
Fixes #2776.
Fixes #3241.
Related to #3194.

cmd_rename_workspace: correct order of events

1. Rename happens
2. Workspace is moved because of assignments
3. Workspace closes because it is empty (#3248)

Fixes #3248.

Improve comment from #3245

configure.ac: fix AC_SEARCH_LIBS([shm_open]) for static linking (#3379)

Without specifying -pthread, the conftest fails and -lrt is missing during
compilation of i3, resulting in a failing build.

Merge pull request #3376 from orestisf1993/floating-disable-scratchpad

Don't disable floating in internal workspaces

Don't disable floating in internal workspaces

Merge pull request #3375 from orestisf1993/ipc_client_timeout

ipc_client_timeout: Fix build when not on linux

ipc_client_timeout: Fix build when not on linux

Also moved the 'end' label because it is unused otherwise.

Reported here: https://github.com/Airblader/i3/issues/233

Merge pull request #3374 from orestisf1993/startup-notification

Support startup-notification in i3-nagbar & i3-config-wizard

Support startup-notification in i3-nagbar & i3-config-wizard

The default i3 config uses the `exec` command without `--no-startup-id`
to launch:
1. i3-nagbar
https://github.com/i3/i3/blob/4cba9fcbdab1487459014dbf8882f5f34e61435e/etc/config#L150
2. i3-config-wizard
https://github.com/i3/i3/blob/4cba9fcbdab1487459014dbf8882f5f34e61435e/etc/config#L194

A user that opens i3 for the first time will be greeted with a "loading"
cursor because of i3-config-wizard.

i3-config-wizard: fix small memleak

Merge pull request #3368 from orestisf1993/render_root

(floating) global fullscreen bugs & others

Merge pull request #3372 from orestisf1993/direction

Resize tile px

Make resize set ppt more accurate

See the testcase for the usecase.

Make resize set px work with tiling containers

541-resize-set-tiling.t: fix "my" variable mask warnings

cmd_resize* statics: remove useless 'way' argument

Make cmd_resize_tiling_width_height work with pixels

Make cmd_resize_tiling_direction work with pixels

Introduces resize_neighboring_cons in resize.c which is also used by
resize_graphical_handler.

Co-authored-by: Andrew Laucius <andrewla@gmail.com>
Authored original code and tests in #3240. I rewrote most of the
resizing code and fixed the failing tests.

precalculate_sizes: don't malloc needlessly

precalculate_sizes: round sizes instead of flooring them

This will lead to more accurate and consistent container sizes.

Needed to fix the failing test of #3240.

Improve resize_graphical_handler code style

- int return type is not useful
- Consistent comment style

Introduce parse_direction

Also fixes the following bug: in the fix for #1011 in
cmd_resize_floating direction "width" is not considered.

Influenced by #3240.

Introduce orientation_from_direction

Merge pull request #3369 from orestisf1993/clang-format

Apply compatible changes from clang-format 6.0.1

Apply compatible changes from clang-format 6.0.1

These are the changes that clang-format 6.0.1 makes to the codebase that
clang-format-3.8 doesn't change back.

Useful for those that use a more recent version of clang-format in their
local machines.

handle_configure_request: check for scratchpad once

handle_configure_request: use 'goto out'

handle_configure_request: reuse 'workspace' variable

con_border_style: check con->fullscreen_mode directly

Introduce con_get_fullscreen_covering_ws

This commit will also fix the following bugs:
1. click.c: Users could drag global fullscreen floating containers.
2. render.c: Floating containers would get rendered with a global fullscreen container in another
workspace.

render_root: fix popup_during_fullscreen logic

The first issue is that there seems to be a typo: fullscreen->window
should have been child->window. The corrected check is redundant since
the while loop checks if the transient_con has a window.

The second issue is that popup_during_fullscreen is never checked even
though the behaviour should be exclusive to the "smart" option.

Merge pull request #3365 from orestisf1993/DRAG_REVERT

floating_drag_window: return on DRAG_REVERT

Merge pull request #3366 from orestisf1993/NUM_HANDLERS

property_notify: use NUM_HANDLERS

Merge pull request #3367 from orestisf1993/dragloop_callback

Call dragloop callback on DRAG_SUCCESS

Call dragloop callback on DRAG_SUCCESS

A race condition is possible. For example, if we first receive a
XCB_MOTION_NOTIFY event and then, while drain_drag_events is still
running, a XCB_BUTTON_RELEASE event the first event is never handled
because we return.

This fixes the flakiness of the tests in #3085.

floating_drag_window: return on DRAG_REVERT

Right now tree_render() is called twice on DRAG_REVERT since
floating_reposition calls it.
Also, on DRAG_REVERT the scratchpad state shouldn't change since the
user canceled the action.

property_notify: use NUM_HANDLERS

Merge pull request #3364 from orestisf1993/warp-randr_query

randr_query_outputs: con_activate -> workspace_show

Use con_focus instead of con_activate for workspace

Typo: output -> content

randr_query_outputs: con_activate -> workspace_show

workspace_show will call x_set_warp_to if needed.

Fixes #3333

Merge pull request #3363 from orestisf1993/regress-scratchpad-focus

Call con_activate after moving scratchpad window to current ws