From 871c0518ec53309a38debbb86a36c035c3470fba Mon Sep 17 00:00:00 2001 From: Johan Hedberg Date: Thu, 16 Aug 2007 19:24:07 +0000 Subject: Revert changes from previous commit that were not supposed to go in --- audio/avdtp.c | 41 ++++++++++++----------------------------- audio/avdtp.h | 2 +- audio/sink.c | 10 ++++------ audio/unix.c | 28 ---------------------------- 4 files changed, 17 insertions(+), 64 deletions(-) diff --git a/audio/avdtp.c b/audio/avdtp.c index d2733d6e..7194177f 100644 --- a/audio/avdtp.c +++ b/audio/avdtp.c @@ -218,11 +218,6 @@ struct avdtp_local_sep { void *data; }; -struct stream_callback { - avdtp_stream_state_cb cb; - void *user_data; -}; - struct avdtp_stream { int sock; uint16_t mtu; @@ -230,8 +225,9 @@ struct avdtp_stream { struct avdtp_local_sep *lsep; uint8_t rseid; GSList *caps; - GSList *callbacks; struct avdtp_service_capability *codec; + avdtp_stream_state_cb cb; + void *user_data; guint io; /* Transport GSource ID */ guint timer; /* Waiting for other side to close or open the transport channel */ @@ -460,13 +456,10 @@ static void stream_free(struct avdtp_stream *stream) if (stream->timer) g_source_remove(stream->timer); - - g_slist_foreach(stream->callbacks, (GFunc) g_free, NULL); - g_slist_free(stream->callbacks); - - g_slist_foreach(stream->caps, (GFunc) g_free, NULL); - g_slist_free(stream->caps); - + if (stream->caps) { + g_slist_foreach(stream->caps, (GFunc) g_free, NULL); + g_slist_free(stream->caps); + } g_free(stream); } @@ -492,14 +485,9 @@ static void avdtp_sep_set_state(struct avdtp *session, old_state = sep->state; sep->state = state; - if (stream) { - GSList *l; - for (l = stream->callbacks; l != NULL; l = g_slist_next(l)) { - struct stream_callback *cb = l->data; - cb->cb(stream, old_state, state, err_ptr, - cb->user_data); - } - } + if (stream && stream->cb) + stream->cb(stream, old_state, state, err_ptr, + stream->user_data); if (state == AVDTP_STATE_IDLE) { session->streams = g_slist_remove(session->streams, stream); @@ -2097,16 +2085,11 @@ int avdtp_get_seps(struct avdtp *session, uint8_t acp_type, uint8_t media_type, return -EINVAL; } -void avdtp_stream_add_cb(struct avdtp *session, struct avdtp_stream *stream, +void avdtp_stream_set_cb(struct avdtp *session, struct avdtp_stream *stream, avdtp_stream_state_cb cb, void *data) { - struct stream_callback *stream_cb; - - stream_cb = g_new(struct stream_callback, 1); - stream_cb->cb = cb; - stream_cb->user_data = data; - - stream->callbacks = g_slist_append(stream->callbacks, stream_cb);; + stream->cb = cb; + stream->user_data = data; } int avdtp_get_configuration(struct avdtp *session, struct avdtp_stream *stream) diff --git a/audio/avdtp.h b/audio/avdtp.h index aa835131..0043c480 100644 --- a/audio/avdtp.h +++ b/audio/avdtp.h @@ -165,7 +165,7 @@ struct avdtp_service_capability *avdtp_get_codec(struct avdtp_remote_sep *sep); int avdtp_discover(struct avdtp *session, avdtp_discover_cb_t cb, void *user_data); -void avdtp_stream_add_cb(struct avdtp *session, struct avdtp_stream *stream, +void avdtp_stream_set_cb(struct avdtp *session, struct avdtp_stream *stream, avdtp_stream_state_cb cb, void *data); gboolean avdtp_stream_get_transport(struct avdtp_stream *stream, int *sock, diff --git a/audio/sink.c b/audio/sink.c index fd6e9db2..0f69bbf6 100644 --- a/audio/sink.c +++ b/audio/sink.c @@ -71,11 +71,9 @@ static void pending_request_free(struct pending_request *pending) g_free(pending); } -static void stream_state_changed(struct avdtp_stream *stream, - avdtp_state_t old_state, - avdtp_state_t new_state, - struct avdtp_error *err, - void *user_data) +void stream_state_changed(struct avdtp_stream *stream, avdtp_state_t old_state, + avdtp_state_t new_state, + struct avdtp_error *err, void *user_data) { struct device *dev = user_data; struct sink *sink = dev->sink; @@ -317,7 +315,7 @@ gboolean sink_new_stream(struct device *dev, struct avdtp *session, sink->stream = stream; sink->initiator = FALSE; - avdtp_stream_add_cb(session, stream, stream_state_changed, dev); + avdtp_stream_set_cb(session, stream, stream_state_changed, dev); return TRUE; } diff --git a/audio/unix.c b/audio/unix.c index 0e5fd5c1..10856b93 100644 --- a/audio/unix.c +++ b/audio/unix.c @@ -140,30 +140,6 @@ static service_type_t select_service(struct device *dev) return TYPE_NONE; } - -static void stream_state_changed(struct avdtp_stream *stream, - avdtp_state_t old_state, - avdtp_state_t new_state, - struct avdtp_error *err, - void *user_data) -{ - struct unix_client *client = user_data; - - if (err) - return; - - switch (new_state) { - case AVDTP_STATE_IDLE: - if (client->data.session) { - avdtp_unref(client->data.session); - client->data.session = NULL; - } - break; - default: - break; - } -} - static void a2dp_setup_complete(struct avdtp *session, struct device *dev, struct avdtp_stream *stream, void *user_data) @@ -247,15 +223,11 @@ static void a2dp_setup_complete(struct avdtp *session, struct device *dev, unix_send_cfg(client->sock, cfg, fd); - avdtp_stream_add_cb(session, stream, stream_state_changed, dev); - return; failed: unix_send_cfg(client->sock, NULL, -1); a2dp_source_unlock(dev, session); - avdtp_unref(client->data.session); - client->data.session = NULL; } static void cfg_event(struct unix_client *client, struct ipc_packet *pkt, -- cgit