remote: conditionally declare tty variable for non-Win32 platforms

The 'tty' variable is only used on Win32. Instead of just annotating it
with ATTRIBUTE_UNUSED, make its declaration conditional on WIN32 so that
it is clear why it is not used.

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Daniel P. Berrangé 2019-07-04 11:24:37 +01:00
parent 6a2e551f82
commit 38e9372125

View File

@ -803,7 +803,9 @@ doRemoteOpen(virConnectPtr conn,
VIR_AUTOFREE(char *) knownHosts = NULL;
bool sanity = true;
bool verify = true;
bool tty ATTRIBUTE_UNUSED = true;
#ifndef WIN32
bool tty = true;
#endif
/* We handle *ALL* URIs here. The caller has rejected any
* URIs we don't care about */
@ -908,7 +910,9 @@ doRemoteOpen(virConnectPtr conn,
EXTRACT_URI_ARG_BOOL("no_sanity", sanity);
EXTRACT_URI_ARG_BOOL("no_verify", verify);
#ifndef WIN32
EXTRACT_URI_ARG_BOOL("no_tty", tty);
#endif
if (STRCASEEQ(var->name, "authfile")) {
/* Strip this param, used by virauth.c */