code style fixes for the previous commit

• our function names use underscores
• rewrote the function’s comment
• function comments must be in the source _and_ in the header
• no blank lines after function signatures

The command to resize a floating window now checks the minimum and maximum size.

Fix memory leaks in config_parser.

push_token() doesn't take ownership of its str argument.

userguide: add section with synonym "screen" for "RandR output"

While the docs describe the feature, plenty of people ask for it on IRC.
Let’s see if this makes it easier to find.

config parser: recover after invalid input

This is done by ignoring the rest of the current line and jumping to the
nearest <error> token.

fixes #879

i3bar: Add current binding mode indicator

complete-run: run 000-load-deps as early as possible

tests: 000-load-deps: bail out when dependencies are not found, test more of them

This should be a better hint for people who forgot to install the
testsuite dependencies :).

tests: 115-ipc-workspaces: ensure the i3 ipc socket is connected

set LC_NUMERIC=C when dumping nodes in the workspace event

Otherwise the resulting JSON might be invalid because it uses the
locale-specific comma separator, e.g. "16,666" instead of "16.666".

t/115: make the test a little shorter by using helper functions

workspace events test

better docs for the focus workspace ipc event

refactor, name changes

We need to send the workspace event earlier, because otherwise 'old'
might already be destroyed (if it was empty).

take care of non-existant old workspaces

more informative `workspace' events

Add a `current' and `old' properties to the `focus' change type,
containing the current and old workspace respectively.  These additions
are not necessary anywhere else because `focus' is always triggered when
changing ws.

simplify yajl related code

Specifically, put the version dependent code in some macros, and put
that plus the `y' and `ystr' macros in a separate file, `yajl_utils.h'.

Merge branch 'master' into next

Merge branch 'fix-baf-after-scratchpad'

bugfix: fix workspace back_and_forth after displaying a scratchpad window

fixes #868

libi3/font: Use "pango:" prefix to avoid confusion

Also add a user-friendly font description syntax to userguide

complete-run: Unset I3SOCK

introduce new command to rename focused workspace

The corresponding command is 'rename workspace to <name>'. As a side-effect
this fixes the command 'rename workspace 1 to to'.

Signed-off-by: Michael Walle <michael@walle.cc>

docs/testsuite: add "Installing the dependencies" section (Thanks bitonic)

fixes #863

Merge branch 'master' into next

Merge branch 'fix-split-indicator'

Bugfix: force rendering when the parent’s orientation changed

Otherwise, the split indicator might not be refreshed even though it
should be.

fixes #858

docs/userguide: s/alt/Alt/

Implement smart popup_during_fullscreen mode

With this commit, the default behavior is to display popups while there
is a fullscreen application only if the popup belongs to that
application (as determined by the WM_TRANSIENT_FOR hint which
applications have to set properly).

fixes #663

spelling error: s/implementaiton/implementation/g

keycode default config: s/bindcode/bindsym (Thanks Tim)

docs/userguide: use $mod consistently (Thanks Conley)

Skip floating windows in the focus stack when moving through the tree

Includes a test for the new behaviour

Focus windows when middle-clicking (X paste)

As with most click-based focusing, this only has an effect when
focus_follows_mouse is disabled.

Skip floating cons in focus (child|parent) and stop them from being split

Focusing child from a workspace should now skip over the floating con and
go directly to its child. Focusing parent from that grandchild should leave
the workspace focused again.

bugfix: don’t send workspace command when at beginning/end of workspaces (Thanks whitequark)

fixes #843

exit with a proper error message when there are no outputs available (Thanks flo)

fixes #842

config parser: make newlines okay before opening braces (Thanks aksr)

Bugfix: Actually set border width in config_directives.c (Thanks strcat)

Allow 'focus $dir' to move out of non-global fullscreen containers

Allow workspace contents to be moved if there are only floating children

Fix fullscreen focus bug and corresponding test flaw

As the workspace might be reached via recursion (e.g. moving from the edge
of a fullscreen split container), it's necessary to check for a fullscreen
container whenever a workspace is reached.

Grab keys with all permutations of lock and numlock

This should prevent all cases of caps lock (or shift lock, on some
keyboards) from interfering with i3 key bindings.

tests: fix error messages, add 'bind' as synonym to mode blocks

Fix warning: exclude NULL parameters from format string (Thanks knopwob)

accept ctrl as synonym of control (Thanks SardemFF7)

error out instead of accepting invalid key bindings (Thanks SardemFF7)

add missing include/config_parser.h (Thanks slowpoke)

bugfix: config-parser: bind is a synonym for bindcode

fix warning about printf() field length

add test.config_parser to .gitignore

use the new parser by default

you can force the old parser with the command line flag
--force-old-config-parser-v4.4-only (which will be present in v4.4 only,
as the name suggests)

move owindow definition into the command parser

link the parser test binaries with -g

change the commands_parser prefix to 'command' for consistency

t/201-config-parser: test more directives, test error handling

port the entire old config parser to the new one

config_parser: implement <number>s, proper error handling

tests: use new assign syntax, drop legacy test

generate-command-parser: support <number>s, state ID replacing and…

…determining the next state within a function like cfg_criteria_pop() by
passing next_state in the ConfigResult (or CommandResult) and using it
after calling.

add first bits of a (custom) config parser

generate-command-parser: make input/output configurable

i3-msg.man: fix reference to the "reply section" (Thanks slowpoke)

allow floating cons to be reached using 'focus parent'

I suppose this was just an oversight. Let’s see if it causes any issues.

fixes #831

Improve startup sequence termination conditions

If a window with _NET_STARTUP_ID set is moved to another workspace, it
will delete any associated startup sequence immediately. This will also
occur if a window has a leader with _NET_STARTUP_ID set, if the leader
has no container (never been mapped).

A startup sequence may also be deleted if it's matched by
startup_workspace_for_window() and its 30-second timeout has elapsed.

Replace the discrete 'split' Con property with a simple function.

Un-fullscreen as needed when moving fullscreen containers

This avoids a case where a fullscreen container could be moved onto a
workspace that already had its own fullscreen container, leading to
two fullscreen containers on top of each other.

shmlog: Remove O_TRUNC flag for shm_open, we truncate ourselves

release-notes: s/mod+l/mod+e/ (Thanks hax404)

docs/ipc: add a warning to use an existing library (Thanks slowpoke)

docs/ipc: remove unnecessary newline (Thanks Merovius)

Merge branch 'fix-ws-layout'

Merge branch 'master' into next

set workspace_layout in create_workspace_on_output

fixes #835

Focus the relevant workspace when clicking any container.

Maintain relative positioning when moving floating windows between outputs.

testsuite: add 'new-test' helper script

add test for previous commit

Only re-focus the workspace when moving a con if the target ws is hidden.

Implement moving workspaces as if they're regular containers

Merge branch 'master' into next

Merge branch 'fix-floating-move'

Bugfix: Correctly move floating windows to invisible workspaces cross-output (Thanks swh)

Bugfix: with one ws per output, don’t crash on cross-output moves (Thanks moju)

fixes #827

Make the resize command honor criteria (Thanks Tblue)

fixes #816

Merge branch 'fix-indicator-pos'

Merge branch 'master' into next

Bugfix: Draw h-split indicator at the right position

fixes #817

Fix 'border toggle' (it "skipped" 1px border) (Thanks joepd)

fixes #818

Merge branch 'fix-floating-move'

Merge branch 'master' into next

Bugfix: don’t crash when dragged floating window closes (Thanks darkraven)

Previously, while the loop would not be executed anymore, the callback
(for actually repositioning the window after a MotionNotify) would still
get called once. This commit avoids that, thus fixing the crash.

fixes #819

userguide: document how to "un-scratchpad" a window (Thanks knopwob)

fix crash: send non-floating window with floating parent to scratchpad (thanks pkordy)

Fix a crash that occured when moving a window to the scratchpad that is
seemingly floating to the user, but actually a descendant of a floating
parent con (and itself non-floating). If that is the case, move the
floating parent container to scratchpad instead of the window.

fixes #740

Revert "raise floating windows when focusing (Thanks Marcos)"

This commit breaks floating window keyboard focus order
(t/135-floating-focus.t) when you have > 2 floating windows. Since
keyboard focus is more important than saving one click to raise floating
windows, I revert the commit.

Note that we cannot implement this without keeping a third list (beneath
floating_windows and focus) for the z coordinate of a floating window.
This seems not worth it.

This reverts commit 064be457e59603e1d4a4b45b4bf138dcc231a849.

raise floating windows when focusing (Thanks Marcos)

add testcase for commit 87525ad

remove async-unsafe functions from signal handler

Revert "Use ev_signal to avoid async-unsafe functions (Thanks Markus)"

This makes our signal handler useless and leads to infinite SIGSEGV
loops because the ev callback handler gets called only from within the
event loop, and control doesn’t necessary get to the event loop…

This reverts commit 514265b529ac78b7778eeee2db3dddb6f3a1c24c.

fix crash: urgent floating con on separate workspace (thanks Piotr)

If there is a single floating con on a separate workspace that is not
focused, and this con becomes urgent, switching back to that workspace
may result in a crash of i3. This is because while setting the urgency
of parent containers, 'parent' may become NULL in case of floating
containers. This commit checks the validity of parent.

fixes #821