diff options
| author | Francois Gouget <fgouget@codeweavers.com> | 2016-03-14 00:39:02 +0100 |
|---|---|---|
| committer | Christophe Fergeau <cfergeau@redhat.com> | 2016-03-24 15:19:38 +0100 |
| commit | 3486ecde03418f5a7a18f73bcc41403d5cea2efc (patch) | |
| tree | 64af84086bc068a522af338b86a888f6a415acb7 | |
| parent | 5e15934dce67bc8f70e0092f20833e6af69b1c02 (diff) | |
server: More clearly identify channels that support multiple clients
Rename the function to channel_supports_multiple_clients() as
channel_is_secondary() is unclear and was coded to return the opposite
of what was originally intended.
Also there are few channel types and whether they support multiple
clients is totally static so we might as well delegate the check to
the compiler.
Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
| -rw-r--r-- | server/reds.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/server/reds.c b/server/reds.c index eb24303d..89422585 100644 --- a/server/reds.c +++ b/server/reds.c @@ -885,16 +885,14 @@ SPICE_GNUC_VISIBLE int spice_server_get_num_clients(SpiceServer *reds) return reds_get_n_clients(reds); } -static const int secondary_channels[] = { - SPICE_CHANNEL_MAIN, SPICE_CHANNEL_DISPLAY, SPICE_CHANNEL_CURSOR, SPICE_CHANNEL_INPUTS}; - -static int channel_is_secondary(RedChannel *channel) +static int channel_supports_multiple_clients(RedChannel *channel) { - int i; - for (i = 0 ; i < G_N_ELEMENTS(secondary_channels); ++i) { - if (channel->type == secondary_channels[i]) { - return TRUE; - } + switch (channel->type) { + case SPICE_CHANNEL_MAIN: + case SPICE_CHANNEL_DISPLAY: + case SPICE_CHANNEL_CURSOR: + case SPICE_CHANNEL_INPUTS: + return TRUE; } return FALSE; } @@ -907,7 +905,8 @@ void reds_fill_channels(RedsState *reds, SpiceMsgChannels *channels_info) channels_info->num_of_channels = reds->num_of_channels; RING_FOREACH(now, &reds->channels) { RedChannel *channel = SPICE_CONTAINEROF(now, RedChannel, link); - if (reds->num_clients > 1 && !channel_is_secondary(channel)) { + if (reds->num_clients > 1 && + !channel_supports_multiple_clients(channel)) { continue; } channels_info->channels[used_channels].type = channel->type; |