swayfx/common
Ryan Dwyer 701fcafc70 Use list_find in more places and refactor/fix workspace prev_next functions
The original purpose of this commit is to replace some for loops with
list_find. But while doing this I found the workspace_prev_next_impl
functions to be difficult to read and also contained a bug, so I
refactored them and fixed the bug.

To reproduce the bug:

* Have two outputs, where the left output has workspaces 1, 2, 3 and the
right output has workspaces 4, 5, 6. Make workspace 2 focused_inactive
and workspace 4 focused.
* Run `workspace prev`.
* Previously it would visit the left output, then apply `workspace prev`
to workspace 2, which focuses workspace 1.
* Now it will focus the rightmost workspace on the left output
(workspace 3).

The refactoring I made to the workspace functions are:

* Added the static keyword.
* They now accept an int dir rather than bool, to avoid an unnecessary
conversion.
* Rather than preparing start and end variables for the purpose of
iterating, just iterate everything.
* Replace for loops with list_find.
* Don't call workspace_output_prev_next_impl (this fixes the bug).
2018-08-15 15:14:35 +10:00
..
background-image.c Update for swaywm/wlroots#1126 2018-07-09 22:54:30 +01:00
cairo.c Address review comments 2018-03-28 14:43:23 -04:00
ipc-client.c ipc-client: fix memory leaks in get_socketpath 2018-08-06 17:30:24 +01:00
list.c Use list_find in more places and refactor/fix workspace prev_next functions 2018-08-15 15:14:35 +10:00
log.c Update for swaywm/wlroots#1126 2018-07-09 22:54:30 +01:00
meson.build Add password buffer, refactor rendering/surfaces 2018-04-04 18:47:48 -04:00
pango.c Update for swaywm/wlroots#1126 2018-07-09 22:54:30 +01:00
readline.c Update for swaywm/wlroots#1126 2018-07-09 22:54:30 +01:00
stringop.c Clean up container title functions 2018-05-25 21:07:59 +10:00
unicode.c utf8_size: fix loop boundary 2018-07-02 08:03:41 +09:00
util.c Address review comments on parse_boolean 2018-07-23 21:37:53 -04:00