Android/scrcpy
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity

Use constant string for known booleans

Browse source 
Boolean options explicitly passed to the server are statically known.
This commit is contained in:
Romain Vimont 2022-01-15 22:57:38 +01:00
parent afa4a1b728
commit 1c71bd16be
1 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
app/src/server.c
View file

@ -188,7 +188,6 @@ execute_server(struct sc_server *server,
} \ } \
cmd[count++] = p; \ cmd[count++] = p; \
} }
#define STRBOOL(v) (v ? "true" : "false")
ADD_PARAM("log_level=%s", log_level_to_server_string(params->log_level)); ADD_PARAM("log_level=%s", log_level_to_server_string(params->log_level));
ADD_PARAM("bit_rate=%" PRIu32, params->bit_rate); ADD_PARAM("bit_rate=%" PRIu32, params->bit_rate);
@ -204,23 +203,23 @@ execute_server(struct sc_server *server,
params->lock_video_orientation); params->lock_video_orientation);
} }
if (server->tunnel.forward) { if (server->tunnel.forward) {
ADD_PARAM("tunnel_forward=%s", STRBOOL(server->tunnel.forward)); ADD_PARAM("tunnel_forward=true");
} }
if (params->crop) { if (params->crop) {
ADD_PARAM("crop=%s", params->crop); ADD_PARAM("crop=%s", params->crop);
} }
if (!params->control) { if (!params->control) {
// By default, control is true // By default, control is true
ADD_PARAM("control=%s", STRBOOL(params->control)); ADD_PARAM("control=false");
} }
if (params->display_id) { if (params->display_id) {
ADD_PARAM("display_id=%" PRIu32, params->display_id); ADD_PARAM("display_id=%" PRIu32, params->display_id);
} }
if (params->show_touches) { if (params->show_touches) {
ADD_PARAM("show_touches=%s", STRBOOL(params->show_touches)); ADD_PARAM("show_touches=true");
} }
if (params->stay_awake) { if (params->stay_awake) {
ADD_PARAM("stay_awake=%s", STRBOOL(params->stay_awake)); ADD_PARAM("stay_awake=true");
} }
if (params->codec_options) { if (params->codec_options) {
ADD_PARAM("codec_options=%s", params->codec_options); ADD_PARAM("codec_options=%s", params->codec_options);
@ -229,11 +228,11 @@ execute_server(struct sc_server *server,
ADD_PARAM("encoder_name=%s", params->encoder_name); ADD_PARAM("encoder_name=%s", params->encoder_name);
} }
if (params->power_off_on_close) { if (params->power_off_on_close) {
ADD_PARAM("power_off_on_close=%s", STRBOOL(params->power_off_on_close)); ADD_PARAM("power_off_on_close=true");
} }
if (!params->clipboard_autosync) { if (!params->clipboard_autosync) {
// By default, clipboard_autosync is true // By default, clipboard_autosync is true
ADD_PARAM("clipboard_autosync=%s", STRBOOL(params->clipboard_autosync)); ADD_PARAM("clipboard_autosync=false");
} }
#undef ADD_PARAM #undef ADD_PARAM