>
> The variable is unnecessary, we can pass the value straight to
> setlogmask().
>
> Signed-off-by: Lukáš Hrázký <lhrazky@xxxxxxxxxx>
Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> src/spice-streaming-agent.cpp | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/src/spice-streaming-agent.cpp b/src/spice-streaming-agent.cpp
> index 392936a..2bdfc32 100644
> --- a/src/spice-streaming-agent.cpp
> +++ b/src/spice-streaming-agent.cpp
> @@ -409,7 +409,6 @@ int main(int argc, char* argv[])
> const char *log_filename = NULL;
> bool log_binary = false;
> bool log_frames = false;
> - int logmask = LOG_UPTO(LOG_WARNING);
> const char *pluginsdir = PLUGINSDIR;
> enum {
> OPT_first = UCHAR_MAX,
> @@ -428,7 +427,7 @@ int main(int argc, char* argv[])
>
> SyslogRAII syslog_raii;
>
> - setlogmask(logmask);
> + setlogmask(LOG_UPTO(LOG_WARNING));
>
> while ((opt = getopt_long(argc, argv, "hp:c:l:d", long_options, NULL))
> != -1) {
> switch (opt) {
> @@ -467,8 +466,7 @@ int main(int argc, char* argv[])
> log_filename = optarg;
> break;
> case 'd':
> - logmask = LOG_UPTO(LOG_DEBUG);
> - setlogmask(logmask);
> + setlogmask(LOG_UPTO(LOG_DEBUG));
> break;
> case 'h':
> usage(argv[0]);
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel