[PATCH 4/4] SndChannel: rename 'connection' to 'client'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Since this field holds the SndChannelClient, rename this variable
'client' to make the code a bit easier to understand.

Signed-off-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
---
 server/sound.c | 56 ++++++++++++++++++++++++++++----------------------------
 1 file changed, 28 insertions(+), 28 deletions(-)

diff --git a/server/sound.c b/server/sound.c
index 7490a19..1e5d56d 100644
--- a/server/sound.c
+++ b/server/sound.c
@@ -166,7 +166,7 @@ GType snd_channel_get_type(void) G_GNUC_CONST;
 struct SndChannel {
     RedChannel parent;
 
-    SndChannelClient *connection; /* Only one client is supported */
+    SndChannelClient *client; /* Only one client is supported */
     SndChannel *next; /* For the global SndChannel list */
 
     gboolean active;
@@ -786,8 +786,8 @@ static bool snd_channel_client_config_socket(RedChannelClient *rcc)
 static void snd_channel_on_disconnect(RedChannelClient *rcc)
 {
     SndChannel *channel = SND_CHANNEL(red_channel_client_get_channel(rcc));
-    if (channel->connection && rcc == RED_CHANNEL_CLIENT(channel->connection)) {
-        channel->connection = NULL;
+    if (channel->client && rcc == RED_CHANNEL_CLIENT(channel->client)) {
+        channel->client = NULL;
     }
 }
 
@@ -824,7 +824,7 @@ static void snd_channel_set_volume(SndChannel *channel,
                                    uint8_t nchannels, uint16_t *volume)
 {
     SpiceVolumeState *st = &channel->volume;
-    SndChannelClient *client = channel->connection;
+    SndChannelClient *client = channel->client;
 
     st->volume_nchannels = nchannels;
     free(st->volume);
@@ -847,7 +847,7 @@ SPICE_GNUC_VISIBLE void spice_server_playback_set_volume(SpicePlaybackInstance *
 static void snd_channel_set_mute(SndChannel *channel, uint8_t mute)
 {
     SpiceVolumeState *st = &channel->volume;
-    SndChannelClient *client = channel->connection;
+    SndChannelClient *client = channel->client;
 
     st->mute = mute;
 
@@ -865,7 +865,7 @@ SPICE_GNUC_VISIBLE void spice_server_playback_set_mute(SpicePlaybackInstance *si
 
 static void snd_playback_start(SndChannel *channel)
 {
-    SndChannelClient *client = channel->connection;
+    SndChannelClient *client = channel->client;
 
     channel->active = TRUE;
     if (!client)
@@ -888,7 +888,7 @@ SPICE_GNUC_VISIBLE void spice_server_playback_start(SpicePlaybackInstance *sin)
 
 SPICE_GNUC_VISIBLE void spice_server_playback_stop(SpicePlaybackInstance *sin)
 {
-    SndChannelClient *client = sin->st->channel.connection;
+    SndChannelClient *client = sin->st->channel.client;
 
     sin->st->channel.active = FALSE;
     if (!client)
@@ -916,7 +916,7 @@ SPICE_GNUC_VISIBLE void spice_server_playback_stop(SpicePlaybackInstance *sin)
 SPICE_GNUC_VISIBLE void spice_server_playback_get_buffer(SpicePlaybackInstance *sin,
                                                          uint32_t **frame, uint32_t *num_samples)
 {
-    SndChannelClient *client = sin->st->channel.connection;
+    SndChannelClient *client = sin->st->channel.client;
 
     *frame = NULL;
     *num_samples = 0;
@@ -952,7 +952,7 @@ SPICE_GNUC_VISIBLE void spice_server_playback_put_samples(SpicePlaybackInstance
         }
     }
     playback_client = frame->client;
-    if (!playback_client || sin->st->channel.connection != SND_CHANNEL_CLIENT(playback_client)) {
+    if (!playback_client || sin->st->channel.client != SND_CHANNEL_CLIENT(playback_client)) {
         /* lost last reference, client has been destroyed previously */
         spice_debug("audio samples belong to a disconnected client");
         return;
@@ -975,16 +975,16 @@ void snd_set_playback_latency(RedClient *client, uint32_t latency)
     for (; now; now = now->next) {
         uint32_t type;
         g_object_get(RED_CHANNEL(now), "channel-type", &type, NULL);
-        if (type == SPICE_CHANNEL_PLAYBACK && now->connection &&
-            red_channel_client_get_client(RED_CHANNEL_CLIENT(now->connection)) == client) {
+        if (type == SPICE_CHANNEL_PLAYBACK && now->client &&
+            red_channel_client_get_client(RED_CHANNEL_CLIENT(now->client)) == client) {
 
-            if (red_channel_client_test_remote_cap(RED_CHANNEL_CLIENT(now->connection),
+            if (red_channel_client_test_remote_cap(RED_CHANNEL_CLIENT(now->client),
                 SPICE_PLAYBACK_CAP_LATENCY)) {
-                PlaybackChannelClient* playback = (PlaybackChannelClient*)now->connection;
+                PlaybackChannelClient* playback = (PlaybackChannelClient*)now->client;
 
                 playback->latency = latency;
-                snd_set_command(now->connection, SND_PLAYBACK_LATENCY_MASK);
-                snd_send(now->connection);
+                snd_set_command(now->client, SND_PLAYBACK_LATENCY_MASK);
+                snd_send(now->client);
             } else {
                 spice_debug("client doesn't not support SPICE_PLAYBACK_CAP_LATENCY");
             }
@@ -1065,7 +1065,7 @@ playback_channel_client_constructed(GObject *object)
     spice_debug("playback client %p using mode %s", playback_client,
                 spice_audio_data_mode_to_string(playback_client->mode));
 
-    channel->connection = scc;
+    channel->client = scc;
     if (red_client_during_migrate_at_target(red_client)) {
         return;
     }
@@ -1086,9 +1086,9 @@ static void snd_set_peer(RedChannel *red_channel, RedClient *client, RedsStream
     SndChannel *channel = SND_CHANNEL(red_channel);
     SndChannelClient *snd_client;
 
-    if (channel->connection) {
-        red_channel_client_disconnect(RED_CHANNEL_CLIENT(channel->connection));
-        channel->connection = NULL;
+    if (channel->client) {
+        red_channel_client_disconnect(RED_CHANNEL_CLIENT(channel->client));
+        channel->client = NULL;
     }
 
     snd_client = g_initable_new(type,
@@ -1129,7 +1129,7 @@ SPICE_GNUC_VISIBLE void spice_server_record_set_mute(SpiceRecordInstance *sin, u
 
 static void snd_record_start(SndChannel *channel)
 {
-    SndChannelClient *client = channel->connection;
+    SndChannelClient *client = channel->client;
 
     channel->active = TRUE;
     if (!client) {
@@ -1155,7 +1155,7 @@ SPICE_GNUC_VISIBLE void spice_server_record_start(SpiceRecordInstance *sin)
 
 SPICE_GNUC_VISIBLE void spice_server_record_stop(SpiceRecordInstance *sin)
 {
-    SndChannelClient *client = sin->st->channel.connection;
+    SndChannelClient *client = sin->st->channel.client;
 
     sin->st->channel.active = FALSE;
     if (!client)
@@ -1173,7 +1173,7 @@ SPICE_GNUC_VISIBLE void spice_server_record_stop(SpiceRecordInstance *sin)
 SPICE_GNUC_VISIBLE uint32_t spice_server_record_get_samples(SpiceRecordInstance *sin,
                                                             uint32_t *samples, uint32_t bufsize)
 {
-    SndChannelClient *client = sin->st->channel.connection;
+    SndChannelClient *client = sin->st->channel.client;
     uint32_t read_pos;
     uint32_t now;
     uint32_t len;
@@ -1223,7 +1223,7 @@ static void snd_set_rate(SndChannel *channel, uint32_t frequency, uint32_t cap_o
 
 SPICE_GNUC_VISIBLE uint32_t spice_server_get_best_playback_rate(SpicePlaybackInstance *sin)
 {
-    return snd_get_best_rate(sin ? sin->st->channel.connection : NULL, SPICE_PLAYBACK_CAP_OPUS);
+    return snd_get_best_rate(sin ? sin->st->channel.client : NULL, SPICE_PLAYBACK_CAP_OPUS);
 }
 
 SPICE_GNUC_VISIBLE void spice_server_set_playback_rate(SpicePlaybackInstance *sin, uint32_t frequency)
@@ -1233,7 +1233,7 @@ SPICE_GNUC_VISIBLE void spice_server_set_playback_rate(SpicePlaybackInstance *si
 
 SPICE_GNUC_VISIBLE uint32_t spice_server_get_best_record_rate(SpiceRecordInstance *sin)
 {
-    return snd_get_best_rate(sin ? sin->st->channel.connection : NULL, SPICE_RECORD_CAP_OPUS);
+    return snd_get_best_rate(sin ? sin->st->channel.client : NULL, SPICE_RECORD_CAP_OPUS);
 }
 
 SPICE_GNUC_VISIBLE void spice_server_set_record_rate(SpiceRecordInstance *sin, uint32_t frequency)
@@ -1261,7 +1261,7 @@ record_channel_client_constructed(GObject *object)
 
     G_OBJECT_CLASS(record_channel_client_parent_class)->constructed(object);
 
-    channel->connection = scc;
+    channel->client = scc;
     if (channel->volume.volume_nchannels) {
         snd_set_command(scc, SND_VOLUME_MUTE_MASK);
     }
@@ -1458,8 +1458,8 @@ void snd_set_playback_compression(bool on)
     for (; now; now = now->next) {
         uint32_t type;
         g_object_get(RED_CHANNEL(now), "channel-type", &type, NULL);
-        if (type == SPICE_CHANNEL_PLAYBACK && now->connection) {
-            PlaybackChannelClient* playback = (PlaybackChannelClient*)now->connection;
+        if (type == SPICE_CHANNEL_PLAYBACK && now->client) {
+            PlaybackChannelClient* playback = (PlaybackChannelClient*)now->client;
             RedChannelClient *rcc = RED_CHANNEL_CLIENT(playback);
             bool client_can_celt = red_channel_client_test_remote_cap(rcc,
                                     SPICE_PLAYBACK_CAP_CELT_0_5_1);
@@ -1469,7 +1469,7 @@ void snd_set_playback_compression(bool on)
                                                       client_can_opus, client_can_celt);
             if (playback->mode != desired_mode) {
                 playback->mode = desired_mode;
-                snd_set_command(now->connection, SND_PLAYBACK_MODE_MASK);
+                snd_set_command(now->client, SND_PLAYBACK_MODE_MASK);
                 spice_debug("playback client %p using mode %s", playback,
                             spice_audio_data_mode_to_string(playback->mode));
             }
-- 
2.9.3

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]