Signed-off-by: Daniel Martí <mvdan@mvdan.cc>

This parameter was always UTF_SIZ, so it is better remove it and
use directly UTF_SIZ in it.

st.c:1321:2: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
            system(cmd);
            ^~~~~~ ~~~

Debatable whether an error here should case exit(EXIT_FAILURE). Just
preserving the existing behaviour for now.

We cannot pass strescseq.args[0] to atoi when nargs is zero,
because in this case it will be null.

LEN(colorname) may be below 256 for some configurations.

WIN_REDRAW flag was not used since introduction of Xdbe
in commit 94771d0588

This way is a bit more accurate.

The check was introduced back when st used gettimeofday.
The condition is also modified to increment the accuaracy of the
calculation.

Not always is desirable to create a pseudo terminal, and some times
we want to open a terminal emulator over a tty line. With this new
patch is possible to do someting like:

	$ st -l /dev/ttyS0 115200

Without this option was needed to launch another terminal emulator
over st (for example minicom, picocom, cu, ...).

This fixes a segmentation fault on some systems.

This change allows execute st as 'st mutt' while it keeps the
compability with xterm and urxt.

ICCCM mandates the use of real timestamps to interact with the
selection, to rule out race conditions if the clients are run at
different speeds. I have implemented the low hanging fruit, putting the
timestamps into text selection. Also, ICCCM mandates a check for whether
XSetSelectionOwner() worked. Not sure my version is correct, though.

tmoveto resets CURSOR_WRAPNEXT.

Simple testcase:

for i in $(seq 1 200); do
	printf '\t.';
	usleep 100000;
	printf '\t@';
	usleep 100000;
done

In st executing this script causes @ and . to overwrite each other in
the last column.

It probably refers to
http://lists.suckless.org/dev/1211/13427.html
and does not seem like a bug in st.

We do not free it until exit anyway.

LIMIT returns value. This fact is already used in x2col and y2row.

It seems that LICENSE files are more common than LICENCE files.
At least this patch makes spelling consistent.

There is no need to check that slide > 0 before executing loop.
If slide <= 0, loop stops immediately.

XFilterEvent usually filters KeyPress events according to input method.
At this point the window is not mapped. The only events that we process
are ConfigureNotify and MapNotify. They should not be filtered by input
method.

It was used only once and its return value was ignored.

control was set, but it was not ever used because it was set
again some lines later.

ascii code may only be checked for characters that have length equal to
1, not width equal to 1

strsep() is not a POSIX function, and it means that every system
needs different defines to expose it. If the prototype of strsep
is not exposed then an ugly int/pointer is done and it might mean
a crash. The best solution?, to remove the strsep and make a custom
loop. If C programmers cannot do this kind of loops without calling
a library function, then maybe we should move all the suckless
software to Java.

Otherwise, pasting the X11 primary selection when empty results an
error and Xlib forcibly exits.

Signed-off-by: Christoph Lohmann <20h@r-36.net>