X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Fbindings%2Fpython%2Fbt2%2Fbt2%2Fnative_bt_component_class.i;h=42feccf5a1d80d451b19879f9af4ea065bc83725;hb=ce95fb2680f08176955eb7cf2fd7cd7b6ad2fbfd;hp=522990a29da8aefc83ae92c9417047acaf4ef98a;hpb=f4e38e70bf6dc1ff8c9226d9c8eedaf26fa19f09;p=babeltrace.git diff --git a/src/bindings/python/bt2/bt2/native_bt_component_class.i b/src/bindings/python/bt2/bt2/native_bt_component_class.i index 522990a2..42feccf5 100644 --- a/src/bindings/python/bt2/bt2/native_bt_component_class.i +++ b/src/bindings/python/bt2/bt2/native_bt_component_class.i @@ -22,445 +22,17 @@ * THE SOFTWARE. */ -/* From component-class-const.h */ - -typedef enum bt_component_class_status { - BT_COMPONENT_CLASS_STATUS_OK = 0, - BT_COMPONENT_CLASS_STATUS_NOMEM = -12, -} bt_component_class_status; - -typedef enum bt_component_class_type { - BT_COMPONENT_CLASS_TYPE_SOURCE = 0, - BT_COMPONENT_CLASS_TYPE_FILTER = 1, - BT_COMPONENT_CLASS_TYPE_SINK = 2, -} bt_component_class_type; - -extern const char *bt_component_class_get_name( - const bt_component_class *component_class); - -extern const char *bt_component_class_get_description( - const bt_component_class *component_class); - -extern const char *bt_component_class_get_help( - const bt_component_class *component_class); - -extern bt_component_class_type bt_component_class_get_type( - const bt_component_class *component_class); - -bt_bool bt_component_class_is_source( - const bt_component_class *component_class); - -bt_bool bt_component_class_is_filter( - const bt_component_class *component_class); - -bt_bool bt_component_class_is_sink( - const bt_component_class *component_class); - -extern void bt_component_class_get_ref( - const bt_component_class *component_class); - -extern void bt_component_class_put_ref( - const bt_component_class *component_class); - -/* From component-class-source-const.h */ - -const bt_component_class * -bt_component_class_source_as_component_class_const( - const bt_component_class_source *comp_cls_source); - -extern void bt_component_class_source_get_ref( - const bt_component_class_source *component_class_source); - -extern void bt_component_class_source_put_ref( - const bt_component_class_source *component_class_source); - -/* From component-class-source.h */ - -typedef bt_self_component_status -(*bt_component_class_source_init_method)( - bt_self_component_source *self_component, - const bt_value *params, void *init_method_data); - -typedef void (*bt_component_class_source_finalize_method)( - bt_self_component_source *self_component); - -typedef bt_self_message_iterator_status -(*bt_component_class_source_message_iterator_init_method)( - bt_self_message_iterator *message_iterator, - bt_self_component_source *self_component, - bt_self_component_port_output *port); - -typedef void -(*bt_component_class_source_message_iterator_finalize_method)( - bt_self_message_iterator *message_iterator); - -typedef bt_self_message_iterator_status -(*bt_component_class_source_message_iterator_next_method)( - bt_self_message_iterator *message_iterator, - bt_message_array_const msgs, uint64_t capacity, - uint64_t *count); - -typedef bt_self_message_iterator_status -(*bt_component_class_source_message_iterator_seek_ns_from_origin_method)( - bt_self_message_iterator *message_iterator, - int64_t ns_from_origin); - -typedef bt_self_message_iterator_status -(*bt_component_class_source_message_iterator_seek_beginning_method)( - bt_self_message_iterator *message_iterator); - -typedef bt_bool -(*bt_component_class_source_message_iterator_can_seek_ns_from_origin_method)( - bt_self_message_iterator *message_iterator, - int64_t ns_from_origin); - -typedef bt_bool -(*bt_component_class_source_message_iterator_can_seek_beginning_method)( - bt_self_message_iterator *message_iterator); - -typedef bt_query_status (*bt_component_class_source_query_method)( - bt_self_component_class_source *comp_class, - const bt_query_executor *query_executor, - const char *object, const bt_value *params, - bt_logging_level log_level, - const bt_value **result); - -typedef bt_self_component_status -(*bt_component_class_source_accept_output_port_connection_method)( - bt_self_component_source *self_component, - bt_self_component_port_output *self_port, - const bt_port_input *other_port); - -typedef bt_self_component_status -(*bt_component_class_source_output_port_connected_method)( - bt_self_component_source *self_component, - bt_self_component_port_output *self_port, - const bt_port_input *other_port); - -bt_component_class *bt_component_class_source_as_component_class( - bt_component_class_source *comp_cls_source); - -extern -bt_component_class_source *bt_component_class_source_create( - const char *name, - bt_component_class_source_message_iterator_next_method method); - -extern bt_component_class_status -bt_component_class_source_set_init_method( - bt_component_class_source *comp_class, - bt_component_class_source_init_method method); - -extern bt_component_class_status -bt_component_class_source_set_finalize_method( - bt_component_class_source *comp_class, - bt_component_class_source_finalize_method method); - -extern bt_component_class_status -bt_component_class_source_set_accept_output_port_connection_method( - bt_component_class_source *comp_class, - bt_component_class_source_accept_output_port_connection_method method); - -extern bt_component_class_status -bt_component_class_source_set_output_port_connected_method( - bt_component_class_source *comp_class, - bt_component_class_source_output_port_connected_method method); - -extern bt_component_class_status -bt_component_class_source_set_query_method( - bt_component_class_source *comp_class, - bt_component_class_source_query_method method); - -extern bt_component_class_status -bt_component_class_source_set_message_iterator_init_method( - bt_component_class_source *comp_class, - bt_component_class_source_message_iterator_init_method method); - -extern bt_component_class_status -bt_component_class_source_set_message_iterator_finalize_method( - bt_component_class_source *comp_class, - bt_component_class_source_message_iterator_finalize_method method); - -extern bt_component_class_status -bt_component_class_source_set_message_iterator_seek_ns_from_origin_method( - bt_component_class_source *comp_class, - bt_component_class_source_message_iterator_seek_ns_from_origin_method method); - -extern bt_component_class_status -bt_component_class_source_set_message_iterator_seek_beginning_method( - bt_component_class_source *comp_class, - bt_component_class_source_message_iterator_seek_beginning_method method); - -extern bt_bool -bt_component_class_source_set_message_iterator_can_seek_ns_from_origin_method( - bt_component_class_source *comp_class, - bt_component_class_source_message_iterator_can_seek_ns_from_origin_method method); - -extern bt_bool -bt_component_class_source_set_message_iterator_can_seek_beginning_method( - bt_component_class_source *comp_class, - bt_component_class_source_message_iterator_can_seek_beginning_method method); - -/* From component-class-filter-const.h */ - -const bt_component_class * -bt_component_class_filter_as_component_class_const( - const bt_component_class_filter *comp_cls_filter); - -extern void bt_component_class_filter_get_ref( - const bt_component_class_filter *component_class_filter); - -extern void bt_component_class_filter_put_ref( - const bt_component_class_filter *component_class_filter); - -/* From component-class-filter.h */ - -typedef bt_self_component_status -(*bt_component_class_filter_init_method)( - bt_self_component_filter *self_component, - const bt_value *params, void *init_method_data); - -typedef void (*bt_component_class_filter_finalize_method)( - bt_self_component_filter *self_component); - -typedef bt_self_message_iterator_status -(*bt_component_class_filter_message_iterator_init_method)( - bt_self_message_iterator *message_iterator, - bt_self_component_filter *self_component, - bt_self_component_port_output *port); - -typedef void -(*bt_component_class_filter_message_iterator_finalize_method)( - bt_self_message_iterator *message_iterator); - -typedef bt_self_message_iterator_status -(*bt_component_class_filter_message_iterator_next_method)( - bt_self_message_iterator *message_iterator, - bt_message_array_const msgs, uint64_t capacity, - uint64_t *count); - -typedef bt_self_message_iterator_status -(*bt_component_class_filter_message_iterator_seek_ns_from_origin_method)( - bt_self_message_iterator *message_iterator, - int64_t ns_from_origin); - -typedef bt_self_message_iterator_status -(*bt_component_class_filter_message_iterator_seek_beginning_method)( - bt_self_message_iterator *message_iterator); - -typedef bt_bool -(*bt_component_class_filter_message_iterator_can_seek_ns_from_origin_method)( - bt_self_message_iterator *message_iterator, - int64_t ns_from_origin); - -typedef bt_bool -(*bt_component_class_filter_message_iterator_can_seek_beginning_method)( - bt_self_message_iterator *message_iterator); - -typedef bt_query_status -(*bt_component_class_filter_query_method)( - bt_self_component_class_filter *comp_class, - const bt_query_executor *query_executor, - const char *object, const bt_value *params, - bt_logging_level log_level, - const bt_value **result); - -typedef bt_self_component_status -(*bt_component_class_filter_accept_input_port_connection_method)( - bt_self_component_filter *self_component, - bt_self_component_port_input *self_port, - const bt_port_output *other_port); - -typedef bt_self_component_status -(*bt_component_class_filter_accept_output_port_connection_method)( - bt_self_component_filter *self_component, - bt_self_component_port_output *self_port, - const bt_port_input *other_port); - -typedef bt_self_component_status -(*bt_component_class_filter_input_port_connected_method)( - bt_self_component_filter *self_component, - bt_self_component_port_input *self_port, - const bt_port_output *other_port); - -typedef bt_self_component_status -(*bt_component_class_filter_output_port_connected_method)( - bt_self_component_filter *self_component, - bt_self_component_port_output *self_port, - const bt_port_input *other_port); - -bt_component_class *bt_component_class_filter_as_component_class( - bt_component_class_filter *comp_cls_filter); - -extern -bt_component_class_filter *bt_component_class_filter_create( - const char *name, - bt_component_class_filter_message_iterator_next_method method); - -extern bt_component_class_status -bt_component_class_filter_set_init_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_init_method method); - -extern bt_component_class_status -bt_component_class_filter_set_finalize_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_finalize_method method); - -extern bt_component_class_status -bt_component_class_filter_set_accept_input_port_connection_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_accept_input_port_connection_method method); - -extern bt_component_class_status -bt_component_class_filter_set_accept_output_port_connection_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_accept_output_port_connection_method method); - -extern bt_component_class_status -bt_component_class_filter_set_input_port_connected_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_input_port_connected_method method); - -extern bt_component_class_status -bt_component_class_filter_set_output_port_connected_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_output_port_connected_method method); - -extern bt_component_class_status -bt_component_class_filter_set_query_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_query_method method); - -extern bt_component_class_status -bt_component_class_filter_set_message_iterator_init_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_message_iterator_init_method method); - -extern bt_component_class_status -bt_component_class_filter_set_message_iterator_finalize_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_message_iterator_finalize_method method); - -extern bt_component_class_status -bt_component_class_filter_set_message_iterator_seek_ns_from_origin_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_message_iterator_seek_ns_from_origin_method method); - -extern bt_component_class_status -bt_component_class_filter_set_message_iterator_seek_beginning_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_message_iterator_seek_beginning_method method); - -extern bt_bool -bt_component_class_filter_set_message_iterator_can_seek_ns_from_origin_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_message_iterator_can_seek_ns_from_origin_method method); - -extern bt_bool -bt_component_class_filter_set_message_iterator_can_seek_beginning_method( - bt_component_class_filter *comp_class, - bt_component_class_filter_message_iterator_can_seek_beginning_method method); - -/* From component-class-sink-const.h */ - -const bt_component_class * -bt_component_class_sink_as_component_class_const( - const bt_component_class_sink *comp_cls_sink); - -extern void bt_component_class_sink_get_ref( - const bt_component_class_sink *component_class_sink); - -extern void bt_component_class_sink_put_ref( - const bt_component_class_sink *component_class_sink); - -/* From component-class-sink.h */ - -typedef bt_self_component_status (*bt_component_class_sink_init_method)( - bt_self_component_sink *self_component, - const bt_value *params, void *init_method_data); - -typedef void (*bt_component_class_sink_finalize_method)( - bt_self_component_sink *self_component); - -typedef bt_query_status -(*bt_component_class_sink_query_method)( - bt_self_component_class_sink *comp_class, - const bt_query_executor *query_executor, - const char *object, const bt_value *params, - bt_logging_level log_level, - const bt_value **result); - -typedef bt_self_component_status -(*bt_component_class_sink_accept_input_port_connection_method)( - bt_self_component_sink *self_component, - bt_self_component_port_input *self_port, - const bt_port_output *other_port); - -typedef bt_self_component_status -(*bt_component_class_sink_input_port_connected_method)( - bt_self_component_sink *self_component, - bt_self_component_port_input *self_port, - const bt_port_output *other_port); - -typedef bt_self_component_status -(*bt_component_class_sink_graph_is_configured_method)( - bt_self_component_sink *self_component); - -typedef bt_self_component_status (*bt_component_class_sink_consume_method)( - bt_self_component_sink *self_component); - -bt_component_class *bt_component_class_sink_as_component_class( - bt_component_class_sink *comp_cls_sink); - -extern -bt_component_class_sink *bt_component_class_sink_create( - const char *name, - bt_component_class_sink_consume_method method); - -extern bt_component_class_status bt_component_class_sink_set_init_method( - bt_component_class_sink *comp_class, - bt_component_class_sink_init_method method); - -extern bt_component_class_status bt_component_class_sink_set_finalize_method( - bt_component_class_sink *comp_class, - bt_component_class_sink_finalize_method method); - -extern bt_component_class_status -bt_component_class_sink_set_accept_input_port_connection_method( - bt_component_class_sink *comp_class, - bt_component_class_sink_accept_input_port_connection_method method); - -extern bt_component_class_status -bt_component_class_sink_set_input_port_connected_method( - bt_component_class_sink *comp_class, - bt_component_class_sink_input_port_connected_method method); - -extern bt_component_class_status -bt_component_class_sink_set_graph_is_configured_method( - bt_component_class_sink *comp_class, - bt_component_class_sink_graph_is_configured_method method); - -extern bt_component_class_status bt_component_class_sink_set_query_method( - bt_component_class_sink *comp_class, - bt_component_class_sink_query_method method); - -/* From self-component-class-source.h */ - -const bt_component_class_source * -bt_self_component_class_source_as_component_class_source( - bt_self_component_class_source *self_comp_cls_source); - -/* From self-component-class-filter.h */ - -const bt_component_class_filter * -bt_self_component_class_filter_as_component_class_filter( - bt_self_component_class_filter *self_comp_cls_filter); - -/* From self-component-class-sink.h */ - -const bt_component_class_sink * -bt_self_component_class_sink_as_component_class_sink( - bt_self_component_class_sink *self_comp_cls_sink); +%include +%include +%include +%include +%include +%include +%include +%include +%include +%include +%include %{ /* @@ -471,7 +43,7 @@ bt_self_component_class_sink_as_component_class_sink( * to create a user Python component anyway. * * This hash table is written to when a user-defined Python component - * class is created by one of the bt_py3_component_class_*_create() + * class is created by one of the bt_bt2_component_class_*_create() * functions. * * This function is read from when a user calls bt_component_create() @@ -483,7 +55,8 @@ bt_self_component_class_sink_as_component_class_sink( static GHashTable *bt_cc_ptr_to_py_cls; -static void register_cc_ptr_to_py_cls(struct bt_component_class *bt_cc, +static +void register_cc_ptr_to_py_cls(struct bt_component_class *bt_cc, PyObject *py_cls) { if (!bt_cc_ptr_to_py_cls) { @@ -501,7 +74,8 @@ static void register_cc_ptr_to_py_cls(struct bt_component_class *bt_cc, (gpointer) py_cls); } -static PyObject *lookup_cc_ptr_to_py_cls(const bt_component_class *bt_cc) +static +PyObject *lookup_cc_ptr_to_py_cls(const bt_component_class *bt_cc) { if (!bt_cc_ptr_to_py_cls) { BT_LOGW("Cannot look up Python component class because hash table is NULL: " @@ -522,12 +96,13 @@ static PyObject *py_mod_bt2 = NULL; static PyObject *py_mod_bt2_exc_error_type = NULL; static PyObject *py_mod_bt2_exc_try_again_type = NULL; static PyObject *py_mod_bt2_exc_stop_type = NULL; -static PyObject *py_mod_bt2_exc_port_connection_refused_type = NULL; static PyObject *py_mod_bt2_exc_msg_iter_canceled_type = NULL; -static PyObject *py_mod_bt2_exc_invalid_query_object_type = NULL; -static PyObject *py_mod_bt2_exc_invalid_query_params_type = NULL; +static PyObject *py_mod_bt2_exc_invalid_object_type = NULL; +static PyObject *py_mod_bt2_exc_invalid_params_type = NULL; +static PyObject *py_mod_bt2_exc_unsupported_type = NULL; -static void bt_py3_cc_init_from_bt2(void) +static +void bt_bt2_cc_init_from_bt2(void) { /* * This is called once the bt2 package is loaded. @@ -548,24 +123,25 @@ static void bt_py3_cc_init_from_bt2(void) py_mod_bt2_exc_stop_type = PyObject_GetAttrString(py_mod_bt2, "Stop"); BT_ASSERT(py_mod_bt2_exc_stop_type); - py_mod_bt2_exc_port_connection_refused_type = - PyObject_GetAttrString(py_mod_bt2, "PortConnectionRefused"); - BT_ASSERT(py_mod_bt2_exc_port_connection_refused_type); - py_mod_bt2_exc_invalid_query_object_type = - PyObject_GetAttrString(py_mod_bt2, "InvalidQueryObject"); - BT_ASSERT(py_mod_bt2_exc_invalid_query_object_type); - py_mod_bt2_exc_invalid_query_params_type = - PyObject_GetAttrString(py_mod_bt2, "InvalidQueryParams"); - BT_ASSERT(py_mod_bt2_exc_invalid_query_params_type); + py_mod_bt2_exc_invalid_object_type = + PyObject_GetAttrString(py_mod_bt2, "InvalidObject"); + BT_ASSERT(py_mod_bt2_exc_invalid_object_type); + py_mod_bt2_exc_invalid_params_type = + PyObject_GetAttrString(py_mod_bt2, "InvalidParams"); + BT_ASSERT(py_mod_bt2_exc_invalid_params_type); + py_mod_bt2_exc_unsupported_type = + PyObject_GetAttrString(py_mod_bt2, "Unsupported"); + BT_ASSERT(py_mod_bt2_exc_unsupported_type); } -static void bt_py3_cc_exit_handler(void) +static +void bt_bt2_cc_exit_handler(void) { /* * This is an exit handler (set by the bt2 package). * * We only give back the references that we took in - * bt_py3_cc_init_from_bt2() here. The global variables continue + * bt_bt2_cc_init_from_bt2() here. The global variables continue * to exist for the code of this file, but they are now borrowed * references. If this code is executed, it means that somehow * the modules are still loaded, so it should be safe to use @@ -578,17 +154,17 @@ static void bt_py3_cc_exit_handler(void) Py_XDECREF(py_mod_bt2_exc_error_type); Py_XDECREF(py_mod_bt2_exc_try_again_type); Py_XDECREF(py_mod_bt2_exc_stop_type); - Py_XDECREF(py_mod_bt2_exc_port_connection_refused_type); Py_XDECREF(py_mod_bt2_exc_msg_iter_canceled_type); - Py_XDECREF(py_mod_bt2_exc_invalid_query_object_type); - Py_XDECREF(py_mod_bt2_exc_invalid_query_params_type); + Py_XDECREF(py_mod_bt2_exc_invalid_object_type); + Py_XDECREF(py_mod_bt2_exc_invalid_params_type); } /* Library destructor */ __attribute__((destructor)) -static void bt_py3_native_comp_class_dtor(void) { +static +void native_comp_class_dtor(void) { /* Destroy component class association hash table */ if (bt_cc_ptr_to_py_cls) { BT_LOGD_STR("Destroying native component class to Python component class hash table."); @@ -596,93 +172,71 @@ static void bt_py3_native_comp_class_dtor(void) { } } - -// TODO: maybe we can wrap code in the Python methods (e.g. _query_from_native) -// in a try catch and print the error there instead, it would be simpler. -static -void bt2_py_loge_exception(void) +static inline +void log_exception_and_maybe_append_error(int log_level, + bool append_error, + bt_self_component_class *self_component_class, + bt_self_component *self_component, + bt_self_message_iterator *self_message_iterator) { - PyObject *type = NULL; - PyObject *value = NULL; - PyObject *traceback = NULL; - PyObject *traceback_module = NULL; - PyObject *format_exception_func = NULL; - PyObject *exc_str_list = NULL; - GString *msg_buf = NULL; - Py_ssize_t i; + GString *gstr; BT_ASSERT(PyErr_Occurred() != NULL); - - PyErr_Fetch(&type, &value, &traceback); - - BT_ASSERT(type != NULL); - - /* - * traceback can be NULL, when we fail to call a Python function from the - * native code (there is not Python stack at that point). E.g.: - * - * TypeError: _accept_port_connection_from_native() takes 3 positional arguments but 4 were given - */ - - - /* Make sure `value` is what we expected - an instance of `type`. */ - PyErr_NormalizeException(&type, &value, &traceback); - - traceback_module = PyImport_ImportModule("traceback"); - if (!traceback_module) { - BT_LOGE_STR("Failed to log Python exception (could not import traceback module)."); + gstr = bt_py_common_format_exception(BT_LOG_OUTPUT_LEVEL); + if (!gstr) { + /* bt_py_common_format_exception() logs errors */ goto end; } - format_exception_func = PyObject_GetAttrString(traceback_module, - traceback ? "format_exception" : "format_exception_only"); - if (!format_exception_func) { - BT_LOGE_STR("Failed to log Python exception (could not find format_exception)."); - goto end; - } - - if (!PyCallable_Check(format_exception_func)) { - BT_LOGE_STR("Failed to log Python exception (format_exception is not callable)."); - goto end; - } - - exc_str_list = PyObject_CallFunctionObjArgs(format_exception_func, type, value, traceback, NULL); - if (!exc_str_list) { - PyErr_Print(); - BT_LOGE_STR("Failed to log Python exception (call to format_exception failed)."); - goto end; - } - - msg_buf = g_string_new(NULL); - - for (i = 0; i < PyList_Size(exc_str_list); i++) { - PyObject *exc_str = PyList_GetItem(exc_str_list, i); - const char *str = PyUnicode_AsUTF8(exc_str); - if (!str) { - BT_LOGE_STR("Failed to log Python exception (failed to convert exception to string)."); - goto end; + BT_LOG_WRITE(log_level, BT_LOG_TAG, "%s", gstr->str); + + if (append_error) { + if (self_component_class) { + BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_COMPONENT_CLASS( + self_component_class, "%s", gstr->str); + } else if (self_component) { + BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_COMPONENT( + self_component, "%s", gstr->str); + } else if (self_message_iterator) { + BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_MESSAGE_ITERATOR( + self_message_iterator, "%s", gstr->str); + } else { + BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_UNKNOWN( + "Python", "%s", gstr->str); } - - g_string_append(msg_buf, str); } - BT_LOGE_STR(msg_buf->str); - end: - if (msg_buf) { - g_string_free(msg_buf, TRUE); + if (gstr) { + g_string_free(gstr, TRUE); } - Py_XDECREF(exc_str_list); - Py_XDECREF(format_exception_func); - Py_XDECREF(traceback_module); +} + +static inline +void loge_exception(void) +{ + log_exception_and_maybe_append_error(BT_LOG_ERROR, true, NULL, NULL, NULL); +} - /* PyErr_Restore takes our references. */ - PyErr_Restore(type, value, traceback); +static +void loge_exception_message_iterator( + bt_self_message_iterator *self_message_iterator) +{ + log_exception_and_maybe_append_error(BT_LOG_ERROR, true, NULL, NULL, self_message_iterator); } -static bt_self_component_status bt_py3_exc_to_self_component_status(void) +static inline +void logw_exception(void) { - bt_self_component_status status = BT_SELF_COMPONENT_STATUS_OK; + log_exception_and_maybe_append_error(BT_LOG_WARNING, false, NULL, NULL, NULL); +} + +static inline +int py_exc_to_status(bt_self_component_class *self_component_class, + bt_self_component *self_component, + bt_self_message_iterator *self_message_iterator) +{ + int status = __BT_FUNC_STATUS_OK; PyObject *exc = PyErr_Occurred(); if (!exc) { @@ -691,16 +245,25 @@ static bt_self_component_status bt_py3_exc_to_self_component_status(void) if (PyErr_GivenExceptionMatches(exc, py_mod_bt2_exc_try_again_type)) { - status = BT_SELF_COMPONENT_STATUS_AGAIN; + status = __BT_FUNC_STATUS_AGAIN; } else if (PyErr_GivenExceptionMatches(exc, py_mod_bt2_exc_stop_type)) { - status = BT_SELF_COMPONENT_STATUS_END; + status = __BT_FUNC_STATUS_END; + } else if (PyErr_GivenExceptionMatches(exc, + py_mod_bt2_exc_invalid_object_type)) { + status = __BT_FUNC_STATUS_INVALID_OBJECT; } else if (PyErr_GivenExceptionMatches(exc, - py_mod_bt2_exc_port_connection_refused_type)) { - status = BT_SELF_COMPONENT_STATUS_REFUSE_PORT_CONNECTION; + py_mod_bt2_exc_invalid_params_type)) { + status = __BT_FUNC_STATUS_INVALID_PARAMS; + } else if (PyErr_GivenExceptionMatches(exc, + py_mod_bt2_exc_unsupported_type)) { + status = __BT_FUNC_STATUS_UNSUPPORTED; } else { - bt2_py_loge_exception(); - status = BT_SELF_COMPONENT_STATUS_ERROR; + /* Unknown exception: convert to general error */ + log_exception_and_maybe_append_error(BT_LOG_WARNING, true, + self_component_class, self_component, + self_message_iterator); + status = __BT_FUNC_STATUS_ERROR; } end: @@ -708,63 +271,29 @@ end: return status; } -/* Component class proxy methods (delegate to the attached Python object) */ - -static bt_self_message_iterator_status -bt_py3_exc_to_self_message_iterator_status(void) +static +int py_exc_to_status_component_class(bt_self_component_class *self_component_class) { - enum bt_self_message_iterator_status status = - BT_SELF_MESSAGE_ITERATOR_STATUS_OK; - PyObject *exc = PyErr_Occurred(); - - if (!exc) { - goto end; - } - - if (PyErr_GivenExceptionMatches(exc, py_mod_bt2_exc_stop_type)) { - status = BT_SELF_MESSAGE_ITERATOR_STATUS_END; - } else if (PyErr_GivenExceptionMatches(exc, py_mod_bt2_exc_try_again_type)) { - status = BT_SELF_MESSAGE_ITERATOR_STATUS_AGAIN; - } else { - bt2_py_loge_exception(); - status = BT_SELF_MESSAGE_ITERATOR_STATUS_ERROR; - } - -end: - PyErr_Clear(); - return status; + return py_exc_to_status(self_component_class, NULL, NULL); } -static enum bt_query_status bt_py3_exc_to_query_status(void) +static +int py_exc_to_status_component(bt_self_component *self_component) { - enum bt_query_status status = BT_QUERY_STATUS_OK; - PyObject *exc = PyErr_Occurred(); - - if (!exc) { - goto end; - } - - if (PyErr_GivenExceptionMatches(exc, - py_mod_bt2_exc_invalid_query_object_type)) { - status = BT_QUERY_STATUS_INVALID_OBJECT; - } else if (PyErr_GivenExceptionMatches(exc, - py_mod_bt2_exc_invalid_query_params_type)) { - status = BT_QUERY_STATUS_INVALID_PARAMS; - } else if (PyErr_GivenExceptionMatches(exc, - py_mod_bt2_exc_try_again_type)) { - status = BT_QUERY_STATUS_AGAIN; - } else { - bt2_py_loge_exception(); - status = BT_QUERY_STATUS_ERROR; - } + return py_exc_to_status(NULL, self_component, NULL); +} -end: - PyErr_Clear(); - return status; +static +int py_exc_to_status_message_iterator( + bt_self_message_iterator *self_message_iterator) +{ + return py_exc_to_status(NULL, NULL, self_message_iterator); } -static bt_self_component_status -bt_py3_component_class_init( +/* Component class proxy methods (delegate to the attached Python object) */ + +static +bt_component_class_init_method_status component_class_init( bt_self_component *self_component, void *self_component_v, swig_type_info *self_comp_cls_type_swig_type, @@ -773,7 +302,7 @@ bt_py3_component_class_init( { const bt_component *component = bt_self_component_as_component(self_component); const bt_component_class *component_class = bt_component_borrow_class_const(component); - bt_self_component_status status = BT_SELF_COMPONENT_STATUS_OK; + bt_component_class_init_method_status status = __BT_FUNC_STATUS_OK; PyObject *py_cls = NULL; PyObject *py_comp = NULL; PyObject *py_params_ptr = NULL; @@ -815,19 +344,18 @@ bt_py3_component_class_init( /* * Do the equivalent of this: * - * py_comp = py_cls._init_from_native(py_comp_ptr, py_params_ptr) + * py_comp = py_cls._bt_init_from_native(py_comp_ptr, py_params_ptr) * - * _UserComponentType._init_from_native() calls the Python + * _UserComponentType._bt_init_from_native() calls the Python * component object's __init__() function. */ py_comp = PyObject_CallMethod(py_cls, - "_init_from_native", "(OO)", py_comp_ptr, py_params_ptr); + "_bt_init_from_native", "(OO)", py_comp_ptr, py_params_ptr); if (!py_comp) { - bt2_py_loge_exception(); - BT_LOGE("Failed to call Python class's _init_from_native() method: " + BT_LOGW("Failed to call Python class's _bt_init_from_native() method: " "py-cls-addr=%p", py_cls); - - goto error; + status = py_exc_to_status_component(self_component); + goto end; } /* @@ -840,7 +368,7 @@ bt_py3_component_class_init( goto end; error: - status = BT_SELF_COMPONENT_STATUS_ERROR; + status = __BT_FUNC_STATUS_ERROR; /* * Clear any exception: we're returning a bad status anyway. If @@ -862,43 +390,47 @@ end: * of that class. */ -static bt_self_component_status -bt_py3_component_class_source_init(bt_self_component_source *self_component_source, +static +bt_component_class_init_method_status component_class_source_init( + bt_self_component_source *self_component_source, const bt_value *params, void *init_method_data) { bt_self_component *self_component = bt_self_component_source_as_self_component(self_component_source); - return bt_py3_component_class_init( + return component_class_init( self_component, self_component_source, SWIGTYPE_p_bt_self_component_source, params, init_method_data); } -static bt_self_component_status -bt_py3_component_class_filter_init(bt_self_component_filter *self_component_filter, +static +bt_component_class_init_method_status component_class_filter_init( + bt_self_component_filter *self_component_filter, const bt_value *params, void *init_method_data) { bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - return bt_py3_component_class_init( + return component_class_init( self_component, self_component_filter, SWIGTYPE_p_bt_self_component_filter, params, init_method_data); } -static bt_self_component_status -bt_py3_component_class_sink_init(bt_self_component_sink *self_component_sink, +static +bt_component_class_init_method_status component_class_sink_init( + bt_self_component_sink *self_component_sink, const bt_value *params, void *init_method_data) { bt_self_component *self_component = bt_self_component_sink_as_self_component(self_component_sink); - return bt_py3_component_class_init( + return component_class_init( self_component, self_component_sink, SWIGTYPE_p_bt_self_component_sink, params, init_method_data); } -static void bt_py3_component_class_finalize(bt_self_component *self_component) +static +void component_class_finalize(bt_self_component *self_component) { PyObject *py_comp = bt_self_component_get_data(self_component); BT_ASSERT(py_comp); @@ -908,7 +440,8 @@ static void bt_py3_component_class_finalize(bt_self_component *self_component) "_finalize", NULL); if (PyErr_Occurred()) { - BT_LOGW("User's _finalize() method raised an exception: ignoring."); + BT_LOGW("User component's _finalize() method raised an exception: ignoring:"); + logw_exception(); } /* @@ -921,170 +454,78 @@ static void bt_py3_component_class_finalize(bt_self_component *self_component) Py_DECREF(py_comp); } -static void -bt_py3_component_class_source_finalize(bt_self_component_source *self_component_source) +static +void component_class_source_finalize(bt_self_component_source *self_component_source) { bt_self_component *self_component = bt_self_component_source_as_self_component(self_component_source); - bt_py3_component_class_finalize(self_component); + component_class_finalize(self_component); } -static void -bt_py3_component_class_filter_finalize(bt_self_component_filter *self_component_filter) +static +void component_class_filter_finalize(bt_self_component_filter *self_component_filter) { bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - bt_py3_component_class_finalize(self_component); + component_class_finalize(self_component); } -static void -bt_py3_component_class_sink_finalize(bt_self_component_sink *self_component_sink) +static +void component_class_sink_finalize(bt_self_component_sink *self_component_sink) { bt_self_component *self_component = bt_self_component_sink_as_self_component(self_component_sink); - bt_py3_component_class_finalize(self_component); + component_class_finalize(self_component); } -static bt_self_component_status -bt_py3_component_class_accept_port_connection( - bt_self_component *self_component, - bt_self_component_port *self_component_port, - bt_port_type self_component_port_type, - const bt_port *other_port) +static +bt_bool component_class_can_seek_beginning( + bt_self_message_iterator *self_message_iterator) { - enum bt_self_component_status status; - PyObject *py_comp = NULL; - PyObject *py_self_port_ptr = NULL; - PyObject *py_other_port_ptr = NULL; - PyObject *py_method_result = NULL; + PyObject *py_iter; + PyObject *py_result = NULL; + bt_bool can_seek_beginning = false; - py_comp = bt_self_component_get_data(self_component); - BT_ASSERT(py_comp); - - swig_type_info *self_component_port_swig_type = NULL; - swig_type_info *other_port_swig_type = NULL; - switch (self_component_port_type) { - case BT_PORT_TYPE_INPUT: - self_component_port_swig_type = SWIGTYPE_p_bt_self_component_port_input; - other_port_swig_type = SWIGTYPE_p_bt_port_output; - break; - case BT_PORT_TYPE_OUTPUT: - self_component_port_swig_type = SWIGTYPE_p_bt_self_component_port_output; - other_port_swig_type = SWIGTYPE_p_bt_port_input; - break; - } - BT_ASSERT(self_component_port_swig_type != NULL); - BT_ASSERT(other_port_swig_type != NULL); - - py_self_port_ptr = SWIG_NewPointerObj(SWIG_as_voidptr(self_component_port), - self_component_port_swig_type, 0); - if (!py_self_port_ptr) { - BT_LOGE_STR("Failed to create a SWIG pointer object."); - goto error; - } - - py_other_port_ptr = SWIG_NewPointerObj(SWIG_as_voidptr(other_port), - other_port_swig_type, 0); - if (!py_other_port_ptr) { - BT_LOGE_STR("Failed to create a SWIG pointer object."); - goto error; - } + py_iter = bt_self_message_iterator_get_data(self_message_iterator); + BT_ASSERT(py_iter); - py_method_result = PyObject_CallMethod(py_comp, - "_accept_port_connection_from_native", "(OiO)", py_self_port_ptr, - self_component_port_type, py_other_port_ptr); + py_result = PyObject_GetAttrString(py_iter, "_bt_can_seek_beginning_from_native"); - status = bt_py3_exc_to_self_component_status(); - if (!py_method_result && status == BT_SELF_COMPONENT_STATUS_OK) { - /* Pretty sure this should never happen, but just in case */ - BT_LOGE("User's _accept_port_connection() method failed without raising an exception: " - "status=%d", status); - goto error; - } + BT_ASSERT(!py_result || PyBool_Check(py_result)); - if (status == BT_SELF_COMPONENT_STATUS_REFUSE_PORT_CONNECTION) { + if (py_result) { + can_seek_beginning = PyObject_IsTrue(py_result); + } else { /* - * Looks like the user method raised - * PortConnectionRefused: accept this like if it - * returned False. + * Once can_seek_beginning can report errors, convert the + * exception to a status. For now, log and return false; */ - goto end; - } else if (status != BT_SELF_COMPONENT_STATUS_OK) { - BT_LOGE("User's _accept_port_connection() raised an unexpected exception: " - "status=%d", status); - goto error; + loge_exception_message_iterator(self_message_iterator); + PyErr_Clear(); } - BT_ASSERT(PyBool_Check(py_method_result)); + Py_XDECREF(py_result); - if (py_method_result == Py_True) { - status = BT_SELF_COMPONENT_STATUS_OK; - } else { - status = BT_SELF_COMPONENT_STATUS_REFUSE_PORT_CONNECTION; - } - - goto end; - -error: - status = BT_SELF_COMPONENT_STATUS_ERROR; - - /* - * Clear any exception: we're returning a bad status anyway. If - * this call originated from Python, then the user gets an - * appropriate error. - */ - PyErr_Clear(); - -end: - Py_XDECREF(py_self_port_ptr); - Py_XDECREF(py_other_port_ptr); - Py_XDECREF(py_method_result); - return status; -} - -static bt_self_component_status -bt_py3_component_class_source_accept_output_port_connection(bt_self_component_source *self_component_source, - bt_self_component_port_output *self_component_port_output, - const bt_port_input *other_port_input) -{ - bt_self_component *self_component = bt_self_component_source_as_self_component(self_component_source); - bt_self_component_port *self_component_port = bt_self_component_port_output_as_self_component_port(self_component_port_output); - const bt_port *other_port = bt_port_input_as_port_const(other_port_input); - return bt_py3_component_class_accept_port_connection(self_component, self_component_port, BT_PORT_TYPE_OUTPUT, other_port); -} - -static bt_self_component_status -bt_py3_component_class_filter_accept_input_port_connection(bt_self_component_filter *self_component_filter, - bt_self_component_port_input *self_component_port_input, - const bt_port_output *other_port_output) -{ - bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - bt_self_component_port *self_component_port = bt_self_component_port_input_as_self_component_port(self_component_port_input); - const bt_port *other_port = bt_port_output_as_port_const(other_port_output); - return bt_py3_component_class_accept_port_connection(self_component, self_component_port, BT_PORT_TYPE_INPUT, other_port); + return can_seek_beginning; } -static bt_self_component_status -bt_py3_component_class_filter_accept_output_port_connection(bt_self_component_filter *self_component_filter, - bt_self_component_port_output *self_component_port_output, - const bt_port_input *other_port_input) -{ - bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - bt_self_component_port *self_component_port = bt_self_component_port_output_as_self_component_port(self_component_port_output); - const bt_port *other_port = bt_port_input_as_port_const(other_port_input); - return bt_py3_component_class_accept_port_connection(self_component, self_component_port, BT_PORT_TYPE_OUTPUT, other_port); -} - -static bt_self_component_status -bt_py3_component_class_sink_accept_input_port_connection(bt_self_component_sink *self_component_sink, - bt_self_component_port_input *self_component_port_input, - const bt_port_output *other_port_output) +static +bt_component_class_message_iterator_seek_beginning_method_status +component_class_seek_beginning(bt_self_message_iterator *self_message_iterator) { - bt_self_component *self_component = bt_self_component_sink_as_self_component(self_component_sink); - bt_self_component_port *self_component_port = bt_self_component_port_input_as_self_component_port(self_component_port_input); - const bt_port *other_port = bt_port_output_as_port_const(other_port_output); - return bt_py3_component_class_accept_port_connection(self_component, self_component_port, BT_PORT_TYPE_INPUT, other_port); + PyObject *py_iter; + PyObject *py_result; + bt_component_class_message_iterator_seek_beginning_method_status status; + + py_iter = bt_self_message_iterator_get_data(self_message_iterator); + BT_ASSERT(py_iter); + py_result = PyObject_CallMethod(py_iter, "_bt_seek_beginning_from_native", + NULL); + BT_ASSERT(!py_result || py_result == Py_None); + status = py_exc_to_status_message_iterator(self_message_iterator); + Py_XDECREF(py_result); + return status; } -static bt_self_component_status -bt_py3_component_class_port_connected( +static +bt_component_class_port_connected_method_status component_class_port_connected( bt_self_component *self_component, void *self_component_port, swig_type_info *self_component_port_swig_type, @@ -1092,7 +533,7 @@ bt_py3_component_class_port_connected( const void *other_port, swig_type_info *other_port_swig_type) { - bt_self_component_status status; + bt_component_class_port_connected_method_status status; PyObject *py_comp = NULL; PyObject *py_self_port_ptr = NULL; PyObject *py_other_port_ptr = NULL; @@ -1100,12 +541,11 @@ bt_py3_component_class_port_connected( py_comp = bt_self_component_get_data(self_component); BT_ASSERT(py_comp); - py_self_port_ptr = SWIG_NewPointerObj(SWIG_as_voidptr(self_component_port), self_component_port_swig_type, 0); if (!py_self_port_ptr) { BT_LOGF_STR("Failed to create a SWIG pointer object."); - status = BT_SELF_COMPONENT_STATUS_NOMEM; + status = __BT_FUNC_STATUS_MEMORY_ERROR; goto end; } @@ -1113,34 +553,32 @@ bt_py3_component_class_port_connected( other_port_swig_type, 0); if (!py_other_port_ptr) { BT_LOGF_STR("Failed to create a SWIG pointer object."); - status = BT_SELF_COMPONENT_STATUS_NOMEM; + status = __BT_FUNC_STATUS_MEMORY_ERROR; goto end; } py_method_result = PyObject_CallMethod(py_comp, - "_port_connected_from_native", "(OiO)", py_self_port_ptr, + "_bt_port_connected_from_native", "(OiO)", py_self_port_ptr, self_component_port_type, py_other_port_ptr); - BT_ASSERT(!py_method_result || py_method_result == Py_None); - - status = bt_py3_exc_to_self_component_status(); + status = py_exc_to_status_component(self_component); end: Py_XDECREF(py_self_port_ptr); Py_XDECREF(py_other_port_ptr); Py_XDECREF(py_method_result); - return status; } -static bt_self_component_status -bt_py3_component_class_source_output_port_connected( +static +bt_component_class_port_connected_method_status +component_class_source_output_port_connected( bt_self_component_source *self_component_source, bt_self_component_port_output *self_component_port_output, const bt_port_input *other_port_input) { bt_self_component *self_component = bt_self_component_source_as_self_component(self_component_source); - return bt_py3_component_class_port_connected( + return component_class_port_connected( self_component, self_component_port_output, SWIGTYPE_p_bt_self_component_port_output, @@ -1149,15 +587,16 @@ bt_py3_component_class_source_output_port_connected( SWIGTYPE_p_bt_port_input); } -static bt_self_component_status -bt_py3_component_class_filter_input_port_connected( +static +bt_component_class_port_connected_method_status +component_class_filter_input_port_connected( bt_self_component_filter *self_component_filter, bt_self_component_port_input *self_component_port_input, const bt_port_output *other_port_output) { bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - return bt_py3_component_class_port_connected( + return component_class_port_connected( self_component, self_component_port_input, SWIGTYPE_p_bt_self_component_port_input, @@ -1166,15 +605,16 @@ bt_py3_component_class_filter_input_port_connected( SWIGTYPE_p_bt_port_output); } -static bt_self_component_status -bt_py3_component_class_filter_output_port_connected( +static +bt_component_class_port_connected_method_status +component_class_filter_output_port_connected( bt_self_component_filter *self_component_filter, bt_self_component_port_output *self_component_port_output, const bt_port_input *other_port_input) { bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - return bt_py3_component_class_port_connected( + return component_class_port_connected( self_component, self_component_port_output, SWIGTYPE_p_bt_self_component_port_output, @@ -1183,15 +623,16 @@ bt_py3_component_class_filter_output_port_connected( SWIGTYPE_p_bt_port_input); } -static bt_self_component_status -bt_py3_component_class_sink_input_port_connected( +static +bt_component_class_port_connected_method_status +component_class_sink_input_port_connected( bt_self_component_sink *self_component_sink, bt_self_component_port_input *self_component_port_input, const bt_port_output *other_port_output) { bt_self_component *self_component = bt_self_component_sink_as_self_component(self_component_sink); - return bt_py3_component_class_port_connected( + return component_class_port_connected( self_component, self_component_port_input, SWIGTYPE_p_bt_self_component_port_input, @@ -1200,30 +641,29 @@ bt_py3_component_class_sink_input_port_connected( SWIGTYPE_p_bt_port_output); } -static bt_self_component_status -bt_py3_component_class_sink_graph_is_configured(bt_self_component_sink *self_component_sink) +static +bt_component_class_sink_graph_is_configured_method_status +component_class_sink_graph_is_configured( + bt_self_component_sink *self_component_sink) { PyObject *py_comp = NULL; PyObject *py_method_result = NULL; - bt_self_component_status status = BT_SELF_COMPONENT_STATUS_OK; + bt_component_class_sink_graph_is_configured_method_status status = __BT_FUNC_STATUS_OK; bt_self_component *self_component = bt_self_component_sink_as_self_component(self_component_sink); py_comp = bt_self_component_get_data(self_component); py_method_result = PyObject_CallMethod(py_comp, - "_graph_is_configured_from_native", NULL); - + "_bt_graph_is_configured_from_native", NULL); BT_ASSERT(!py_method_result || py_method_result == Py_None); - - status = bt_py3_exc_to_self_component_status(); - + status = py_exc_to_status_component(self_component); Py_XDECREF(py_method_result); - return status; } -static bt_query_status -bt_py3_component_class_query( +static +bt_component_class_query_method_status component_class_query( const bt_component_class *component_class, + bt_self_component_class *self_component_class, const bt_query_executor *query_executor, const char *object, const bt_value *params, bt_logging_level log_level, @@ -1235,7 +675,7 @@ bt_py3_component_class_query( PyObject *py_query_func = NULL; PyObject *py_object = NULL; PyObject *py_results_addr = NULL; - bt_query_status status = BT_QUERY_STATUS_OK; + bt_component_class_query_method_status status = __BT_FUNC_STATUS_OK; py_cls = lookup_cc_ptr_to_py_cls(component_class); if (!py_cls) { @@ -1265,13 +705,12 @@ bt_py3_component_class_query( } py_results_addr = PyObject_CallMethod(py_cls, - "_query_from_native", "(OOOi)", py_query_exec_ptr, + "_bt_query_from_native", "(OOOi)", py_query_exec_ptr, py_object, py_params_ptr, (int) log_level); - if (!py_results_addr) { - BT_LOGE("Failed to call Python class's _query_from_native() method: " + BT_LOGW("Failed to call Python class's _bt_query_from_native() method: " "py-cls-addr=%p", py_cls); - status = bt_py3_exc_to_query_status(); + status = py_exc_to_status_component_class(self_component_class); goto end; } @@ -1287,7 +726,7 @@ bt_py3_component_class_query( error: PyErr_Clear(); - status = BT_QUERY_STATUS_ERROR; + status = __BT_FUNC_STATUS_ERROR; end: Py_XDECREF(py_params_ptr); @@ -1298,8 +737,8 @@ end: return status; } -static bt_query_status -bt_py3_component_class_source_query( +static +bt_component_class_query_method_status component_class_source_query( bt_self_component_class_source *self_component_class_source, const bt_query_executor *query_executor, const char *object, const bt_value *params, @@ -1308,11 +747,13 @@ bt_py3_component_class_source_query( { const bt_component_class_source *component_class_source = bt_self_component_class_source_as_component_class_source(self_component_class_source); const bt_component_class *component_class = bt_component_class_source_as_component_class_const(component_class_source); - return bt_py3_component_class_query(component_class, query_executor, object, params, log_level, result); + bt_self_component_class *self_component_class = bt_self_component_class_source_as_self_component_class(self_component_class_source); + + return component_class_query(component_class, self_component_class, query_executor, object, params, log_level, result); } -static bt_query_status -bt_py3_component_class_filter_query( +static +bt_component_class_query_method_status component_class_filter_query( bt_self_component_class_filter *self_component_class_filter, const bt_query_executor *query_executor, const char *object, const bt_value *params, @@ -1321,11 +762,13 @@ bt_py3_component_class_filter_query( { const bt_component_class_filter *component_class_filter = bt_self_component_class_filter_as_component_class_filter(self_component_class_filter); const bt_component_class *component_class = bt_component_class_filter_as_component_class_const(component_class_filter); - return bt_py3_component_class_query(component_class, query_executor, object, params, log_level, result); + bt_self_component_class *self_component_class = bt_self_component_class_filter_as_self_component_class(self_component_class_filter); + + return component_class_query(component_class, self_component_class, query_executor, object, params, log_level, result); } -static bt_query_status -bt_py3_component_class_sink_query( +static +bt_component_class_query_method_status component_class_sink_query( bt_self_component_class_sink *self_component_class_sink, const bt_query_executor *query_executor, const char *object, const bt_value *params, @@ -1334,16 +777,19 @@ bt_py3_component_class_sink_query( { const bt_component_class_sink *component_class_sink = bt_self_component_class_sink_as_component_class_sink(self_component_class_sink); const bt_component_class *component_class = bt_component_class_sink_as_component_class_const(component_class_sink); - return bt_py3_component_class_query(component_class, query_executor, object, params, log_level, result); + bt_self_component_class *self_component_class = bt_self_component_class_sink_as_self_component_class(self_component_class_sink); + + return component_class_query(component_class, self_component_class, query_executor, object, params, log_level, result); } -static bt_self_message_iterator_status -bt_py3_component_class_message_iterator_init( +static +bt_component_class_message_iterator_init_method_status +component_class_message_iterator_init( bt_self_message_iterator *self_message_iterator, bt_self_component *self_component, bt_self_component_port_output *self_component_port_output) { - bt_self_message_iterator_status status = BT_SELF_MESSAGE_ITERATOR_STATUS_OK; + bt_component_class_message_iterator_init_method_status status = __BT_FUNC_STATUS_OK; PyObject *py_comp_cls = NULL; PyObject *py_iter_cls = NULL; PyObject *py_iter_ptr = NULL; @@ -1358,19 +804,23 @@ bt_py3_component_class_message_iterator_init( py_comp_cls = PyObject_GetAttrString(py_comp, "__class__"); if (!py_comp_cls) { BT_LOGE_STR("Cannot get Python object's `__class__` attribute."); - goto error; + goto python_error; } py_iter_cls = PyObject_GetAttrString(py_comp_cls, "_iter_cls"); if (!py_iter_cls) { BT_LOGE_STR("Cannot get Python class's `_iter_cls` attribute."); - goto error; + goto python_error; } py_iter_ptr = SWIG_NewPointerObj(SWIG_as_voidptr(self_message_iterator), SWIGTYPE_p_bt_self_message_iterator, 0); if (!py_iter_ptr) { - BT_LOGE_STR("Failed to create a SWIG pointer object."); + const char *err = "Failed to create a SWIG pointer object."; + + BT_LOGE_STR(err); + BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_MESSAGE_ITERATOR( + self_message_iterator, err); goto error; } @@ -1385,8 +835,7 @@ bt_py3_component_class_message_iterator_init( if (!py_iter) { BT_LOGE("Failed to call Python class's __new__() method: " "py-cls-addr=%p", py_iter_cls); - bt2_py_loge_exception(); - goto error; + goto python_error; } /* @@ -1401,18 +850,23 @@ bt_py3_component_class_message_iterator_init( * user Python component object from which the iterator was * created). */ - py_component_port_output_ptr = SWIG_NewPointerObj(SWIG_as_voidptr(self_component_port_output), + py_component_port_output_ptr = SWIG_NewPointerObj( + SWIG_as_voidptr(self_component_port_output), SWIGTYPE_p_bt_self_component_port_output, 0); if (!py_component_port_output_ptr) { - BT_LOGE_STR("Failed to create a SWIG pointer object."); + const char *err = "Failed to create a SWIG pointer object."; + + BT_LOGE_STR(err); + BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_MESSAGE_ITERATOR( + self_message_iterator, err); goto error; } - py_init_method_result = PyObject_CallMethod(py_iter, "_init_from_native", "O", py_component_port_output_ptr); + py_init_method_result = PyObject_CallMethod(py_iter, + "_bt_init_from_native", "O", py_component_port_output_ptr); if (!py_init_method_result) { - BT_LOGE_STR("User's __init__() method failed."); - bt2_py_loge_exception(); - goto error; + BT_LOGE_STR("User's __init__() method failed:"); + goto python_error; } /* @@ -1437,24 +891,19 @@ bt_py3_component_class_message_iterator_init( py_iter = NULL; goto end; -error: - status = bt_py3_exc_to_self_message_iterator_status(); - if (status == BT_SELF_MESSAGE_ITERATOR_STATUS_OK) { - /* - * Looks like there wasn't any exception from the Python - * side, but we're still in an error state here. - */ - status = BT_SELF_MESSAGE_ITERATOR_STATUS_ERROR; - } +python_error: + /* Handling of errors that cause a Python exception to be set. */ + status = py_exc_to_status_message_iterator(self_message_iterator); + BT_ASSERT(status != __BT_FUNC_STATUS_OK); + goto end; - /* - * Clear any exception: we're returning a bad status anyway. If - * this call originated from Python, then the user gets an - * appropriate creation error. - */ - PyErr_Clear(); +error: + /* Handling of errors that don't cause a Python exception to be set. */ + status = __BT_FUNC_STATUS_ERROR; end: + BT_ASSERT(!PyErr_Occurred()); + Py_XDECREF(py_comp_cls); Py_XDECREF(py_iter_cls); Py_XDECREF(py_iter_ptr); @@ -1464,28 +913,32 @@ end: return status; } -static bt_self_message_iterator_status -bt_py3_component_class_source_message_iterator_init( +static +bt_component_class_message_iterator_init_method_status +component_class_source_message_iterator_init( bt_self_message_iterator *self_message_iterator, bt_self_component_source *self_component_source, bt_self_component_port_output *self_component_port_output) { bt_self_component *self_component = bt_self_component_source_as_self_component(self_component_source); - return bt_py3_component_class_message_iterator_init(self_message_iterator, self_component, self_component_port_output); + + return component_class_message_iterator_init(self_message_iterator, self_component, self_component_port_output); } -static bt_self_message_iterator_status -bt_py3_component_class_filter_message_iterator_init( +static +bt_component_class_message_iterator_init_method_status +component_class_filter_message_iterator_init( bt_self_message_iterator *self_message_iterator, bt_self_component_filter *self_component_filter, bt_self_component_port_output *self_component_port_output) { bt_self_component *self_component = bt_self_component_filter_as_self_component(self_component_filter); - return bt_py3_component_class_message_iterator_init(self_message_iterator, self_component, self_component_port_output); + + return component_class_message_iterator_init(self_message_iterator, self_component, self_component_port_output); } -static void -bt_py3_component_class_message_iterator_finalize( +static +void component_class_message_iterator_finalize( bt_self_message_iterator *message_iterator) { PyObject *py_message_iter = bt_self_message_iterator_get_data(message_iterator); @@ -1498,7 +951,8 @@ bt_py3_component_class_message_iterator_finalize( "_finalize", NULL); if (PyErr_Occurred()) { - BT_LOGW("User's _finalize() method raised an exception: ignoring."); + BT_LOGW("User's _finalize() method raised an exception: ignoring:"); + logw_exception(); } /* @@ -1513,22 +967,23 @@ bt_py3_component_class_message_iterator_finalize( /* Valid for both sources and filters. */ -static bt_self_message_iterator_status -bt_py3_component_class_message_iterator_next( - bt_self_message_iterator *message_iterator, - bt_message_array_const msgs, uint64_t capacity, - uint64_t *count) +static +bt_component_class_message_iterator_next_method_status +component_class_message_iterator_next( + bt_self_message_iterator *message_iterator, + bt_message_array_const msgs, uint64_t capacity, + uint64_t *count) { - bt_self_message_iterator_status status = BT_SELF_MESSAGE_ITERATOR_STATUS_OK; + bt_component_class_message_iterator_next_method_status status = __BT_FUNC_STATUS_OK; PyObject *py_message_iter = bt_self_message_iterator_get_data(message_iterator); PyObject *py_method_result = NULL; BT_ASSERT(py_message_iter); py_method_result = PyObject_CallMethod(py_message_iter, - "_next_from_native", NULL); + "_bt_next_from_native", NULL); if (!py_method_result) { - status = bt_py3_exc_to_self_message_iterator_status(); - BT_ASSERT(status != BT_SELF_MESSAGE_ITERATOR_STATUS_OK); + status = py_exc_to_status_message_iterator(message_iterator); + BT_ASSERT(status != __BT_FUNC_STATUS_OK); goto end; } @@ -1549,24 +1004,24 @@ end: return status; } -static bt_self_component_status -bt_py3_component_class_sink_consume(bt_self_component_sink *self_component_sink) +static +bt_component_class_sink_consume_method_status +component_class_sink_consume(bt_self_component_sink *self_component_sink) { bt_self_component *self_component = bt_self_component_sink_as_self_component(self_component_sink); PyObject *py_comp = bt_self_component_get_data(self_component); PyObject *py_method_result = NULL; - bt_self_component_status status; + bt_component_class_sink_consume_method_status status; BT_ASSERT(py_comp); py_method_result = PyObject_CallMethod(py_comp, "_consume", NULL); - - status = bt_py3_exc_to_self_component_status(); - if (!py_method_result && status == BT_SELF_COMPONENT_STATUS_OK) { + status = py_exc_to_status_component(self_component); + if (!py_method_result && status == __BT_FUNC_STATUS_OK) { /* Pretty sure this should never happen, but just in case */ BT_LOGE("User's _consume() method failed without raising an exception: " "status=%d", status); - status = BT_SELF_COMPONENT_STATUS_ERROR; + status = __BT_FUNC_STATUS_ERROR; } Py_XDECREF(py_method_result); @@ -1574,7 +1029,7 @@ bt_py3_component_class_sink_consume(bt_self_component_sink *self_component_sink) } static -int bt_py3_component_class_set_help_and_desc( +int component_class_set_help_and_desc( bt_component_class *component_class, const char *description, const char *help) { @@ -1605,7 +1060,7 @@ end: } static -bt_component_class_source *bt_py3_component_class_source_create( +bt_component_class_source *bt_bt2_component_class_source_create( PyObject *py_cls, const char *name, const char *description, const char *help) { @@ -1614,9 +1069,8 @@ bt_component_class_source *bt_py3_component_class_source_create( int ret; BT_ASSERT(py_cls); - component_class_source = bt_component_class_source_create(name, - bt_py3_component_class_message_iterator_next); + component_class_message_iterator_next); if (!component_class_source) { BT_LOGE_STR("Cannot create source component class."); goto end; @@ -1624,29 +1078,31 @@ bt_component_class_source *bt_py3_component_class_source_create( component_class = bt_component_class_source_as_component_class(component_class_source); - if (bt_py3_component_class_set_help_and_desc(component_class, description, help)) { + if (component_class_set_help_and_desc(component_class, description, help)) { goto end; } - ret = bt_component_class_source_set_init_method(component_class_source, bt_py3_component_class_source_init); + ret = bt_component_class_source_set_init_method(component_class_source, component_class_source_init); + BT_ASSERT(ret == 0); + ret = bt_component_class_source_set_finalize_method(component_class_source, component_class_source_finalize); BT_ASSERT(ret == 0); - ret = bt_component_class_source_set_finalize_method (component_class_source, bt_py3_component_class_source_finalize); + ret = bt_component_class_source_set_message_iterator_can_seek_beginning_method(component_class_source, + component_class_can_seek_beginning); BT_ASSERT(ret == 0); - ret = bt_component_class_source_set_accept_output_port_connection_method(component_class_source, - bt_py3_component_class_source_accept_output_port_connection); + ret = bt_component_class_source_set_message_iterator_seek_beginning_method(component_class_source, + component_class_seek_beginning); BT_ASSERT(ret == 0); ret = bt_component_class_source_set_output_port_connected_method(component_class_source, - bt_py3_component_class_source_output_port_connected); + component_class_source_output_port_connected); BT_ASSERT(ret == 0); - ret = bt_component_class_source_set_query_method(component_class_source, bt_py3_component_class_source_query); + ret = bt_component_class_source_set_query_method(component_class_source, component_class_source_query); BT_ASSERT(ret == 0); ret = bt_component_class_source_set_message_iterator_init_method( - component_class_source, bt_py3_component_class_source_message_iterator_init); + component_class_source, component_class_source_message_iterator_init); BT_ASSERT(ret == 0); ret = bt_component_class_source_set_message_iterator_finalize_method( - component_class_source, bt_py3_component_class_message_iterator_finalize); + component_class_source, component_class_message_iterator_finalize); BT_ASSERT(ret == 0); - register_cc_ptr_to_py_cls(component_class, py_cls); end: @@ -1654,7 +1110,7 @@ end: } static -bt_component_class_filter *bt_py3_component_class_filter_create( +bt_component_class_filter *bt_bt2_component_class_filter_create( PyObject *py_cls, const char *name, const char *description, const char *help) { @@ -1663,9 +1119,8 @@ bt_component_class_filter *bt_py3_component_class_filter_create( int ret; BT_ASSERT(py_cls); - component_class_filter = bt_component_class_filter_create(name, - bt_py3_component_class_message_iterator_next); + component_class_message_iterator_next); if (!component_class_filter) { BT_LOGE_STR("Cannot create filter component class."); goto end; @@ -1673,35 +1128,34 @@ bt_component_class_filter *bt_py3_component_class_filter_create( component_class = bt_component_class_filter_as_component_class(component_class_filter); - if (bt_py3_component_class_set_help_and_desc(component_class, description, help)) { + if (component_class_set_help_and_desc(component_class, description, help)) { goto end; } - ret = bt_component_class_filter_set_init_method(component_class_filter, bt_py3_component_class_filter_init); + ret = bt_component_class_filter_set_init_method(component_class_filter, component_class_filter_init); BT_ASSERT(ret == 0); - ret = bt_component_class_filter_set_finalize_method (component_class_filter, bt_py3_component_class_filter_finalize); + ret = bt_component_class_filter_set_finalize_method (component_class_filter, component_class_filter_finalize); BT_ASSERT(ret == 0); - ret = bt_component_class_filter_set_accept_input_port_connection_method(component_class_filter, - bt_py3_component_class_filter_accept_input_port_connection); + ret = bt_component_class_filter_set_message_iterator_can_seek_beginning_method(component_class_filter, + component_class_can_seek_beginning); BT_ASSERT(ret == 0); - ret = bt_component_class_filter_set_accept_output_port_connection_method(component_class_filter, - bt_py3_component_class_filter_accept_output_port_connection); + ret = bt_component_class_filter_set_message_iterator_seek_beginning_method(component_class_filter, + component_class_seek_beginning); BT_ASSERT(ret == 0); ret = bt_component_class_filter_set_input_port_connected_method(component_class_filter, - bt_py3_component_class_filter_input_port_connected); + component_class_filter_input_port_connected); BT_ASSERT(ret == 0); ret = bt_component_class_filter_set_output_port_connected_method(component_class_filter, - bt_py3_component_class_filter_output_port_connected); + component_class_filter_output_port_connected); BT_ASSERT(ret == 0); - ret = bt_component_class_filter_set_query_method(component_class_filter, bt_py3_component_class_filter_query); + ret = bt_component_class_filter_set_query_method(component_class_filter, component_class_filter_query); BT_ASSERT(ret == 0); ret = bt_component_class_filter_set_message_iterator_init_method( - component_class_filter, bt_py3_component_class_filter_message_iterator_init); + component_class_filter, component_class_filter_message_iterator_init); BT_ASSERT(ret == 0); ret = bt_component_class_filter_set_message_iterator_finalize_method( - component_class_filter, bt_py3_component_class_message_iterator_finalize); + component_class_filter, component_class_message_iterator_finalize); BT_ASSERT(ret == 0); - register_cc_ptr_to_py_cls(component_class, py_cls); end: @@ -1709,7 +1163,7 @@ end: } static -bt_component_class_sink *bt_py3_component_class_sink_create( +bt_component_class_sink *bt_bt2_component_class_sink_create( PyObject *py_cls, const char *name, const char *description, const char *help) { @@ -1718,8 +1172,7 @@ bt_component_class_sink *bt_py3_component_class_sink_create( int ret; BT_ASSERT(py_cls); - - component_class_sink = bt_component_class_sink_create(name, bt_py3_component_class_sink_consume); + component_class_sink = bt_component_class_sink_create(name, component_class_sink_consume); if (!component_class_sink) { BT_LOGE_STR("Cannot create sink component class."); @@ -1728,26 +1181,22 @@ bt_component_class_sink *bt_py3_component_class_sink_create( component_class = bt_component_class_sink_as_component_class(component_class_sink); - if (bt_py3_component_class_set_help_and_desc(component_class, description, help)) { + if (component_class_set_help_and_desc(component_class, description, help)) { goto end; } - ret = bt_component_class_sink_set_init_method(component_class_sink, bt_py3_component_class_sink_init); + ret = bt_component_class_sink_set_init_method(component_class_sink, component_class_sink_init); BT_ASSERT(ret == 0); - ret = bt_component_class_sink_set_finalize_method(component_class_sink, bt_py3_component_class_sink_finalize); - BT_ASSERT(ret == 0); - ret = bt_component_class_sink_set_accept_input_port_connection_method(component_class_sink, - bt_py3_component_class_sink_accept_input_port_connection); + ret = bt_component_class_sink_set_finalize_method(component_class_sink, component_class_sink_finalize); BT_ASSERT(ret == 0); ret = bt_component_class_sink_set_input_port_connected_method(component_class_sink, - bt_py3_component_class_sink_input_port_connected); + component_class_sink_input_port_connected); BT_ASSERT(ret == 0); ret = bt_component_class_sink_set_graph_is_configured_method(component_class_sink, - bt_py3_component_class_sink_graph_is_configured); + component_class_sink_graph_is_configured); BT_ASSERT(ret == 0); - ret = bt_component_class_sink_set_query_method(component_class_sink, bt_py3_component_class_sink_query); + ret = bt_component_class_sink_set_query_method(component_class_sink, component_class_sink_query); BT_ASSERT(ret == 0); - register_cc_ptr_to_py_cls(component_class, py_cls); end: @@ -1755,14 +1204,14 @@ end: } %} -struct bt_component_class_source *bt_py3_component_class_source_create( +struct bt_component_class_source *bt_bt2_component_class_source_create( PyObject *py_cls, const char *name, const char *description, const char *help); -struct bt_component_class_filter *bt_py3_component_class_filter_create( +struct bt_component_class_filter *bt_bt2_component_class_filter_create( PyObject *py_cls, const char *name, const char *description, const char *help); -struct bt_component_class_sink *bt_py3_component_class_sink_create( +struct bt_component_class_sink *bt_bt2_component_class_sink_create( PyObject *py_cls, const char *name, const char *description, const char *help); -void bt_py3_cc_init_from_bt2(void); -void bt_py3_cc_exit_handler(void); +void bt_bt2_cc_init_from_bt2(void); +void bt_bt2_cc_exit_handler(void);