mirror of
https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git
synced 2025-11-07 13:30:03 -05:00
finish modernizations in pulse/, s/assert/pa_assert/g
git-svn-id: file:///home/lennart/svn/public/pulseaudio/branches/lennart@1805 fefdeb5f-60dc-0310-8127-8f9354f1896f
This commit is contained in:
Lennart Poettering
parent
038e560152
commit
27f13b3853
5 changed files with 151 additions and 133 deletions
|
|
@ -25,7 +25,6 @@
|
|||
#include "config.h"
|
||||
#endif
|
||||
|
||||
#include <assert.h>
|
||||
#include <string.h>
|
||||
|
||||
#include <avahi-client/lookup.h>
|
||||
|
|
@ -36,8 +35,9 @@
|
|||
|
||||
#include <pulsecore/log.h>
|
||||
#include <pulsecore/core-util.h>
|
||||
|
||||
#include <pulsecore/avahi-wrap.h>
|
||||
#include <pulsecore/refcnt.h>
|
||||
#include <pulsecore/macro.h>
|
||||
|
||||
#include "browser.h"
|
||||
|
||||
|
|
@ -46,7 +46,8 @@
|
|||
#define SERVICE_TYPE_SERVER "_pulse-server._tcp."
|
||||
|
||||
struct pa_browser {
|
||||
int ref;
|
||||
PA_REFCNT_DECLARE;
|
||||
|
||||
pa_mainloop_api *mainloop;
|
||||
AvahiPoll* avahi_poll;
|
||||
|
||||
|
|
@ -62,6 +63,7 @@ struct pa_browser {
|
|||
};
|
||||
|
||||
static int map_to_opcode(const char *type, int new) {
|
||||
|
||||
if (avahi_domain_equal(type, SERVICE_TYPE_SINK))
|
||||
return new ? PA_BROWSE_NEW_SINK : PA_BROWSE_REMOVE_SINK;
|
||||
else if (avahi_domain_equal(type, SERVICE_TYPE_SOURCE))
|
||||
|
|
@ -97,7 +99,8 @@ static void resolve_callback(
|
|||
int ss_valid = 0;
|
||||
char *key = NULL, *value = NULL;
|
||||
|
||||
assert(b);
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
memset(&i, 0, sizeof(i));
|
||||
i.name = name;
|
||||
|
|
@ -109,12 +112,12 @@ static void resolve_callback(
|
|||
goto fail;
|
||||
|
||||
opcode = map_to_opcode(type, 1);
|
||||
assert(opcode >= 0);
|
||||
pa_assert(opcode >= 0);
|
||||
|
||||
if (aa->proto == AVAHI_PROTO_INET)
|
||||
pa_snprintf(a, sizeof(a), "tcp:%s:%u", avahi_address_snprint(ip, sizeof(ip), aa), port);
|
||||
else {
|
||||
assert(aa->proto == AVAHI_PROTO_INET6);
|
||||
pa_assert(aa->proto == AVAHI_PROTO_INET6);
|
||||
pa_snprintf(a, sizeof(a), "tcp6:%s:%u", avahi_address_snprint(ip, sizeof(ip), aa), port);
|
||||
}
|
||||
i.server = a;
|
||||
|
|
@ -146,7 +149,7 @@ static void resolve_callback(
|
|||
value = NULL;
|
||||
|
||||
l = strlen(a);
|
||||
assert(l+1 <= sizeof(a));
|
||||
pa_assert(l+1 <= sizeof(a));
|
||||
strncat(a, " ", sizeof(a)-l-1);
|
||||
strncat(a, i.fqdn, sizeof(a)-l-2);
|
||||
} else if (!strcmp(key, "cookie")) {
|
||||
|
|
@ -211,7 +214,9 @@ fail:
|
|||
|
||||
static void handle_failure(pa_browser *b) {
|
||||
const char *e = NULL;
|
||||
assert(b);
|
||||
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
if (b->sink_browser)
|
||||
avahi_service_browser_free(b->sink_browser);
|
||||
|
|
@ -245,7 +250,9 @@ static void browse_callback(
|
|||
void *userdata) {
|
||||
|
||||
pa_browser *b = userdata;
|
||||
assert(b);
|
||||
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
switch (event) {
|
||||
case AVAHI_BROWSER_NEW: {
|
||||
|
|
@ -276,7 +283,7 @@ static void browse_callback(
|
|||
i.name = name;
|
||||
|
||||
opcode = map_to_opcode(type, 0);
|
||||
assert(opcode >= 0);
|
||||
pa_assert(opcode >= 0);
|
||||
|
||||
b->callback(b, opcode, &i, b->userdata);
|
||||
}
|
||||
|
|
@ -295,7 +302,10 @@ static void browse_callback(
|
|||
|
||||
static void client_callback(AvahiClient *s, AvahiClientState state, void *userdata) {
|
||||
pa_browser *b = userdata;
|
||||
assert(s);
|
||||
|
||||
pa_assert(s);
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
if (state == AVAHI_CLIENT_FAILURE)
|
||||
handle_failure(b);
|
||||
|
|
@ -311,14 +321,14 @@ pa_browser *pa_browser_new_full(pa_mainloop_api *mainloop, pa_browse_flags_t fla
|
|||
pa_browser *b;
|
||||
int error;
|
||||
|
||||
assert(mainloop);
|
||||
pa_assert(mainloop);
|
||||
|
||||
if (flags & ~(PA_BROWSE_FOR_SERVERS|PA_BROWSE_FOR_SINKS|PA_BROWSE_FOR_SOURCES) || flags == 0)
|
||||
return NULL;
|
||||
|
||||
b = pa_xnew(pa_browser, 1);
|
||||
b->mainloop = mainloop;
|
||||
b->ref = 1;
|
||||
PA_REFCNT_INIT(b);
|
||||
b->callback = NULL;
|
||||
b->userdata = NULL;
|
||||
b->error_callback = NULL;
|
||||
|
|
@ -391,7 +401,8 @@ fail:
|
|||
}
|
||||
|
||||
static void browser_free(pa_browser *b) {
|
||||
assert(b && b->mainloop);
|
||||
pa_assert(b);
|
||||
pa_assert(b->mainloop);
|
||||
|
||||
if (b->sink_browser)
|
||||
avahi_service_browser_free(b->sink_browser);
|
||||
|
|
@ -410,29 +421,32 @@ static void browser_free(pa_browser *b) {
|
|||
}
|
||||
|
||||
pa_browser *pa_browser_ref(pa_browser *b) {
|
||||
assert(b);
|
||||
assert(b->ref >= 1);
|
||||
b->ref++;
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
PA_REFCNT_INC(b);
|
||||
return b;
|
||||
}
|
||||
|
||||
void pa_browser_unref(pa_browser *b) {
|
||||
assert(b);
|
||||
assert(b->ref >= 1);
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
if ((-- (b->ref)) <= 0)
|
||||
if (PA_REFCNT_DEC(b) <= 0)
|
||||
browser_free(b);
|
||||
}
|
||||
|
||||
void pa_browser_set_callback(pa_browser *b, pa_browse_cb_t cb, void *userdata) {
|
||||
assert(b);
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
b->callback = cb;
|
||||
b->userdata = userdata;
|
||||
}
|
||||
|
||||
void pa_browser_set_error_callback(pa_browser *b, pa_browser_error_cb_t cb, void *userdata) {
|
||||
assert(b);
|
||||
pa_assert(b);
|
||||
pa_assert(PA_REFCNT_VALUE(b) >= 1);
|
||||
|
||||
b->error_callback = cb;
|
||||
b->error_userdata = userdata;
|
||||
|
|
|
|||
|
|
@ -27,7 +27,6 @@
|
|||
#endif
|
||||
|
||||
#include <stdio.h>
|
||||
#include <assert.h>
|
||||
#include <stdlib.h>
|
||||
#include <string.h>
|
||||
#include <sys/types.h>
|
||||
|
|
@ -67,6 +66,7 @@
|
|||
#include <pulsecore/log.h>
|
||||
#include <pulsecore/socket-util.h>
|
||||
#include <pulsecore/creds.h>
|
||||
#include <pulsecore/macro.h>
|
||||
|
||||
#include "internal.h"
|
||||
|
||||
|
|
@ -90,7 +90,7 @@ static const pa_pdispatch_cb_t command_table[PA_COMMAND_MAX] = {
|
|||
};
|
||||
|
||||
static void unlock_autospawn_lock_file(pa_context *c) {
|
||||
assert(c);
|
||||
pa_assert(c);
|
||||
|
||||
if (c->autospawn_lock_fd >= 0) {
|
||||
char lf[PATH_MAX];
|
||||
|
|
@ -106,8 +106,8 @@ static void context_free(pa_context *c);
|
|||
pa_context *pa_context_new(pa_mainloop_api *mainloop, const char *name) {
|
||||
pa_context *c;
|
||||
|
||||
assert(mainloop);
|
||||
assert(name);
|
||||
pa_assert(mainloop);
|
||||
pa_assert(name);
|
||||
|
||||
c = pa_xnew(pa_context, 1);
|
||||
PA_REFCNT_INIT(c);
|
||||
|
|
@ -168,7 +168,7 @@ pa_context *pa_context_new(pa_mainloop_api *mainloop, const char *name) {
|
|||
}
|
||||
|
||||
static void context_free(pa_context *c) {
|
||||
assert(c);
|
||||
pa_assert(c);
|
||||
|
||||
unlock_autospawn_lock_file(c);
|
||||
|
||||
|
|
@ -206,24 +206,24 @@ static void context_free(pa_context *c) {
|
|||
}
|
||||
|
||||
pa_context* pa_context_ref(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_REFCNT_INC(c);
|
||||
return c;
|
||||
}
|
||||
|
||||
void pa_context_unref(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
if (PA_REFCNT_DEC(c) <= 0)
|
||||
context_free(c);
|
||||
}
|
||||
|
||||
void pa_context_set_state(pa_context *c, pa_context_state_t st) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
if (c->state == st)
|
||||
return;
|
||||
|
|
@ -264,16 +264,16 @@ void pa_context_set_state(pa_context *c, pa_context_state_t st) {
|
|||
}
|
||||
|
||||
void pa_context_fail(pa_context *c, int error) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
pa_context_set_error(c, error);
|
||||
pa_context_set_state(c, PA_CONTEXT_FAILED);
|
||||
}
|
||||
|
||||
int pa_context_set_error(pa_context *c, int error) {
|
||||
assert(error >= 0);
|
||||
assert(error < PA_ERR_MAX);
|
||||
pa_assert(error >= 0);
|
||||
pa_assert(error < PA_ERR_MAX);
|
||||
|
||||
if (c)
|
||||
c->error = error;
|
||||
|
|
@ -284,8 +284,8 @@ int pa_context_set_error(pa_context *c, int error) {
|
|||
static void pstream_die_callback(pa_pstream *p, void *userdata) {
|
||||
pa_context *c = userdata;
|
||||
|
||||
assert(p);
|
||||
assert(c);
|
||||
pa_assert(p);
|
||||
pa_assert(c);
|
||||
|
||||
pa_context_fail(c, PA_ERR_CONNECTIONTERMINATED);
|
||||
}
|
||||
|
|
@ -293,9 +293,9 @@ static void pstream_die_callback(pa_pstream *p, void *userdata) {
|
|||
static void pstream_packet_callback(pa_pstream *p, pa_packet *packet, const pa_creds *creds, void *userdata) {
|
||||
pa_context *c = userdata;
|
||||
|
||||
assert(p);
|
||||
assert(packet);
|
||||
assert(c);
|
||||
pa_assert(p);
|
||||
pa_assert(packet);
|
||||
pa_assert(c);
|
||||
|
||||
pa_context_ref(c);
|
||||
|
||||
|
|
@ -309,18 +309,19 @@ static void pstream_memblock_callback(pa_pstream *p, uint32_t channel, int64_t o
|
|||
pa_context *c = userdata;
|
||||
pa_stream *s;
|
||||
|
||||
assert(p);
|
||||
assert(chunk);
|
||||
assert(chunk->memblock);
|
||||
assert(chunk->length);
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(p);
|
||||
pa_assert(chunk);
|
||||
pa_assert(chunk->memblock);
|
||||
pa_assert(chunk->length);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
pa_context_ref(c);
|
||||
|
||||
if ((s = pa_dynarray_get(c->record_streams, channel))) {
|
||||
|
||||
assert(seek == PA_SEEK_RELATIVE && offset == 0);
|
||||
pa_assert(seek == PA_SEEK_RELATIVE);
|
||||
pa_assert(offset == 0);
|
||||
|
||||
pa_memblockq_seek(s->record_memblockq, offset, seek);
|
||||
pa_memblockq_push_align(s->record_memblockq, chunk);
|
||||
|
|
@ -337,11 +338,11 @@ static void pstream_memblock_callback(pa_pstream *p, uint32_t channel, int64_t o
|
|||
}
|
||||
|
||||
int pa_context_handle_error(pa_context *c, uint32_t command, pa_tagstruct *t) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
if (command == PA_COMMAND_ERROR) {
|
||||
assert(t);
|
||||
pa_assert(t);
|
||||
|
||||
if (pa_tagstruct_getu32(t, &c->error) < 0) {
|
||||
pa_context_fail(c, PA_ERR_PROTOCOL);
|
||||
|
|
@ -361,9 +362,9 @@ int pa_context_handle_error(pa_context *c, uint32_t command, pa_tagstruct *t) {
|
|||
static void setup_complete_callback(pa_pdispatch *pd, uint32_t command, uint32_t tag, pa_tagstruct *t, void *userdata) {
|
||||
pa_context *c = userdata;
|
||||
|
||||
assert(pd);
|
||||
assert(c);
|
||||
assert(c->state == PA_CONTEXT_AUTHORIZING || c->state == PA_CONTEXT_SETTING_NAME);
|
||||
pa_assert(pd);
|
||||
pa_assert(c);
|
||||
pa_assert(c->state == PA_CONTEXT_AUTHORIZING || c->state == PA_CONTEXT_SETTING_NAME);
|
||||
|
||||
pa_context_ref(c);
|
||||
|
||||
|
|
@ -423,7 +424,7 @@ static void setup_complete_callback(pa_pdispatch *pd, uint32_t command, uint32_t
|
|||
break;
|
||||
|
||||
default:
|
||||
assert(0);
|
||||
pa_assert(0);
|
||||
}
|
||||
|
||||
finish:
|
||||
|
|
@ -434,19 +435,19 @@ static void setup_context(pa_context *c, pa_iochannel *io) {
|
|||
pa_tagstruct *t;
|
||||
uint32_t tag;
|
||||
|
||||
assert(c);
|
||||
assert(io);
|
||||
pa_assert(c);
|
||||
pa_assert(io);
|
||||
|
||||
pa_context_ref(c);
|
||||
|
||||
assert(!c->pstream);
|
||||
pa_assert(!c->pstream);
|
||||
c->pstream = pa_pstream_new(c->mainloop, io, c->mempool);
|
||||
|
||||
pa_pstream_set_die_callback(c->pstream, pstream_die_callback, c);
|
||||
pa_pstream_set_recieve_packet_callback(c->pstream, pstream_packet_callback, c);
|
||||
pa_pstream_set_recieve_memblock_callback(c->pstream, pstream_memblock_callback, c);
|
||||
|
||||
assert(!c->pdispatch);
|
||||
pa_assert(!c->pdispatch);
|
||||
c->pdispatch = pa_pdispatch_new(c->mainloop, command_table, PA_COMMAND_MAX);
|
||||
|
||||
if (!c->conf->cookie_valid)
|
||||
|
|
@ -602,8 +603,8 @@ static int try_next_connection(pa_context *c) {
|
|||
char *u = NULL;
|
||||
int r = -1;
|
||||
|
||||
assert(c);
|
||||
assert(!c->client);
|
||||
pa_assert(c);
|
||||
pa_assert(!c->client);
|
||||
|
||||
for (;;) {
|
||||
pa_xfree(u);
|
||||
|
|
@ -648,9 +649,9 @@ finish:
|
|||
static void on_connection(pa_socket_client *client, pa_iochannel*io, void *userdata) {
|
||||
pa_context *c = userdata;
|
||||
|
||||
assert(client);
|
||||
assert(c);
|
||||
assert(c->state == PA_CONTEXT_CONNECTING);
|
||||
pa_assert(client);
|
||||
pa_assert(c);
|
||||
pa_assert(c->state == PA_CONTEXT_CONNECTING);
|
||||
|
||||
pa_context_ref(c);
|
||||
|
||||
|
|
@ -683,8 +684,8 @@ int pa_context_connect(
|
|||
|
||||
int r = -1;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY(c, c->state == PA_CONTEXT_UNCONNECTED, PA_ERR_BADSTATE);
|
||||
PA_CHECK_VALIDITY(c, !(flags & ~PA_CONTEXT_NOAUTOSPAWN), PA_ERR_INVALID);
|
||||
|
|
@ -695,7 +696,7 @@ int pa_context_connect(
|
|||
|
||||
pa_context_ref(c);
|
||||
|
||||
assert(!c->server_list);
|
||||
pa_assert(!c->server_list);
|
||||
|
||||
if (server) {
|
||||
if (!(c->server_list = pa_strlist_parse(server))) {
|
||||
|
|
@ -735,7 +736,7 @@ int pa_context_connect(
|
|||
|
||||
pa_runtime_path(AUTOSPAWN_LOCK, lf, sizeof(lf));
|
||||
pa_make_secure_parent_dir(lf, 0700, (uid_t)-1, (gid_t)-1);
|
||||
assert(c->autospawn_lock_fd <= 0);
|
||||
pa_assert(c->autospawn_lock_fd <= 0);
|
||||
c->autospawn_lock_fd = pa_lock_lockfile(lf);
|
||||
|
||||
if (api)
|
||||
|
|
@ -755,37 +756,37 @@ finish:
|
|||
}
|
||||
|
||||
void pa_context_disconnect(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
pa_context_set_state(c, PA_CONTEXT_TERMINATED);
|
||||
}
|
||||
|
||||
pa_context_state_t pa_context_get_state(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
return c->state;
|
||||
}
|
||||
|
||||
int pa_context_errno(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
return c->error;
|
||||
}
|
||||
|
||||
void pa_context_set_state_callback(pa_context *c, pa_context_notify_cb_t cb, void *userdata) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
c->state_callback = cb;
|
||||
c->state_userdata = userdata;
|
||||
}
|
||||
|
||||
int pa_context_is_pending(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY(c,
|
||||
c->state == PA_CONTEXT_CONNECTING ||
|
||||
|
|
@ -811,11 +812,11 @@ static void pstream_drain_callback(PA_GCC_UNUSED pa_pstream *s, void *userdata)
|
|||
static void set_dispatch_callbacks(pa_operation *o) {
|
||||
int done = 1;
|
||||
|
||||
assert(o);
|
||||
assert(PA_REFCNT_VALUE(o) >= 1);
|
||||
assert(o->context);
|
||||
assert(PA_REFCNT_VALUE(o->context) >= 1);
|
||||
assert(o->context->state == PA_CONTEXT_READY);
|
||||
pa_assert(o);
|
||||
pa_assert(PA_REFCNT_VALUE(o) >= 1);
|
||||
pa_assert(o->context);
|
||||
pa_assert(PA_REFCNT_VALUE(o->context) >= 1);
|
||||
pa_assert(o->context->state == PA_CONTEXT_READY);
|
||||
|
||||
pa_pstream_set_drain_callback(o->context->pstream, NULL, NULL);
|
||||
pa_pdispatch_set_drain_callback(o->context->pdispatch, NULL, NULL);
|
||||
|
|
@ -844,8 +845,8 @@ static void set_dispatch_callbacks(pa_operation *o) {
|
|||
pa_operation* pa_context_drain(pa_context *c, pa_context_notify_cb_t cb, void *userdata) {
|
||||
pa_operation *o;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE);
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, pa_context_is_pending(c), PA_ERR_BADSTATE);
|
||||
|
|
@ -860,9 +861,9 @@ void pa_context_simple_ack_callback(pa_pdispatch *pd, uint32_t command, PA_GCC_U
|
|||
pa_operation *o = userdata;
|
||||
int success = 1;
|
||||
|
||||
assert(pd);
|
||||
assert(o);
|
||||
assert(PA_REFCNT_VALUE(o) >= 1);
|
||||
pa_assert(pd);
|
||||
pa_assert(o);
|
||||
pa_assert(PA_REFCNT_VALUE(o) >= 1);
|
||||
|
||||
if (!o->context)
|
||||
goto finish;
|
||||
|
|
@ -892,8 +893,8 @@ pa_operation* pa_context_exit_daemon(pa_context *c, pa_context_success_cb_t cb,
|
|||
pa_operation *o;
|
||||
uint32_t tag;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE);
|
||||
|
||||
|
|
@ -911,8 +912,8 @@ pa_operation* pa_context_send_simple_command(pa_context *c, uint32_t command, pa
|
|||
pa_operation *o;
|
||||
uint32_t tag;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE);
|
||||
|
||||
|
|
@ -930,8 +931,8 @@ pa_operation* pa_context_set_default_sink(pa_context *c, const char *name, pa_co
|
|||
pa_operation *o;
|
||||
uint32_t tag;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE);
|
||||
|
||||
|
|
@ -950,8 +951,8 @@ pa_operation* pa_context_set_default_source(pa_context *c, const char *name, pa_
|
|||
pa_operation *o;
|
||||
uint32_t tag;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE);
|
||||
|
||||
|
|
@ -966,7 +967,7 @@ pa_operation* pa_context_set_default_source(pa_context *c, const char *name, pa_
|
|||
}
|
||||
|
||||
int pa_context_is_local(pa_context *c) {
|
||||
assert(c);
|
||||
pa_assert(c);
|
||||
|
||||
return c->is_local;
|
||||
}
|
||||
|
|
@ -976,9 +977,9 @@ pa_operation* pa_context_set_name(pa_context *c, const char *name, pa_context_su
|
|||
pa_operation *o;
|
||||
uint32_t tag;
|
||||
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
assert(name);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(name);
|
||||
|
||||
PA_CHECK_VALIDITY_RETURN_NULL(c, c->state == PA_CONTEXT_READY, PA_ERR_BADSTATE);
|
||||
|
||||
|
|
@ -997,8 +998,8 @@ const char* pa_get_library_version(void) {
|
|||
}
|
||||
|
||||
const char* pa_context_get_server(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
if (!c->server)
|
||||
return NULL;
|
||||
|
|
@ -1016,8 +1017,8 @@ uint32_t pa_context_get_protocol_version(PA_GCC_UNUSED pa_context *c) {
|
|||
}
|
||||
|
||||
uint32_t pa_context_get_server_protocol_version(pa_context *c) {
|
||||
assert(c);
|
||||
assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
pa_assert(c);
|
||||
pa_assert(PA_REFCNT_VALUE(c) >= 1);
|
||||
|
||||
return c->version;
|
||||
}
|
||||
|
|
@ -1025,8 +1026,8 @@ uint32_t pa_context_get_server_protocol_version(pa_context *c) {
|
|||
pa_tagstruct *pa_tagstruct_command(pa_context *c, uint32_t command, uint32_t *tag) {
|
||||
pa_tagstruct *t;
|
||||
|
||||
assert(c);
|
||||
assert(tag);
|
||||
pa_assert(c);
|
||||
pa_assert(tag);
|
||||
|
||||
t = pa_tagstruct_new(NULL, 0);
|
||||
pa_tagstruct_putu32(t, command);
|
||||
|
|
|
|||
|
|
@ -161,7 +161,8 @@ pa_signal_event* pa_signal_new(int sig, void (*_callback) (pa_mainloop_api *api,
|
|||
struct sigaction sa;
|
||||
#endif
|
||||
|
||||
pa_assert(sig > 0 && _callback);
|
||||
pa_assert(sig > 0);
|
||||
pa_assert(_callback);
|
||||
|
||||
for (e = signals; e; e = e->next)
|
||||
if (e->sig == sig)
|
||||
|
|
|
|||
|
|
@ -677,7 +677,8 @@ static int dispatch_pollfds(pa_mainloop *m) {
|
|||
if (e->dead || !e->pollfd || !e->pollfd->revents)
|
||||
continue;
|
||||
|
||||
pa_assert(e->pollfd->fd == e->fd && e->callback);
|
||||
pa_assert(e->pollfd->fd == e->fd);
|
||||
pa_assert(e->callback);
|
||||
e->callback(&m->api, e, e->fd, map_flags_from_libc(e->pollfd->revents), e->userdata);
|
||||
e->pollfd->revents = 0;
|
||||
r++;
|
||||
|
|
|
|||
|
|
@ -1,3 +1,4 @@
|
|||
|
||||
/* $Id$ */
|
||||
|
||||
/***
|
||||
|
|
@ -27,7 +28,6 @@
|
|||
|
||||
#include <stdio.h>
|
||||
#include <string.h>
|
||||
#include <assert.h>
|
||||
#include <stdlib.h>
|
||||
|
||||
#include <pulse/pulseaudio.h>
|
||||
|
|
@ -36,6 +36,7 @@
|
|||
|
||||
#include <pulsecore/native-common.h>
|
||||
#include <pulsecore/log.h>
|
||||
#include <pulsecore/macro.h>
|
||||
|
||||
#include "simple.h"
|
||||
|
||||
|
|
@ -83,8 +84,8 @@ if (!(p)->context || pa_context_get_state((p)->context) != PA_CONTEXT_READY || \
|
|||
|
||||
static void context_state_cb(pa_context *c, void *userdata) {
|
||||
pa_simple *p = userdata;
|
||||
assert(c);
|
||||
assert(p);
|
||||
pa_assert(c);
|
||||
pa_assert(p);
|
||||
|
||||
switch (pa_context_get_state(c)) {
|
||||
case PA_CONTEXT_READY:
|
||||
|
|
@ -103,8 +104,8 @@ static void context_state_cb(pa_context *c, void *userdata) {
|
|||
|
||||
static void stream_state_cb(pa_stream *s, void * userdata) {
|
||||
pa_simple *p = userdata;
|
||||
assert(s);
|
||||
assert(p);
|
||||
pa_assert(s);
|
||||
pa_assert(p);
|
||||
|
||||
switch (pa_stream_get_state(s)) {
|
||||
|
||||
|
|
@ -122,7 +123,7 @@ static void stream_state_cb(pa_stream *s, void * userdata) {
|
|||
|
||||
static void stream_request_cb(pa_stream *s, size_t length, void *userdata) {
|
||||
pa_simple *p = userdata;
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
pa_threaded_mainloop_signal(p->mainloop, 0);
|
||||
}
|
||||
|
|
@ -130,21 +131,21 @@ static void stream_request_cb(pa_stream *s, size_t length, void *userdata) {
|
|||
static void stream_latency_update_cb(pa_stream *s, void *userdata) {
|
||||
pa_simple *p = userdata;
|
||||
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
pa_threaded_mainloop_signal(p->mainloop, 0);
|
||||
}
|
||||
|
||||
pa_simple* pa_simple_new(
|
||||
const char *server,
|
||||
const char *name,
|
||||
pa_stream_direction_t dir,
|
||||
const char *dev,
|
||||
const char *stream_name,
|
||||
const pa_sample_spec *ss,
|
||||
const pa_channel_map *map,
|
||||
const pa_buffer_attr *attr,
|
||||
int *rerror) {
|
||||
const char *server,
|
||||
const char *name,
|
||||
pa_stream_direction_t dir,
|
||||
const char *dev,
|
||||
const char *stream_name,
|
||||
const pa_sample_spec *ss,
|
||||
const pa_channel_map *map,
|
||||
const pa_buffer_attr *attr,
|
||||
int *rerror) {
|
||||
|
||||
pa_simple *p;
|
||||
int error = PA_ERR_INTERNAL, r;
|
||||
|
|
@ -232,7 +233,7 @@ fail:
|
|||
}
|
||||
|
||||
void pa_simple_free(pa_simple *s) {
|
||||
assert(s);
|
||||
pa_assert(s);
|
||||
|
||||
if (s->mainloop)
|
||||
pa_threaded_mainloop_stop(s->mainloop);
|
||||
|
|
@ -250,7 +251,7 @@ void pa_simple_free(pa_simple *s) {
|
|||
}
|
||||
|
||||
int pa_simple_write(pa_simple *p, const void*data, size_t length, int *rerror) {
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
CHECK_VALIDITY_RETURN_ANY(rerror, p->direction == PA_STREAM_PLAYBACK, PA_ERR_BADSTATE, -1);
|
||||
CHECK_VALIDITY_RETURN_ANY(rerror, data && length, PA_ERR_INVALID, -1);
|
||||
|
|
@ -289,7 +290,7 @@ unlock_and_fail:
|
|||
}
|
||||
|
||||
int pa_simple_read(pa_simple *p, void*data, size_t length, int *rerror) {
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
CHECK_VALIDITY_RETURN_ANY(rerror, p->direction == PA_STREAM_RECORD, PA_ERR_BADSTATE, -1);
|
||||
CHECK_VALIDITY_RETURN_ANY(rerror, data && length, PA_ERR_INVALID, -1);
|
||||
|
|
@ -346,8 +347,8 @@ unlock_and_fail:
|
|||
static void success_cb(pa_stream *s, int success, void *userdata) {
|
||||
pa_simple *p = userdata;
|
||||
|
||||
assert(s);
|
||||
assert(p);
|
||||
pa_assert(s);
|
||||
pa_assert(p);
|
||||
|
||||
p->operation_success = success;
|
||||
pa_threaded_mainloop_signal(p->mainloop, 0);
|
||||
|
|
@ -356,7 +357,7 @@ static void success_cb(pa_stream *s, int success, void *userdata) {
|
|||
int pa_simple_drain(pa_simple *p, int *rerror) {
|
||||
pa_operation *o = NULL;
|
||||
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
CHECK_VALIDITY_RETURN_ANY(rerror, p->direction == PA_STREAM_PLAYBACK, PA_ERR_BADSTATE, -1);
|
||||
|
||||
|
|
@ -392,7 +393,7 @@ unlock_and_fail:
|
|||
int pa_simple_flush(pa_simple *p, int *rerror) {
|
||||
pa_operation *o = NULL;
|
||||
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
CHECK_VALIDITY_RETURN_ANY(rerror, p->direction == PA_STREAM_PLAYBACK, PA_ERR_BADSTATE, -1);
|
||||
|
||||
|
|
@ -429,7 +430,7 @@ pa_usec_t pa_simple_get_latency(pa_simple *p, int *rerror) {
|
|||
pa_usec_t t;
|
||||
int negative;
|
||||
|
||||
assert(p);
|
||||
pa_assert(p);
|
||||
|
||||
pa_threaded_mainloop_lock(p->mainloop);
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue