[Asterisk-code-review] sorcery.c: Minor optimizations. (asterisk[master])
Joshua Colp
asteriskteam at digium.com
Tue Aug 16 08:24:31 CDT 2016
Joshua Colp has submitted this change and it was merged.
Change subject: sorcery.c: Minor optimizations.
......................................................................
sorcery.c: Minor optimizations.
* Remove some unused parameters from internal functions:
sorcery_wizard_create()
sorcery_wizard_update()
sorcery_wizard_delete()
* Created the struct sorcery_observer_invocation ao2 object without a lock
since it is not needed in sorcery_observer_invocation_alloc().
* Cleanup generic ao2 container sorcery object id hash, sort, and cmp
functions.
Change-Id: Iff71d75f52bc1b8cee955456838c149faaa4f92e
---
M main/sorcery.c
1 file changed, 50 insertions(+), 57 deletions(-)
Approvals:
Mark Michelson: Looks good to me, but someone else must approve
Joshua Colp: Looks good to me, approved; Verified
diff --git a/main/sorcery.c b/main/sorcery.c
index 1fb1b3c..55ee830 100644
--- a/main/sorcery.c
+++ b/main/sorcery.c
@@ -608,7 +608,7 @@
{
const struct sorcery_global_observer *observer = obj;
- return (observer->callbacks == arg) ? CMP_MATCH | CMP_STOP : 0;
+ return (observer->callbacks == arg) ? CMP_MATCH : 0;
}
int ast_sorcery_global_observer_add(const struct ast_sorcery_global_observer *callbacks)
@@ -1352,8 +1352,10 @@
/*! \brief Allocator function for observer invocation */
static struct sorcery_observer_invocation *sorcery_observer_invocation_alloc(struct ast_sorcery_object_type *object_type, void *object)
{
- struct sorcery_observer_invocation *invocation = ao2_alloc(sizeof(*invocation), sorcery_observer_invocation_destroy);
+ struct sorcery_observer_invocation *invocation;
+ invocation = ao2_alloc_options(sizeof(*invocation),
+ sorcery_observer_invocation_destroy, AO2_ALLOC_OPT_LOCK_NOLOCK);
if (!invocation) {
return NULL;
}
@@ -1978,11 +1980,8 @@
}
/*! \brief Internal function which returns if the wizard has created the object */
-static int sorcery_wizard_create(void *obj, void *arg, int flags)
+static int sorcery_wizard_create(const struct ast_sorcery_object_wizard *object_wizard, const struct sorcery_details *details)
{
- const struct ast_sorcery_object_wizard *object_wizard = obj;
- const struct sorcery_details *details = arg;
-
if (!object_wizard->wizard->callbacks.create) {
ast_debug(5, "Sorcery wizard '%s' does not support creation\n", object_wizard->wizard->callbacks.name);
return 0;
@@ -2037,7 +2036,8 @@
AST_VECTOR_RW_RDLOCK(&object_type->wizards);
for (i = 0; i < AST_VECTOR_SIZE(&object_type->wizards); i++) {
found_wizard = AST_VECTOR_GET(&object_type->wizards, i);
- if (!found_wizard->caching && sorcery_wizard_create(found_wizard, &sdetails, 0) == CMP_MATCH) {
+ if (!found_wizard->caching
+ && sorcery_wizard_create(found_wizard, &sdetails) == CMP_MATCH) {
object_wizard = found_wizard;
}
}
@@ -2046,14 +2046,14 @@
for (i = 0; i < AST_VECTOR_SIZE(&object_type->wizards); i++) {
found_wizard = AST_VECTOR_GET(&object_type->wizards, i);
if (found_wizard->caching) {
- sorcery_wizard_create(found_wizard, &sdetails, 0);
+ sorcery_wizard_create(found_wizard, &sdetails);
}
}
if (ao2_container_count(object_type->observers)) {
- struct sorcery_observer_invocation *invocation = sorcery_observer_invocation_alloc(
- object_type, object);
+ struct sorcery_observer_invocation *invocation;
+ invocation = sorcery_observer_invocation_alloc(object_type, object);
if (invocation
&& ast_taskprocessor_push(object_type->serializer, sorcery_observers_notify_create,
invocation)) {
@@ -2091,11 +2091,8 @@
}
/*! \brief Internal function which returns if a wizard has updated the object */
-static int sorcery_wizard_update(void *obj, void *arg, int flags)
+static int sorcery_wizard_update(const struct ast_sorcery_object_wizard *object_wizard, const struct sorcery_details *details)
{
- const struct ast_sorcery_object_wizard *object_wizard = obj;
- const struct sorcery_details *details = arg;
-
if (!object_wizard->wizard->callbacks.update) {
ast_debug(5, "Sorcery wizard '%s' does not support updating\n", object_wizard->wizard->callbacks.name);
return 0;
@@ -2127,7 +2124,8 @@
AST_VECTOR_RW_RDLOCK(&object_type->wizards);
for (i = 0; i < AST_VECTOR_SIZE(&object_type->wizards); i++) {
found_wizard = AST_VECTOR_GET(&object_type->wizards, i);
- if (!found_wizard->caching && sorcery_wizard_update(found_wizard, &sdetails, 0) == CMP_MATCH) {
+ if (!found_wizard->caching
+ && sorcery_wizard_update(found_wizard, &sdetails) == CMP_MATCH) {
object_wizard = found_wizard;
}
}
@@ -2136,14 +2134,14 @@
for (i = 0; i < AST_VECTOR_SIZE(&object_type->wizards); i++) {
found_wizard = AST_VECTOR_GET(&object_type->wizards, i);
if (found_wizard->caching) {
- sorcery_wizard_update(found_wizard, &sdetails, 0);
+ sorcery_wizard_update(found_wizard, &sdetails);
}
}
if (ao2_container_count(object_type->observers)) {
- struct sorcery_observer_invocation *invocation = sorcery_observer_invocation_alloc(
- object_type, object);
+ struct sorcery_observer_invocation *invocation;
+ invocation = sorcery_observer_invocation_alloc(object_type, object);
if (invocation
&& ast_taskprocessor_push(object_type->serializer, sorcery_observers_notify_update,
invocation)) {
@@ -2181,11 +2179,8 @@
}
/*! \brief Internal function which returns if a wizard has deleted the object */
-static int sorcery_wizard_delete(void *obj, void *arg, int flags)
+static int sorcery_wizard_delete(const struct ast_sorcery_object_wizard *object_wizard, const struct sorcery_details *details)
{
- const struct ast_sorcery_object_wizard *object_wizard = obj;
- const struct sorcery_details *details = arg;
-
if (!object_wizard->wizard->callbacks.delete) {
ast_debug(5, "Sorcery wizard '%s' does not support deletion\n", object_wizard->wizard->callbacks.name);
return 0;
@@ -2217,7 +2212,8 @@
AST_VECTOR_RW_RDLOCK(&object_type->wizards);
for (i = 0; i < AST_VECTOR_SIZE(&object_type->wizards); i++) {
found_wizard = AST_VECTOR_GET(&object_type->wizards, i);
- if (!found_wizard->caching && sorcery_wizard_delete(found_wizard, &sdetails, 0) == CMP_MATCH) {
+ if (!found_wizard->caching
+ && sorcery_wizard_delete(found_wizard, &sdetails) == CMP_MATCH) {
object_wizard = found_wizard;
}
}
@@ -2226,14 +2222,14 @@
for (i = 0; i < AST_VECTOR_SIZE(&object_type->wizards); i++) {
found_wizard = AST_VECTOR_GET(&object_type->wizards, i);
if (found_wizard->caching) {
- sorcery_wizard_delete(found_wizard, &sdetails, 0);
+ sorcery_wizard_delete(found_wizard, &sdetails);
}
}
if (ao2_container_count(object_type->observers)) {
- struct sorcery_observer_invocation *invocation = sorcery_observer_invocation_alloc(
- object_type, object);
+ struct sorcery_observer_invocation *invocation;
+ invocation = sorcery_observer_invocation_alloc(object_type, object);
if (invocation
&& ast_taskprocessor_push(object_type->serializer, sorcery_observers_notify_delete,
invocation)) {
@@ -2377,7 +2373,7 @@
{
const struct ast_sorcery_object_type_observer *observer = obj;
- return (observer->callbacks == arg) ? CMP_MATCH | CMP_STOP : 0;
+ return (observer->callbacks == arg) ? CMP_MATCH : 0;
}
void ast_sorcery_observer_remove(const struct ast_sorcery *sorcery, const char *type, const struct ast_sorcery_observer *callbacks)
@@ -2399,18 +2395,20 @@
int ast_sorcery_object_id_sort(const void *obj, const void *arg, int flags)
{
+ const void *object_left = obj;
+ const void *object_right = arg;
const char *right_key = arg;
int cmp;
switch (flags & OBJ_SEARCH_MASK) {
case OBJ_SEARCH_OBJECT:
- right_key = ast_sorcery_object_get_id(arg);
+ right_key = ast_sorcery_object_get_id(object_right);
/* Fall through */
case OBJ_SEARCH_KEY:
- cmp = strcmp(ast_sorcery_object_get_id(obj), right_key);
+ cmp = strcmp(ast_sorcery_object_get_id(object_left), right_key);
break;
case OBJ_SEARCH_PARTIAL_KEY:
- cmp = strncmp(ast_sorcery_object_get_id(obj), right_key, strlen(right_key));
+ cmp = strncmp(ast_sorcery_object_get_id(object_left), right_key, strlen(right_key));
break;
default:
cmp = 0;
@@ -2421,37 +2419,32 @@
int ast_sorcery_object_id_compare(void *obj, void *arg, int flags)
{
- const char *right_key = arg;
- int cmp = 0;
+ int cmp;
- switch (flags & OBJ_SEARCH_MASK) {
- case OBJ_SEARCH_OBJECT:
- right_key = ast_sorcery_object_get_id(arg);
- /* Fall through */
- case OBJ_SEARCH_KEY:
- if (strcmp(ast_sorcery_object_get_id(obj), right_key) == 0) {
- cmp = CMP_MATCH | CMP_STOP;
- }
- break;
- case OBJ_SEARCH_PARTIAL_KEY:
- if (strncmp(ast_sorcery_object_get_id(obj), right_key, strlen(right_key)) == 0) {
- cmp = CMP_MATCH;
- }
- break;
- default:
- cmp = 0;
- break;
+ cmp = ast_sorcery_object_id_sort(obj, arg, flags);
+ if (cmp) {
+ return 0;
}
- return cmp;
+ return CMP_MATCH;
}
-int ast_sorcery_object_id_hash(const void *obj, int flags) {
- if (flags & OBJ_SEARCH_OBJECT) {
- return ast_str_hash(ast_sorcery_object_get_id(obj));
- } else if (flags & OBJ_SEARCH_KEY) {
- return ast_str_hash(obj);
+int ast_sorcery_object_id_hash(const void *obj, int flags)
+{
+ const char *key;
+
+ switch (flags & OBJ_SEARCH_MASK) {
+ case OBJ_SEARCH_KEY:
+ key = obj;
+ break;
+ case OBJ_SEARCH_OBJECT:
+ key = ast_sorcery_object_get_id(obj);
+ break;
+ default:
+ /* Hash can only work on something with a full key. */
+ ast_assert(0);
+ return 0;
}
- return -1;
+ return ast_str_hash(key);
}
struct ast_sorcery_object_type *ast_sorcery_get_object_type(const struct ast_sorcery *sorcery,
@@ -2468,7 +2461,7 @@
if (object_field->name_regex
&& !regexec(object_field->name_regex, name, 0, NULL, 0)) {
- rc = CMP_MATCH | CMP_STOP;
+ rc = CMP_MATCH;
}
return rc;
--
To view, visit https://gerrit.asterisk.org/3499
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iff71d75f52bc1b8cee955456838c149faaa4f92e
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
More information about the asterisk-code-review
mailing list