API : add_trace return the trace_handle id
authorJulien Desfossez <julien.desfossez@efficios.com>
Thu, 16 Feb 2012 18:13:50 +0000 (13:13 -0500)
committerMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Thu, 16 Feb 2012 18:13:50 +0000 (13:13 -0500)
Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
include/babeltrace/context.h
lib/context.c

index 32282cf4cc7acb7f91e275da6bbb7f852257751a..261c4d47b625c6aa70382ba93ba02389dc9e5003 100644 (file)
@@ -41,7 +41,8 @@ struct bt_context *bt_context_create(void);
  *
  * Open a trace
  *
- * Return: 0 on success, nonzero on failure.
+ * Return: the trace handle id (>= 0) on success, a negative
+ * value on error.
  */
 int bt_context_add_trace(struct bt_context *ctx, const char *path,
                const char *format);
index 5408bc81c942c5ceead7a951b164e8b60c4d46f4..90d77df88f6840af5a954616048f5fbc0f81c631 100644 (file)
@@ -62,6 +62,7 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path,
        struct trace_descriptor *td;
        struct format *fmt;
        struct bt_trace_handle *handle;
+       int ret;
 
        fmt = bt_lookup_format(g_quark_from_string(format_str));
        td = fmt->open_trace(path, O_RDONLY,
@@ -69,7 +70,8 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path,
        if (!td) {
                fprintf(stderr, "[error] [Context] Cannot open_trace of the format %s .\n\n",
                                path);
-               return -1;
+               ret = -1;
+               goto end;
        }
 
        /* Create an handle for the trace */
@@ -77,7 +79,8 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path,
        if (handle < 0) {
                fprintf(stderr, "[error] [Context] Creating trace handle %s .\n\n",
                                path);
-               return -1;
+               ret = -1;
+               goto end;
        }
        handle->format = fmt;
        handle->td = td;
@@ -88,7 +91,11 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path,
        g_hash_table_insert(ctx->trace_handles,
                (gpointer) (unsigned long) handle->id,
                handle);
-       return trace_collection_add(ctx->tc, td);
+       ret = trace_collection_add(ctx->tc, td);
+       if (ret == 0)
+               return handle->id;
+end:
+       return ret;
 }
 
 void bt_context_remove_trace(struct bt_context *ctx, int handle_id)
This page took 0.034495 seconds and 4 git commands to generate.