621 Commits (bb4614d55899bd1df00b14d3c7f3bdac3a5c818a)

Author	SHA1	Message	Date
Frank Leon Rose	aaf7875d92	Ensure get_server_path() retval is freeable ... PR #2276 <https://github.com/Genymobile/scrcpy/pull/2276> Signed-off-by: Romain Vimont <rom@rom1v.com>	3 years ago
brunoais	b9c3f65fd8	Provide actions for the extra mouse buttons ... Bind APP_SWITCH to button 4 and expand notification panel on button 5. PR #2258 <https://github.com/Genymobile/scrcpy/pull/2258> Signed-off-by: Romain Vimont <rom@rom1v.com>	3 years ago
Romain Vimont	d0739911a3	Forward DOWN and UP separately for right-click ... The shortcut "back on screen on" is a bit special: the control is requested by the client, but the actual event injection (POWER or BACK) is determined on the device. To properly inject DOWN and UP events for BACK, transmit the action as a control parameter. If the screen is off: - on DOWN, inject POWER (DOWN and UP) (wake up the device immediately) - on UP, do nothing If the screen is on: - on DOWN, inject BACK DOWN - on UP, inject BACK UP A corner case is when the screen turns off between the DOWN and UP event. In that case, a BACK UP event will be injected, so it's harmless. As a consequence of this change, the BACK button is now handled by Android on mouse released. This is consistent with the keyboard shortcut (Mod+b) behavior. PR #2259 <https://github.com/Genymobile/scrcpy/pull/2259> Refs #2258 <https://github.com/Genymobile/scrcpy/pull/2258>	3 years ago
Romain Vimont	964b6d2243	Forward DOWN and UP separately for middle-click ... As a consequence of this change, the HOME button is now handled by Android on mouse released. This is consistent with the keyboard shortcut (MOD+h) behavior. PR #2259 <https://github.com/Genymobile/scrcpy/pull/2259> Refs #2258 <https://github.com/Genymobile/scrcpy/pull/2258>	3 years ago
Romain Vimont	8cc057c8f1	Prevent forwarding only "mouse released" events ... Some mouse clicks DOWN are captured for shortcuts, but the matching UP events were still forwarded to the device. Instead, capture both DOWN and UP for shortcuts, and do nothing on UP. PR #2259 <https://github.com/Genymobile/scrcpy/pull/2259> Refs #2258 <https://github.com/Genymobile/scrcpy/pull/2258>	3 years ago
Romain Vimont	edee69d637	Fix options alphabetical order ... "verbosity" < "version"	3 years ago
Romain Vimont	8ef4c044fa	Do not forward SDL_DROPFILE event ... The event is handled by scrcpy.c, it is not necessary to send it to screen or input_manager.	3 years ago
Romain Vimont	c23c38f99d	Move resizing workaround to screen.c	3 years ago
Romain Vimont	65c4f487b3	Set initial fullscreen from screen.c	3 years ago
Romain Vimont	c6d7f5ee96	Make screen_show_window() static ... It is only used from screen.c now.	3 years ago
Romain Vimont	28f6cbaea6	Destroy screen once stream is finished ... The screen receives callbacks from the decoder, fed by the stream. The decoder is run from the stream thread, so waiting for the end of stream is sufficient to avoid possible use-after-destroy.	3 years ago
Romain Vimont	08fc6694e1	Do not destroy uninitialized screen ... When --no-display was passed, screen_destroy() was called while screen_init() was never called. In practice, it did not crash because it just freed NULL pointers, but it was still incorrect.	3 years ago
Romain Vimont	d0983db592	Make internal recorder function static	3 years ago
Romain Vimont	fb7870500a	Remove unused field from input_manager	3 years ago
Romain Vimont	33006561c7	Remove useless forward declaration from stream.h	3 years ago
Romain Vimont	a09733d175	Remove useless includes from decoder.c	3 years ago
Romain Vimont	6231f683af	Fix compilation error for old decoding API ... Commits cb9c42bdcb and 441d3fb119 updated the code only for the new decoding API.	3 years ago
Romain Vimont	9826c5c4a4	Remove HiDPI compilation flag ... Always enable HiDPI support, there is no reason to expose a compilation flag.	3 years ago
Yu-Chen Lin	1d615a0d51	Support power off on close ... PR #824 <https://github.com/Genymobile/scrcpy/pull/824> Signed-off-by: Yu-Chen Lin <npes87184@gmail.com> Signed-off-by: Romain Vimont <rom@rom1v.com>	3 years ago
Romain Vimont	eb7e1070cf	Release frame data as soon as possible ... During a frame swap, one of the two frames involved can be released.	3 years ago
Romain Vimont	386f017ba9	Factorize frame swap	3 years ago
Romain Vimont	cc48b24324	Simplify screen initialization ... Use a single function to initialize the screen instance.	3 years ago
Romain Vimont	597c54f049	Group screen parameters into a struct ... The function screen_init_rendering had too many parameters.	3 years ago
Romain Vimont	955da3b578	Remove screen static initializer ... Most of the fields are initialized dynamically.	3 years ago
Romain Vimont	cb9c42bdcb	Use a callback to notify frame skip ... A skipped frame is detected when the producer offers a frame while the current pending frame has not been consumed. However, the producer (in practice the decoder) is not interested in the fact that a frame has been skipped, only the consumer (the renderer) is. Therefore, notify frame skip via a consumer callback. This allows to manage the skipped and rendered frames count at the same place, and remove fps_counter from decoder.	3 years ago
Romain Vimont	fb9f9848bd	Use a callback to notify a new frame ... Make the decoder independant of the SDL even mechanism, by making the consumer register a callback on the video_buffer.	3 years ago
Romain Vimont	c50b958ee4	Initialize screen before starting the stream ... As soon as the stream is started, the video buffer could notify a new frame available. In order to pass this event to the screen without race condition, the screen must be initialized before the screen is started.	3 years ago
Romain Vimont	441d3fb119	Make video buffer more generic ... Video buffer is a tool between a frame producer and a frame consumer. For now, it is used between a decoder and a renderer, but in the future another instance might be used to swscale decoded frames.	3 years ago
Romain Vimont	cb197ee3a2	Move fps counter out of video buffer ... In order to make video buffer more generic, move out its specific responsibility to count the fps between the decoder and the renderer.	3 years ago
Romain Vimont	b16b65a715	Simplify default values ... It makes sense to extract default values for bitrate and port range (which are arbitrary and might be changed in the future). However, the default values for "max size" and "lock video orientation" are naturally unlimited/unlocked, and will never be changed. Extracting these options just added complexity for no benefit, so hardcode them.	3 years ago
Romain Vimont	a3aa5ac95e	Insert numerical values statically in usage string	3 years ago
Romain Vimont	0207e3df33	Remove unused no_window field	3 years ago
Romain Vimont	24b637b972	Handle im-related events from input_manager.c	3 years ago
Romain Vimont	76a3d9805b	Inline window events handling ... Now that all screen-related events are handled from screen.c, there is no need for a separate method for window events.	3 years ago
Romain Vimont	50b4a730e3	Handle screen-related events from screen.c	3 years ago
Romain Vimont	ea2369f568	Reference video buffer from screen ... This paves the way to handle EVENT_NEW_FRAME from screen.c, by allowing to call screen_update_frame() without an explicit video_buffer instance.	3 years ago
Romain Vimont	0538e9645b	Improve error handling in screen initialization ... After the struct screen is initialized, the window, the renderer and the texture are necessarily valid, so there is no need to check in screen_destroy().	3 years ago
Romain Vimont	626094ad13	Handle window events only once visible ... This will avoid corner cases where we need to resize while no frame has been received yet.	3 years ago
Romain Vimont	a566635c43	Log mipmaps error only if mipmaps are enabled	3 years ago
Romain Vimont	862948b132	Make use_opengl local ... The flag is used only locally, there is no need to store it in the screen structure.	3 years ago
Romain Vimont	c0c4ba7009	Add intermediate frame in video buffer ... There were only two frames simultaneously: - one used by the decoder; - one used by the renderer. When the decoder finished decoding a frame, it swapped it with the rendering frame. Adding a third frame provides several benefits: - the decoder do not have to wait for the renderer to release the mutex; - it simplifies the video_buffer API; - it makes the rendering frame valid until the next call to video_buffer_take_rendering_frame(), which will be useful for swscaling on window resize.	3 years ago
Romain Vimont	c53bd4d8b6	Assert non-recursive usage of mutexes	3 years ago
Romain Vimont	54f5c42d7b	Add mutex assertions	3 years ago
Romain Vimont	21d206f360	Expose mutex assertions ... Add a function to assert that the mutex is held (or not).	3 years ago
Romain Vimont	d2689fc168	Expose thread id	3 years ago
Romain Vimont	f6320c7e31	Wrap SDL thread functions into scrcpy-specific API ... The goal is to expose a consistent API for system tools, and paves the way to make the "core" independant of SDL in the future.	3 years ago
Romain Vimont	30e619d37f	Replace SDL_strdup() by strdup() ... The functions SDL_malloc(), SDL_free() and SDL_strdup() were used only because strdup() was not available everywhere. Now that it is available, use the native version of these functions.	3 years ago
Romain Vimont	c0dde0fade	Provide strdup() compat ... Make strdup() available on all platforms.	3 years ago
Romain Vimont	ace438e52a	Remove unused port_range field ... The port_range is used from "struct server_params", the copy in "struct server" was unused.	3 years ago
Romain Vimont	8e83f3e8af	Remove unused custom event	3 years ago