When user clicks LMB, one character is selected, but will not be copied
to selection until the user moves cursor a bit. Therefore, the character
should not be highlighted as selected yet.

Before the patch, the trick was not to mark line as dirty to avoid
highlighting it. However, if user has already selected something and
clicks in line that contains selection, selclear sets the line as dirty
and one character is highlighted when it should not.

This patch replaces dirty trick with explicit check for sel.mode inside
selected().

This patch also prevents sel.mode from increasing beyond 2. It is almost
impossible, but sel.mode may overflow if mouse is moved around for too
long while selecting.

Special case is when regular selection spans multiple lines.
Otherwise, just sort sel.ob.x and sel.ob.y.

Also make sure y never exceeds term.row-1 even if ATTR_WRAP is set for
some reason.

Only wait for termination of the shell.

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

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

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.