aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2014-08-26 10:04:27 +0200
committerMax Kellermann <max@duempel.org>2014-08-26 10:04:27 +0200
commitb111aa0111565c4d6f1d7e56a16a34c5287a5145 (patch)
tree9bc0db9dab9bbb612bb20b26385af7277d7accbd
parent773d24ebf722822b027c3c61a5f7ae846ac6725d (diff)
downloadmpd-b111aa0111565c4d6f1d7e56a16a34c5287a5145.tar.gz
mpd-b111aa0111565c4d6f1d7e56a16a34c5287a5145.tar.xz
mpd-b111aa0111565c4d6f1d7e56a16a34c5287a5145.zip
ZeroconfAvahi: fix coding style
-rw-r--r--src/zeroconf/ZeroconfAvahi.cxx161
1 files changed, 87 insertions, 74 deletions
diff --git a/src/zeroconf/ZeroconfAvahi.cxx b/src/zeroconf/ZeroconfAvahi.cxx
index 9643802ae..5adda38f9 100644
--- a/src/zeroconf/ZeroconfAvahi.cxx
+++ b/src/zeroconf/ZeroconfAvahi.cxx
@@ -38,20 +38,23 @@
static constexpr Domain avahi_domain("avahi");
-static char *avahiName;
+static char *avahi_name;
static MyAvahiPoll *avahi_poll;
-static AvahiClient *avahiClient;
-static AvahiEntryGroup *avahiGroup;
+static AvahiClient *avahi_client;
+static AvahiEntryGroup *avahi_group;
-static void avahiRegisterService(AvahiClient * c);
+static void
+AvahiRegisterService(AvahiClient *c);
-/* Callback when the EntryGroup changes state */
-static void avahiGroupCallback(AvahiEntryGroup * g,
- AvahiEntryGroupState state,
- gcc_unused void *userdata)
+/**
+ * Callback when the EntryGroup changes state.
+ */
+static void
+AvahiGroupCallback(AvahiEntryGroup *g,
+ AvahiEntryGroupState state,
+ gcc_unused void *userdata)
{
- char *n;
- assert(g);
+ assert(g != nullptr);
FormatDebug(avahi_domain,
"Service group changed to state %d", state);
@@ -61,53 +64,58 @@ static void avahiGroupCallback(AvahiEntryGroup * g,
/* The entry group has been established successfully */
FormatDefault(avahi_domain,
"Service '%s' successfully established.",
- avahiName);
+ avahi_name);
break;
case AVAHI_ENTRY_GROUP_COLLISION:
/* A service name collision happened. Let's pick a new name */
- n = avahi_alternative_service_name(avahiName);
- avahi_free(avahiName);
- avahiName = n;
+ {
+ char *n = avahi_alternative_service_name(avahi_name);
+ avahi_free(avahi_name);
+ avahi_name = n;
+ }
FormatDefault(avahi_domain,
"Service name collision, renaming service to '%s'",
- avahiName);
+ avahi_name);
/* And recreate the services */
- avahiRegisterService(avahi_entry_group_get_client(g));
+ AvahiRegisterService(avahi_entry_group_get_client(g));
break;
case AVAHI_ENTRY_GROUP_FAILURE:
FormatError(avahi_domain,
"Entry group failure: %s",
- avahi_strerror(avahi_client_errno
- (avahi_entry_group_get_client(g))));
- /* Some kind of failure happened while we were registering our services */
+ avahi_strerror(avahi_client_errno(avahi_entry_group_get_client(g))));
+ /* Some kind of failure happened while we were
+ registering our services */
break;
case AVAHI_ENTRY_GROUP_UNCOMMITED:
LogDebug(avahi_domain, "Service group is UNCOMMITED");
break;
+
case AVAHI_ENTRY_GROUP_REGISTERING:
LogDebug(avahi_domain, "Service group is REGISTERING");
}
}
-/* Registers a new service with avahi */
-static void avahiRegisterService(AvahiClient * c)
+/**
+ * Registers a new service with avahi.
+ */
+static void
+AvahiRegisterService(AvahiClient *c)
{
- FormatDebug(avahi_domain, "Registering service %s/%s",
- SERVICE_TYPE, avahiName);
+ assert(c != nullptr);
- int ret;
- assert(c);
+ FormatDebug(avahi_domain, "Registering service %s/%s",
+ SERVICE_TYPE, avahi_name);
/* If this is the first time we're called,
* let's create a new entry group */
- if (!avahiGroup) {
- avahiGroup = avahi_entry_group_new(c, avahiGroupCallback, nullptr);
- if (!avahiGroup) {
+ if (!avahi_group) {
+ avahi_group = avahi_entry_group_new(c, AvahiGroupCallback, nullptr);
+ if (!avahi_group) {
FormatError(avahi_domain,
"Failed to create avahi EntryGroup: %s",
avahi_strerror(avahi_client_errno(c)));
@@ -119,44 +127,48 @@ static void avahiRegisterService(AvahiClient * c)
/* TODO: This currently binds to ALL interfaces.
* We could maybe add a service per actual bound interface,
* if that's better. */
- ret = avahi_entry_group_add_service(avahiGroup,
- AVAHI_IF_UNSPEC, AVAHI_PROTO_UNSPEC,
- AvahiPublishFlags(0),
- avahiName, SERVICE_TYPE, nullptr,
- nullptr, listen_port, nullptr);
- if (ret < 0) {
+ int result = avahi_entry_group_add_service(avahi_group,
+ AVAHI_IF_UNSPEC,
+ AVAHI_PROTO_UNSPEC,
+ AvahiPublishFlags(0),
+ avahi_name, SERVICE_TYPE,
+ nullptr, nullptr,
+ listen_port, nullptr);
+ if (result < 0) {
FormatError(avahi_domain, "Failed to add service %s: %s",
- SERVICE_TYPE, avahi_strerror(ret));
+ SERVICE_TYPE, avahi_strerror(result));
return;
}
/* Tell the server to register the service group */
- ret = avahi_entry_group_commit(avahiGroup);
- if (ret < 0) {
+ result = avahi_entry_group_commit(avahi_group);
+ if (result < 0) {
FormatError(avahi_domain, "Failed to commit service group: %s",
- avahi_strerror(ret));
+ avahi_strerror(result));
return;
}
}
/* Callback when avahi changes state */
-static void avahiClientCallback(AvahiClient * c, AvahiClientState state,
- gcc_unused void *userdata)
+static void
+MyAvahiClientCallback(AvahiClient *c, AvahiClientState state,
+ gcc_unused void *userdata)
{
- int reason;
- assert(c);
+ assert(c != nullptr);
/* Called whenever the client or server state changes */
FormatDebug(avahi_domain, "Client changed to state %d", state);
switch (state) {
+ int reason;
+
case AVAHI_CLIENT_S_RUNNING:
LogDebug(avahi_domain, "Client is RUNNING");
/* The server has startup successfully and registered its host
* name on the network, so it's time to create our services */
- if (!avahiGroup)
- avahiRegisterService(c);
+ if (avahi_group == nullptr)
+ AvahiRegisterService(c);
break;
case AVAHI_CLIENT_FAILURE:
@@ -164,38 +176,39 @@ static void avahiClientCallback(AvahiClient * c, AvahiClientState state,
if (reason == AVAHI_ERR_DISCONNECTED) {
LogDefault(avahi_domain,
"Client Disconnected, will reconnect shortly");
- if (avahiGroup) {
- avahi_entry_group_free(avahiGroup);
- avahiGroup = nullptr;
+ if (avahi_group != nullptr) {
+ avahi_entry_group_free(avahi_group);
+ avahi_group = nullptr;
}
- if (avahiClient)
- avahi_client_free(avahiClient);
- avahiClient =
+
+ if (avahi_client != nullptr)
+ avahi_client_free(avahi_client);
+ avahi_client =
avahi_client_new(avahi_poll,
AVAHI_CLIENT_NO_FAIL,
- avahiClientCallback, nullptr,
+ MyAvahiClientCallback, nullptr,
&reason);
- if (!avahiClient) {
+ if (avahi_client == nullptr)
FormatWarning(avahi_domain,
"Could not reconnect: %s",
avahi_strerror(reason));
- }
} else {
FormatWarning(avahi_domain,
"Client failure: %s (terminal)",
avahi_strerror(reason));
}
+
break;
case AVAHI_CLIENT_S_COLLISION:
LogDebug(avahi_domain, "Client is COLLISION");
- /* Let's drop our registered services. When the server is back
- * in AVAHI_SERVER_RUNNING state we will register them
- * again with the new host name. */
- if (avahiGroup) {
+ /* Let's drop our registered services. When the server
+ is back in AVAHI_SERVER_RUNNING state we will
+ register them again with the new host name. */
+ if (avahi_group != nullptr) {
LogDebug(avahi_domain, "Resetting group");
- avahi_entry_group_reset(avahiGroup);
+ avahi_entry_group_reset(avahi_group);
}
break;
@@ -208,9 +221,9 @@ static void avahiClientCallback(AvahiClient * c, AvahiClientState state,
* for our own records to register until the host name is
* properly esatblished. */
- if (avahiGroup) {
+ if (avahi_group != nullptr) {
LogDebug(avahi_domain, "Resetting group");
- avahi_entry_group_reset(avahiGroup);
+ avahi_entry_group_reset(avahi_group);
}
break;
@@ -229,15 +242,15 @@ AvahiInit(EventLoop &loop, const char *serviceName)
if (!avahi_is_valid_service_name(serviceName))
FormatFatalError("Invalid zeroconf_name \"%s\"", serviceName);
- avahiName = avahi_strdup(serviceName);
+ avahi_name = avahi_strdup(serviceName);
avahi_poll = new MyAvahiPoll(loop);
int error;
- avahiClient = avahi_client_new(avahi_poll, AVAHI_CLIENT_NO_FAIL,
- avahiClientCallback, nullptr, &error);
-
- if (!avahiClient) {
+ avahi_client = avahi_client_new(avahi_poll, AVAHI_CLIENT_NO_FAIL,
+ MyAvahiClientCallback, nullptr,
+ &error);
+ if (avahi_client == nullptr) {
FormatError(avahi_domain, "Failed to create client: %s",
avahi_strerror(error));
AvahiDeinit();
@@ -245,25 +258,25 @@ AvahiInit(EventLoop &loop, const char *serviceName)
}
void
-AvahiDeinit(void)
+AvahiDeinit()
{
LogDebug(avahi_domain, "Shutting down interface");
- if (avahiGroup) {
- avahi_entry_group_free(avahiGroup);
- avahiGroup = nullptr;
+ if (avahi_group != nullptr) {
+ avahi_entry_group_free(avahi_group);
+ avahi_group = nullptr;
}
- if (avahiClient) {
- avahi_client_free(avahiClient);
- avahiClient = nullptr;
+ if (avahi_client != nullptr) {
+ avahi_client_free(avahi_client);
+ avahi_client = nullptr;
}
delete avahi_poll;
avahi_poll = nullptr;
- avahi_free(avahiName);
- avahiName = nullptr;
+ avahi_free(avahi_name);
+ avahi_name = nullptr;
dbus_shutdown();
}