The size, point and position structs were defined in common.h. Move them
to coords.h so that common.h could be used for generic code to be
included in all source files.

The header libavformat/version.h was included, but not
libavcodec/version.h.

As a consequence, the LIBAVCODEC_VERSION_INT definition depended on the
caller includes.

It had been replaced by struct sc_port_range in scrcpy.h.

On Linux, waitpid() both waits for the process to terminate and reaps it
(closes its handle). On Windows, these actions are separated into
WaitForSingleObject() and CloseHandle().

Expose these actions separately, so that it is possible to send a signal
to a process while waiting for its termination without race condition.

This allows to wait for server termination normally, but kill the
process without race condition if it is not terminated after some delay.

Adding "simple" in the function name brings no benefit.

Declare all the source files (including the platform-specific ones) at
the beginning.

The process API provides the system-specific implementation, the adb API
uses it to expose adb commands.

Some devices use big display id values.

Refs #2009 <https://github.com/Genymobile/scrcpy/issues/2009>

Due to gotos, "ret" may be returned uninitialized.

PR #2004 <https://github.com/Genymobile/scrcpy/pull/2004>
Signed-off-by: NRomain Vimont <rom@rom1v.com>

Let the server terminate properly once all the sockets are closed.

If it does not terminate (this can happen if the device is asleep), then
kill it.

Note: since the server process termination is detected by a flag set
after waitpid() returns, there is a small chance that the process
terminates (and the PID assigned to a new process) before the flag is
set but before the kill() call. This race condition already existed
before this commit.

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

TerminateProcess() is "equivalent" to kill(), while
WaitForSingleObject() is "equivalent" to waitpid(), so the handle must
be closed after WaitForSingleObject().

This will allow to add mutex/cond fields.

The (non-daemon) threads were not interrupted on video stream stopped,
leaving the server process alive.

Interrupt them to wake up their blocking call so that they terminate
properly.

Refs #1992 <https://github.com/Genymobile/scrcpy/issues/1992>

Mention that it is possible to add default arguments by editing the
wrapper scripts.

Include the latest version of SDL in Windows releases.

It's more standard, and benefits from syntax coloration in vi.

Before this change, release.sh built some native stuff, and
Makefile.CrossWindows built the Windows releases.

Instead, use a single release.make to build the whole release. It also
avoids to build the server one more time.

This simplifies the build system.

Refs <https://github.com/Genymobile/scrcpy/issues/1975#issuecomment-745314161>

PR #1973 <https://github.com/Genymobile/scrcpy/pull/1973>
Reviewed-by: NYu-Chen Lin <npes87184@gmail.com>

On Windows, scrcpy paused on error before exiting to give the user a
chance to see the user message.

This was a hack and causes issues when using scrcpy from batch scripts.

Disable this pause from the scrcpy binary, and provide a batch wrapper
(scrcpy-console.bat) to pause on error.

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

Refs #1753 <https://github.com/Genymobile/scrcpy/issues/1753>

Include the latest version of adb in Windows releases.

PR #1960 <https://github.com/Genymobile/scrcpy/pull/1960>
Signed-off-by: NRomain Vimont <rom@rom1v.com>

PR #1944 <https://github.com/Genymobile/scrcpy/pull/1944>
Signed-off-by: NRomain Vimont <rom@rom1v.com>

Use "%Iu" on Windows. This fixes the following warning:

    ../app/src/sys/win/command.c:17:14: warning: unknown conversion type character ‘l’ in format [-Wformat=]
       17 |         LOGE("Command too long (%" PRIsizet " chars)", len - 1);

Add documentation for the new option --encoder in the manpage and in
README.md.

If an invalid encoder name is given via the --encoder option, list all
the H.264 encoders available on the device.

When the encoder is selected automatically, log the name of the selected
encoder.

Some devices have more than one encoder, and some encoders may cause
issues or crash. With this option we can specify which encoder we want
the device to use.

PR #1827 <https://github.com/Genymobile/scrcpy/pull/1827>
Fixes #1810 <https://github.com/Genymobile/scrcpy/issues/1810>
Signed-off-by: NRomain Vimont <rom@rom1v.com>