[asterisk-commits] file: trunk r60363 - in /trunk:
include/asterisk/speech.h res/res_speech.c
asterisk-commits at lists.digium.com
asterisk-commits at lists.digium.com
Thu Apr 5 18:29:28 MST 2007
Author: file
Date: Thu Apr 5 20:29:28 2007
New Revision: 60363
URL: http://svn.digium.com/view/asterisk?view=rev&rev=60363
Log:
Major res_speech cleanup. It looks much better now!
Modified:
trunk/include/asterisk/speech.h
trunk/res/res_speech.c
Modified: trunk/include/asterisk/speech.h
URL: http://svn.digium.com/view/asterisk/trunk/include/asterisk/speech.h?view=diff&rev=60363&r1=60362&r2=60363
==============================================================================
--- trunk/include/asterisk/speech.h (original)
+++ trunk/include/asterisk/speech.h Thu Apr 5 20:29:28 2007
@@ -28,14 +28,18 @@
#endif
/* Speech structure flags */
-#define AST_SPEECH_QUIET (1 << 0) /* Quiet down output... they are talking */
-#define AST_SPEECH_SPOKE (1 << 1) /* Speaker did not speak */
+enum ast_speech_flags {
+ AST_SPEECH_QUIET = (1 << 0), /* Quiet down output... they are talking */
+ AST_SPEECH_SPOKE = (1 << 1), /* Speaker spoke! */
+};
/* Speech structure states - in order of expected change */
-#define AST_SPEECH_STATE_NOT_READY 0 /* Not ready to accept audio */
-#define AST_SPEECH_STATE_READY 1 /* Accepting audio */
-#define AST_SPEECH_STATE_WAIT 2 /* Wait for results to become available */
-#define AST_SPEECH_STATE_DONE 3 /* Processing is done */
+enum ast_speech_states {
+ AST_SPEECH_STATE_NOT_READY = 0, /* Not ready to accept audio */
+ AST_SPEECH_STATE_READY, /* Accepting audio */
+ AST_SPEECH_STATE_WAIT, /* Wait for results to become available */
+ AST_SPEECH_STATE_DONE, /* Processing is all done */
+};
enum ast_speech_results_type {
AST_SPEECH_RESULTS_TYPE_NORMAL = 0,
Modified: trunk/res/res_speech.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_speech.c?view=diff&rev=60363&r1=60362&r2=60363
==============================================================================
--- trunk/res/res_speech.c (original)
+++ trunk/res/res_speech.c Thu Apr 5 20:29:28 2007
@@ -42,7 +42,7 @@
#include "asterisk/speech.h"
-static AST_LIST_HEAD_STATIC(engines, ast_speech_engine);
+static AST_RWLIST_HEAD_STATIC(engines, ast_speech_engine);
static struct ast_speech_engine *default_engine = NULL;
/*! \brief Find a speech recognition engine of specified name, if NULL then use the default one */
@@ -51,18 +51,16 @@
struct ast_speech_engine *engine = NULL;
/* If no name is specified -- use the default engine */
- if (engine_name == NULL || strlen(engine_name) == 0) {
+ if (ast_strlen_zero(engine_name))
return default_engine;
- }
-
- AST_LIST_LOCK(&engines);
- AST_LIST_TRAVERSE_SAFE_BEGIN(&engines, engine, list) {
+
+ AST_RWLIST_RDLOCK(&engines);
+ AST_RWLIST_TRAVERSE(&engines, engine, list) {
if (!strcasecmp(engine->name, engine_name)) {
break;
}
}
- AST_LIST_TRAVERSE_SAFE_END
- AST_LIST_UNLOCK(&engines);
+ AST_RWLIST_UNLOCK(&engines);
return engine;
}
@@ -70,61 +68,31 @@
/*! \brief Activate a loaded (either local or global) grammar */
int ast_speech_grammar_activate(struct ast_speech *speech, char *grammar_name)
{
- int res = 0;
-
- if (speech->engine->activate != NULL) {
- res = speech->engine->activate(speech, grammar_name);
- }
-
- return res;
+ return (speech->engine->activate ? speech->engine->activate(speech, grammar_name) : -1);
}
/*! \brief Deactivate a loaded grammar on a speech structure */
int ast_speech_grammar_deactivate(struct ast_speech *speech, char *grammar_name)
{
- int res = 0;
-
- if (speech->engine->deactivate != NULL) {
- res = speech->engine->deactivate(speech, grammar_name);
- }
-
- return res;
+ return (speech->engine->deactivate ? speech->engine->deactivate(speech, grammar_name) : -1);
}
/*! \brief Load a local grammar on a speech structure */
int ast_speech_grammar_load(struct ast_speech *speech, char *grammar_name, char *grammar)
{
- int res = 0;
-
- if (speech->engine->load != NULL) {
- res = speech->engine->load(speech, grammar_name, grammar);
- }
-
- return res;
+ return (speech->engine->load ? speech->engine->load(speech, grammar_name, grammar) : -1);
}
/*! \brief Unload a local grammar from a speech structure */
int ast_speech_grammar_unload(struct ast_speech *speech, char *grammar_name)
{
- int res = 0;
-
- if (speech->engine->unload != NULL) {
- res = speech->engine->unload(speech, grammar_name);
- }
-
- return res;
+ return (speech->engine->unload ? speech->engine->unload(speech, grammar_name) : -1);
}
/*! \brief Return the results of a recognition from the speech structure */
struct ast_speech_result *ast_speech_results_get(struct ast_speech *speech)
{
- struct ast_speech_result *result = NULL;
-
- if (speech->engine->get != NULL) {
- result = speech->engine->get(speech);
- }
-
- return result;
+ return (speech->engine->get ? speech->engine->get(speech) : NULL);
}
/*! \brief Free a list of results */
@@ -162,15 +130,14 @@
ast_clear_flag(speech, AST_SPEECH_QUIET);
/* If results are on the structure, free them since we are starting again */
- if (speech->results != NULL) {
+ if (speech->results) {
ast_speech_results_free(speech->results);
speech->results = NULL;
}
/* If the engine needs to start stuff up, do it */
- if (speech->engine->start != NULL) {
+ if (speech->engine->start)
speech->engine->start(speech);
- }
return;
}
@@ -178,30 +145,17 @@
/*! \brief Write in signed linear audio to be recognized */
int ast_speech_write(struct ast_speech *speech, void *data, int len)
{
- int res = 0;
-
/* Make sure the speech engine is ready to accept audio */
- if (speech->state != AST_SPEECH_STATE_READY) {
+ if (speech->state != AST_SPEECH_STATE_READY)
return -1;
- }
-
- if (speech->engine->write != NULL) {
- speech->engine->write(speech, data, len);
- }
-
- return res;
+
+ return speech->engine->write(speech, data, len);
}
/*! \brief Change an engine specific attribute */
int ast_speech_change(struct ast_speech *speech, char *name, const char *value)
{
- int res = 0;
-
- if (speech->engine->change != NULL) {
- res = speech->engine->change(speech, name, value);
- }
-
- return res;
+ return (speech->engine->change ? speech->engine->change(speech, name, value) : -1);
}
/*! \brief Create a new speech structure using the engine specified */
@@ -211,18 +165,12 @@
struct ast_speech *new_speech = NULL;
/* Try to find the speech recognition engine that was requested */
- engine = find_engine(engine_name);
- if (engine == NULL) {
- /* Invalid engine or no engine available */
+ if (!(engine = find_engine(engine_name)))
return NULL;
- }
/* Allocate our own speech structure, and try to allocate a structure from the engine too */
- new_speech = ast_calloc(1, sizeof(*new_speech));
- if (new_speech == NULL) {
- /* Ran out of memory while trying to allocate some for a speech structure */
+ if (!(new_speech = ast_calloc(1, sizeof(*new_speech))))
return NULL;
- }
/* Initialize the lock */
ast_mutex_init(&new_speech->lock);
@@ -258,20 +206,15 @@
ast_mutex_destroy(&speech->lock);
/* If results exist on the speech structure, destroy them */
- if (speech->results != NULL) {
+ if (speech->results)
ast_speech_results_free(speech->results);
- speech->results = NULL;
- }
/* If a processing sound is set - free the memory used by it */
- if (speech->processing_sound != NULL) {
+ if (speech->processing_sound)
free(speech->processing_sound);
- speech->processing_sound = NULL;
- }
/* Aloha we are done */
free(speech);
- speech = NULL;
return res;
}
@@ -296,14 +239,9 @@
/*! \brief Change the type of results we want */
int ast_speech_change_results_type(struct ast_speech *speech, enum ast_speech_results_type results_type)
{
- int res = 0;
-
speech->results_type = results_type;
- if (speech->engine->change_results_type)
- res = speech->engine->change_results_type(speech, results_type);
-
- return res;
+ return (speech->engine->change_results_type ? speech->engine->change_results_type(speech, results_type) : 0);
}
/*! \brief Register a speech recognition engine */
@@ -312,9 +250,15 @@
struct ast_speech_engine *existing_engine = NULL;
int res = 0;
- existing_engine = find_engine(engine->name);
- if (existing_engine != NULL) {
- /* Engine already loaded */
+ /* Confirm the engine meets the minimum API requirements */
+ if (!engine->new || !engine->write || !engine->destroy) {
+ ast_log(LOG_WARNING, "Speech recognition engine '%s' did not meet minimum API requirements.\n", engine->name);
+ return -1;
+ }
+
+ /* If an engine is already loaded with this name, error out */
+ if ((existing_engine = find_engine(engine->name))) {
+ ast_log(LOG_WARNING, "Speech recognition engine '%s' already exists.\n", engine->name);
return -1;
}
@@ -322,14 +266,14 @@
ast_verbose(VERBOSE_PREFIX_2 "Registered speech recognition engine '%s'\n", engine->name);
/* Add to the engine linked list and make default if needed */
- AST_LIST_LOCK(&engines);
- AST_LIST_INSERT_HEAD(&engines, engine, list);
- if (default_engine == NULL) {
+ AST_RWLIST_WRLOCK(&engines);
+ AST_RWLIST_INSERT_HEAD(&engines, engine, list);
+ if (!default_engine) {
default_engine = engine;
if (option_verbose > 1)
ast_verbose(VERBOSE_PREFIX_2 "Made '%s' the default speech recognition engine\n", engine->name);
}
- AST_LIST_UNLOCK(&engines);
+ AST_RWLIST_UNLOCK(&engines);
return res;
}
@@ -340,19 +284,17 @@
struct ast_speech_engine *engine = NULL;
int res = -1;
- if (engine_name == NULL) {
- return res;
- }
-
- AST_LIST_LOCK(&engines);
- AST_LIST_TRAVERSE_SAFE_BEGIN(&engines, engine, list) {
+ if (ast_strlen_zero(engine_name))
+ return -1;
+
+ AST_RWLIST_WRLOCK(&engines);
+ AST_RWLIST_TRAVERSE_SAFE_BEGIN(&engines, engine, list) {
if (!strcasecmp(engine->name, engine_name)) {
/* We have our engine... removed it */
- AST_LIST_REMOVE_CURRENT(&engines, list);
+ AST_RWLIST_REMOVE_CURRENT(&engines, list);
/* If this was the default engine, we need to pick a new one */
- if (default_engine == engine) {
- default_engine = AST_LIST_FIRST(&engines);
- }
+ if (!default_engine)
+ default_engine = AST_RWLIST_FIRST(&engines);
if (option_verbose > 1)
ast_verbose(VERBOSE_PREFIX_2 "Unregistered speech recognition engine '%s'\n", engine_name);
/* All went well */
@@ -360,8 +302,8 @@
break;
}
}
- AST_LIST_TRAVERSE_SAFE_END
- AST_LIST_UNLOCK(&engines);
+ AST_RWLIST_TRAVERSE_SAFE_END
+ AST_RWLIST_UNLOCK(&engines);
return res;
}
@@ -374,12 +316,7 @@
static int load_module(void)
{
- int res = 0;
-
- /* Initialize our list of engines */
- AST_LIST_HEAD_INIT_NOLOCK(&engines);
-
- return res;
+ return 0;
}
AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS, "Generic Speech Recognition API",
More information about the asterisk-commits
mailing list