fix remaining warnings

travis: enable more warnings, add -Werror

This will ensure that our code stays free of warnings, at least for the
warnings that we have enabled. (Using _FORTIFY_SOURCE > 0 leads to more
warnings, some of them are harder to address.)

Merge pull request #1575 from hwangcc23/next

Fix warnings

Add a safe wrapper for write and fix some warnings

1. Add a function writeall and make swrite wrap that function. Use either writeall or swrite, depending on whether we want to exit on errors or not.
2. Fix warnings when compiling with a higher optimisation level.
(CFLAGS ?= -pipe -O3 -march=native -mtune=native -freorder-blocks-and-partition)

Signed-off-by: hwangcc <hwangcc@csie.nctu.edu.tw>

Merge pull request #1596 from Deiz/wizard-add-up-down

Switch the wizard's modifier choice when up or down are pressed

Switch the wizard's modifier choice when up or down are pressed

Merge pull request #1590 from acrisci/bug/i3bar-no-markup-default

i3bar: don't parse markup on statusline by default

i3bar: don't parse markup on statusline by default

fixes #1565

Merge pull request #1591 from Airblader/feature-child-cleanup

Return in child parsing as soon as the match was made.

Merge pull request #1582 from shdown/allow-escaping-backslashes

Allow escaping backslashes

Merge pull request #1580 from Airblader/feature-nop

Allow nop command without argument

Allow nop command without argument

fixes #1318

Return in child parsing as soon as the match was made.

As soon as we found the current key for which the value should be handled we can return
from the function to save some string comparisons.

Merge pull request #1454 from acrisci/feature/i3bar-update-strut-partial-reload

reconfigure dock clients on strut partial change

Add tests for backslash unescaping

update default fonts for i3-{input,config-wizard,nagbar}

We’ve done this a while ago in i3.config itself, but the tools in
question aren’t hooked up to that setting, so we need to do it here as
well.

make i3-input work properly on hi-dpi displays

make i3-config-wizard work properly on hi-dpi displays

make i3-nagbar work properly on hi-dpi displays

i3-sensible-terminal: launch i3-nagbar when no terminal is found

fixes #1587

debian: promote suggests to recommends to install a terminal emulator

fixes #1587

Handle strut partial changes for dock clients

Handle changes to the property _NET_WM_STRUT_PARTIAL by reallocating
screen space in the dock area when the dock client requests a new
position.

This will allow changes to the bar config directive `position` to be
effective on the `reload` command.

fixes #1294

i3bar: reconfigure strut partial on reload

Strut partial tells i3 where to reserve space for i3bar. It is affected
by the `position` bar config directive.

Merge pull request #1578 from Airblader/feature-custom-separator

Add support for a custom separator symbol

Don’t focus placeholder windows.

Add support for a custom separator symbol

This introduces a "separator_symbol" property for the i3bar configuration.
If set, the specified string will be used as a separator instead of a vertical line. Since it is an optional configuration, complete backwards compatibility is given.

fixes #1472

Bugfix: skip restart_mode swallow specifications

Otherwise, placeholder windows would accumulate these over restarts. The
worst part is that they don’t get dumped properly, meaning an empty
swallows specification ends up in the state, which matches any window,
so other random windows would be matched into the placeholder (e.g.
i3bar).

fixes #1502

Restore placeholder windows after restarting (Thanks Airblader).

fixes #1502

Merge pull request #1583 from shdown/next

i3bar: spelling fixes (2)

Allow escaping backslashes

Fixes #1577.

i3bar: spelling fixes (2)

Merge pull request #1574 from acrisci/bug/bar-text-null-check

i3bar: check block text for NULL

i3bar: check `short_text` for NULL on dump

Check `short_text` for NULL on statusline dump, since this value can be
NULL when not given. Fixes a segfault in that case.

i3bar: check block text for NULL

Add a null check to `full_text` and `short_text` for a block before
setting the markup property to avoid a segfault.

Revert "Workspace command number selection"

This reverts commit cbd51ce6642be4e1163a64dce8ec20a454f1bf05.

If you’ve started using this feature, I’m very sorry, but I think it’s
better to revert it entirely, as it will only cause more problems in the
future as more and more people upgrade and get bitten by this if we
didn’t revert it…

fixes #1536

Merge pull request #1564 from acrisci/feature/i3bar-markup-per-block

i3bar: set markup per block

i3bar: set markup per block

Add `markup` to the i3bar protocol as a block member.

This is a string that determines how the block should be parsed as
markup. "pango" indicates the block should be parsed as Pango markup.
"none" indicates the block should not be parsed as markup.

Merge pull request #1570 from shdown/next

i3bar: spelling fixes

i3bar: spelling fixes

Merge pull request #1562 from Airblader/feature-short-text

Add support for the short_text property

Merge pull request #1568 from acrisci/feature/valgrind-no-verbose

testcases: dont run valgrind verbose

testcases: dont run valgrind verbose

Add support for short_text

When the statusline is too long, try to use the short_text property of each status block before falling back to truncating it.
fixes #1092

Introduce a function to copy i3strings

Introduce the short_text property on blocks and parse it according to the documentation.

clang-format and rearrange comments in i3bar/src/xcb.c

fix travis config with regards to clang-format

Merge pull request #1560 from shdown/refactoring

i3bar: refactoring

Merge pull request #1557 from acrisci/bug/fix-validation-segfault

Bugfix: validation segfault

i3bar: introduce get_tray_width() function to reduce duplication

i3bar: refactor magic numbers to *_px constants

Bugfix: validation segfault

When `new_window` is given in the config, config validation with `i3 -C`
would segfault.

Add a NULL check in logical_px() to check for the case when the config
is being validated without an X server to prevent this.

Don’t overwrite border width when already set (placeholders).

fixes #1501

i3bar: set correct initial position when reconfiguring

fixes #1542

Merge pull request #1556 from Airblader/feature-i3-save-tree-by-number

Allow referencing the workspace by number.

increase timeouts for travis

See https://travis-ci.org/i3/i3/jobs/55319961 (line 4064) for a job
where the window could not be mapped within 2 seconds.

Merge pull request #1561 from Airblader/feature-append-layout-docs

Added a troubleshooting section to the restoring layout docs

Added a troubleshooting section to the restoring layout docs and described solution for vertically split containers

Merge pull request #1559 from shdown/next

various i3bar bugfixes

i3bar: fix click events for workspace buttons with long statusline

This combines detecting of current and clicked workspaces into one cycle
and then checks if the x coordinate of the clicked point is greater than
the workspace buttons width.

i3bar: fix spelling is messages

i3bar: rename a variable

Give it a more meaningful name.

i3bar: add binding indicator width to workspace buttons width

Just to prevent the indicator from being overlapped by the statusline.

Allow referencing the workspace by number.

This allows "--workspace 3" for numbered workspaces, e.g., if the name is actually
"3:foo". This introduces the same functionality the IPC already offers in many places.

i3bar: cut long statuslines from the left

This draws the statusline after drawing (and calculating width of) the
workspace buttons and fixes calculation of its maximum visible width.
This (hopefully) guarantees that these two will never overlap.

i3bar: fix clearing of the statusline pixmap

This fixes the bug when statusline pixmap wasn't entirely cleared that
caused random artifacts to appear when the statusline width is greater
than the screen width.

Merge pull request #1555 from Airblader/feature-i3-save-tree-current-ws

Make the --workspace optional by defaulting to the focused workspace.

Merge pull request #1549 from shdown/y-offset-fix

Fix incorrect y-offset for text in i3bar (2)

adapted usage text

Make the --workspace optional by defaulting to the focused workspace.

If no option is given, i3-save-tree should default to the currently focused workspace.
Specifying both --workspace and --output will still yield an error.

Merge pull request #1551 from acrisci/bug/rename-workspace-mismatch

bugfix: workspace rename focus mismatch

Merge pull request #1553 from Airblader/feature-remove-output-assignment

Remove the A_TO_OUTPUT option

fix warnings when CONTINUOUS_INTEGRATION env var is unset

relax timings of t/200-urgency-timer.t to reduce flakiness

Remove the A_TO_OUTPUT option as it is not needed anymore and will not be implemented in favor of normal assignments.

bugfix: workspace rename focus mismatch

When renaming a workspace on another output that moves the workspace to
the current output, the renamed workspace would be shown even though it
does not contain the focused container.

Explicitly show the focused workspace after the move. This is necessary
because `workspace_move_to_output` will show the workspace that is
moved.

Fix incorrect y-offset for text in i3bar (2)

014aa7ff7441a8a2bb53bd0956aff61faaf6f37f fixed incorrect align of small
symbols, but also introduced a problem with texts with mixed large and
normal symbols. Fix it by centering the text vertically only if its
height is smaller that the saved font height.
Fixes #1543.

travis: verify code was formatted using clang-format

clang-format src/commands.c

Merge pull request #1539 from mh21/wsrename-1527

Rename workspaces in startup sequences

travis: cat complete-run.log on failures

tests: exit with status 1 when tests fail

tests: don’t print interactive status when running on travis

(or without a tty)

travis: run tests serialized, install latest AnyEvent::I3

travis: install as many perl pkgs as possible via apt instead of cpanm

travis: fix pinning setup (newlines and YAML are complicated)

travis: escape strings for YAML

travis: install more perl dependencies, pull in trusty packages

travis: also cat cpanm build logs after failure

travis: cpanm: set -v so that we get error messages in the travis output

Rename workspaces in startup sequences.

When renaming workspaces, any workspace names in pending startup
sequences also need to be renamed.

Test: startup workspaces updating on rename.

travis: directly use cpanm -n to skip tests

X11::XCB currently fails its tests

travis runs on ubuntu 12.04, so use libinline-perl

travis: mk-build-deps needs equivs

add configuration for travis-ci.com

Merge pull request #1521 from Airblader/feature-use-refactored-function

Use the already existing function to get rid of some code duplication

Merge pull request #1513 from Airblader/defect-1473

Move workspace to assigned output when renaming it

Merge pull request #1512 from mdtipton/unclutter_scroll

Fix scrolling in window decoration with hidden cursor.

Merge pull request #1546 from rosetree/next

Fix comment box