The cleanup is not linear: for example, the server must be stopped and
its sockets must be shutdown after the stream and controller are stopped
(so that they don't continue processing garbage), but before they are
joined, to avoid a deadlock if they are blocked on a socket read.

Simplify the spaghetti-cleanup by keeping trace of initialization at
runtime.

Partial read is impossible for 1 byte, so net_recv_all() is useless.

Only use 2 branches, using either forward or remote tunnel.

The resulting socket is accessible from the server instance, there is no
need to return it.

This paves the way to use several sockets in parallel.

Replace the file_handler_request_queue implementation by cbuf.

Replace the control_event_queue implementation by cbuf.

Add a circular buffer implementation, to factorize multiple specific
queues implementation.

Add a function to read 16 bits in big-endian to a uint16_t.

So that they can be used both on const and non-const input buffers.

control_event_serialize() returns the number of bytes written, so the
type should be size_t.

The variable condition was not destroyed on strdup() failure.

This will avoid conflicts with future device events.

Use parentheses to avoid unexpected results.

For example, make:

    2 * SERIALIZED_EVENT_MAX_SIZE

expand to:

    2 * (3 + TEXT_MAX_LENGTH)

instead of:

    2 * 3 + TEXT_MAX_LENGTH

The struct control_event does not use mutexes, and net.h does not need
SDL_platform.h.

Previous refactorings broke indentation.

Commit dfed1b25 replaced SDL types by
standard types in sources, but tests were not updated.

To avoid mixing SDL_malloc()/SDL_strdup() with free(), or malloc() with
SDL_free(), always use the SDL version.

Checkstyle wants a specific order of imports.

On pressing Home key on the computer, move the cursor to the beginning
of the line instead of going back to the home screen.

<https://developer.android.com/reference/android/view/KeyEvent.html#KEYCODE_HOME>
<https://developer.android.com/reference/android/view/KeyEvent.html#KEYCODE_MOVE_HOME>

Fixes (part of) <https://github.com/Genymobile/scrcpy/issues/555>.

The file handler owns the filename string, so it needs to free it.
Therefore, it should not be a pointer-to-const.

The file handler owns the serial, so it needs to free it. Therefore, it
should not be a pointer-to-const.

If loading the icon from xpm fails, launch scrcpy without window icon.

<https://github.com/Genymobile/scrcpy/issues/539>

SDL_CreateRGBSurfaceFrom() may return NULL, causing a segfault.

<https://github.com/Genymobile/scrcpy/issues/539>

Some characters may not be injected (e.g. '\r`). Log them instead of
ignoring them silently.

Return success count in injectText

It will insert as many text as possible now.
Fix #509, tested on Windows 10 and Arch Linux.
Signed-off-by: NYu-Chen Lin <npes87184@gmail.com>

When the execution of an adb command fails, print the command. This will
help to understand what went wrong.

See <https://github.com/Genymobile/scrcpy/issues/530>.

The size of an array should have type size_t.

Disable X11 compositor bypass

Compositor bypass is meant for fullscreen games consuming lots of GPU
resources. For a light app that will usually be windowed, this only
causes unnecessary compositor suspends, especially visible (and
annoying) with complying window manager like KWin.
Signed-off-by: NRomain Vimont <rom@rom1v.com>

Mouse focus clickthrough didn't work due to compat.h header not being
included in scrcpy.c.
Signed-off-by: NRomain Vimont <rom@rom1v.com>

Some devices don't have the required method. Recover gracefully without
crashing the server.

Fixes <https://github.com/Genymobile/scrcpy/issues/506>.

Add Ln.e(message) in addition to Ln.e(message, error).