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>

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>

Build failed on WSL because of lack of reference to WinMain@16 during
linking.

Fixes <https://github.com/Genymobile/scrcpy/issues/316>
Signed-off-by: NRomain Vimont <rom@rom1v.com>

This may capture more errors (like
e2ef39fa).

Fix errors reported by:

    gradle -p server check

The serialized text is not nul-terminated (its size is explicitely
provided), but the input text in the event is a nul-terminated string.

The test was failing with ASAN enabled.

The `scrcpy-prebuiltserver` has been deprecated in favor of the `scrcpy`
package.

<https://aur.archlinux.org/cgit/aur.git/commit/?h=scrcpy-prebuiltserver&id=2ef4359b2e45fc278a191fae014d381b486ffcfe>
Signed-off-by: NRomain Vimont <rom@rom1v.com>

Ctrl+C kills the app on Windows, so the recorded file is broken.

The "initial texture" is logged at INFO level. For consistency, log "new
texture" at the same level.

Some decoder and recorder functions must not write to AVCodec and
AVPacket.

server/meson.build: support relative path for prebuilt_server

Replace the "global" control flag in the input_manager by a function
parameter to make explicit that the behavior depends whether
--no-control has been set.

If we don't do this trick, the prebuilt_server will be
../server/[the_user_defined_path]. In general, we will not give an relative path
based on build directory, which leads to wrong prebuilt_server path.

The building error:

ninja: error: '../scrcpy-server-v1.7.jar', needed by
'server/scrcpy-server.jar', missing and no known rule to make it
Signed-off-by: NYu-Chen Lin <npes87184@gmail.com>

The SDL video subsystem is not necessary if we don't display the video.

Move the sdl_init_and_configure() function from screen.c to scrcpy.c,
because it is not only related to the screen display.

The function video_buffer_offer_decoded_frame() returned a bool to
indicate whether the previous frame had been consumed.

This was confusing, because we could expect the returned bool report
whether the action succeeded.

Make the semantic explicit by using an output parameter.

Also revert the flag (report if the frame has been skipped instead of
consumed) to avoid confusion for the first frame (the previous is
neither skipped nor consumed because there is no previous frame).

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

Scrcpy is a C11 project. Use the C99 standard types instead of the
SDL-specific types:

    SDL_bool -> bool
    SintXX   -> intXX_t
    UintXX   -> uintXX_t

Add an option to disable device control: -n/--no-control.

The description of scrcpy is "Display and control your Android device".
We want an option to disable display, another one to disable control.

For naming consistency, name it --no-display.

Also change the shortname to -N, so that we can use -n for --no-control
later.

Initialize the field no_window in "struct args"

If --no-window is passed, there is no need to register an event watcher.

To avoid too many levels of nested blocks, move the event handling logic
in a separate function.

Limit source code to 80 chars, and declare functions return type and
modifiers on a separate line.

This allows to avoid very long lines, and all function names are
aligned.

(We do this on VLC, and I like it.)

The field pts is declared Uint64 in struct frame_meta.

Disable decoder, screen (display), file_handler and controller when
--no-window is requested.

<https://github.com/Genymobile/scrcpy/pull/418>