Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Version 0.5 (protocol not changed; ABI changed)
[simgrid.git] / src / gras / DataDesc / ddt_exchange.c
index 05905c3..fb5b7b4 100644 (file)
@@ -9,9 +9,17 @@
    under the terms of the license (GNU LGPL) which comes with this package. */
 
 #include "DataDesc/datadesc_private.h"
+#include "Transport/transport_interface.h" /* gras_trp_chunk_send/recv */
 
 GRAS_LOG_NEW_DEFAULT_SUBCATEGORY(exchange,datadesc);
 
+#undef DETECT_CYCLE
+/* CRUDE HACK to turn all cycle detection of */
+#ifndef DETECT_CYCLE
+#define gras_dict_get_ext(a,b,c,d) mismatch_error
+#define gras_dict_set_ext(a,b,c,d,e) no_error
+#endif
+
 const char *gras_datadesc_cat_names[9] = { 
   "undefined", 
   "scalar", "struct", "union", "ref", "array", "ignored",
@@ -19,25 +27,25 @@ const char *gras_datadesc_cat_names[9] = {
 
 static gras_datadesc_type_t *int_type = NULL;
 static gras_datadesc_type_t *pointer_type = NULL;    
-static gras_error_t gras_dd_send_int(gras_socket_t *sock,             int  i);
-static gras_error_t gras_dd_recv_int(gras_socket_t *sock, int r_arch, int *i);
+static inline gras_error_t gras_dd_send_int(gras_socket_t *sock,             int  i);
+static inline gras_error_t gras_dd_recv_int(gras_socket_t *sock, int r_arch, int *i);
 
-static gras_error_t
+static inline gras_error_t
 gras_dd_alloc_ref(gras_dict_t *refs,  long int     size,
                  char       **r_ref, long int     r_len,
                  char       **l_ref);
-static int 
+static inline in
 gras_dd_is_r_null(char **r_ptr, long int length);
 
 static gras_error_t 
 gras_datadesc_send_rec(gras_socket_t        *sock,
-                      gras_dd_cbps_t       *state,
+                      gras_cbps_t       *state,
                       gras_dict_t          *refs,
                       gras_datadesc_type_t *type, 
                       char                 *data);
 static gras_error_t
 gras_datadesc_recv_rec(gras_socket_t        *sock, 
-                      gras_dd_cbps_t       *state,
+                      gras_cbps_t       *state,
                       gras_dict_t          *refs,
                       gras_datadesc_type_t *type,
                       int                   r_arch,
@@ -47,7 +55,7 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
                       int                   subsize);
 
 
-static gras_error_t
+static inline gras_error_t
 gras_dd_send_int(gras_socket_t *sock,int i) {
 
   if (!int_type) {
@@ -59,7 +67,7 @@ gras_dd_send_int(gras_socket_t *sock,int i) {
   return gras_trp_chunk_send(sock, (char*)&i, int_type->size[GRAS_THISARCH]);
 }
 
-static gras_error_t
+static inline gras_error_t
 gras_dd_recv_int(gras_socket_t *sock, int r_arch, int *i) {
   gras_error_t errcode;
 
@@ -70,12 +78,14 @@ gras_dd_recv_int(gras_socket_t *sock, int r_arch, int *i) {
 
   if (int_type->size[GRAS_THISARCH] >= int_type->size[r_arch]) {
     TRY(gras_trp_chunk_recv(sock, (char*)i, int_type->size[r_arch]));
-    TRY(gras_dd_convert_elm(int_type,1,r_arch, i,i));
+    if (r_arch != GRAS_THISARCH)
+      TRY(gras_dd_convert_elm(int_type,1,r_arch, i,i));
   } else {
     void *ptr = NULL;
     ptr = malloc((size_t)int_type->size[r_arch]);
     TRY(gras_trp_chunk_recv(sock, (char*)ptr, int_type->size[r_arch]));
-    TRY(gras_dd_convert_elm(int_type,1,r_arch, ptr,i));
+    if (r_arch != GRAS_THISARCH)
+      TRY(gras_dd_convert_elm(int_type,1,r_arch, ptr,i));
     free(ptr);
   }
   DEBUG1("recv_int(%d)",*i);
@@ -88,7 +98,7 @@ gras_dd_recv_int(gras_socket_t *sock, int r_arch, int *i) {
  *       of 'length' bytes set to 0.
  * FIXME: Check in configure?
  */
-static int 
+static inline in
 gras_dd_is_r_null(char **r_ptr, long int length) {
   int i;
 
@@ -101,25 +111,26 @@ gras_dd_is_r_null(char **r_ptr, long int length) {
   return 1;
 }
 
-static gras_error_t
+static inline gras_error_t
 gras_dd_alloc_ref(gras_dict_t *refs,
                  long int     size,
                  char       **r_ref,
                  long int     r_len, /* pointer_type->size[r_arch] */
                  char       **l_ref) {
   char *l_data = NULL;
-  gras_error_t errcode;
 
-  gras_assert1(size>0,"Cannot allocate %d bytes!", size);
+  gras_assert1(size>0,"Cannot allocate %ld bytes!", size);
   if (! (l_data = malloc((size_t)size)) )
     RAISE_MALLOC;
 
   *l_ref = l_data;
   DEBUG2("l_data=%p, &l_data=%p",l_data,&l_data);
 
-  DEBUG3("alloc_ref: r_ref=%p; *r_ref=%p, r_len=%d",
+  DEBUG3("alloc_ref: r_ref=%p; *r_ref=%p, r_len=%ld",
         r_ref, r_ref?*r_ref:NULL, r_len);
+#ifdef DETECT_CYCLE
   if (r_ref && !gras_dd_is_r_null( r_ref, r_len)) {
+    gras_error_t errcode;
     void *ptr = malloc(sizeof(void *));
     if (!ptr)
       RAISE_MALLOC;
@@ -130,6 +141,7 @@ gras_dd_alloc_ref(gras_dict_t *refs,
 
     TRY(gras_dict_set_ext(refs,(const char *) r_ref, r_len, ptr, free));
   }
+#endif
   return no_error;
 }
 
@@ -164,21 +176,21 @@ int gras_datadesc_type_cmp(const gras_datadesc_type_t *d1,
 
   for (cpt=0; cpt<gras_arch_count; cpt++) {
     if (d1->size[cpt] != d2->size[cpt]) {
-      DEBUG5("ddt_cmp: %s->size=%d  !=  %s->size=%d (on %s)",
+      DEBUG5("ddt_cmp: %s->size=%ld  !=  %s->size=%ld (on %s)",
             d1->name,d1->size[cpt],d2->name,d2->size[cpt],
             gras_arches[cpt].name);
       return d1->size[cpt] >  d2->size[cpt] ? 1 : -1;
     }
 
     if (d1->alignment[cpt] != d2->alignment[cpt]) {
-      DEBUG5("ddt_cmp: %s->alignment=%d  !=  %s->alignment=%d (on %s)",
+      DEBUG5("ddt_cmp: %s->alignment=%ld  !=  %s->alignment=%ld (on %s)",
             d1->name,d1->alignment[cpt],d2->name,d2->alignment[cpt],
             gras_arches[cpt].name);
       return d1->alignment[cpt] > d2->alignment[cpt] ? 1 : -1;
     }
 
     if (d1->aligned_size[cpt] != d2->aligned_size[cpt]) {
-      DEBUG5("ddt_cmp: %s->aligned_size=%d  !=  %s->aligned_size=%d (on %s)",
+      DEBUG5("ddt_cmp: %s->aligned_size=%ld  !=  %s->aligned_size=%ld (on %s)",
             d1->name,d1->aligned_size[cpt],d2->name,d2->aligned_size[cpt],
             gras_arches[cpt].name);
       return d1->aligned_size[cpt] > d2->aligned_size[cpt] ? 1 : -1;
@@ -192,16 +204,16 @@ int gras_datadesc_type_cmp(const gras_datadesc_type_t *d1,
     return d1->category_code > d2->category_code ? 1 : -1;
   }
 
-  if (d1->pre != d2->pre) {
-    DEBUG4("ddt_cmp: %s->pre=%p  !=  %s->pre=%p",
-          d1->name,d1->pre, d2->name,d2->pre);
-    return d1->pre > d2->pre ? 1 : -1;
+  if (d1->send != d2->send) {
+    DEBUG4("ddt_cmp: %s->send=%p  !=  %s->send=%p",
+          d1->name,d1->send, d2->name,d2->send);
+    return d1->send > d2->send ? 1 : -1;
   }
 
-  if (d1->post != d2->post) {
-    DEBUG4("ddt_cmp: %s->post=%p  !=  %s->post=%p",
-          d1->name,d1->post, d2->name,d2->post);
-    return d1->post > d2->post ? 1 : -1;
+  if (d1->recv != d2->recv) {
+    DEBUG4("ddt_cmp: %s->recv=%p  !=  %s->recv=%p",
+          d1->name,d1->recv, d2->name,d2->recv);
+    return d1->recv > d2->recv ? 1 : -1;
   }
 
   switch (d1->category_code) {
@@ -224,8 +236,8 @@ int gras_datadesc_type_cmp(const gras_datadesc_type_t *d1,
     gras_dynar_foreach(d1->category.struct_data.fields, cpt, field1) {
       
       gras_dynar_get(d2->category.struct_data.fields, cpt, &field2);
-      gras_datadesc_by_id(field1->code,&field_desc_1); /* FIXME: errcode ignored */
-      gras_datadesc_by_id(field2->code,&field_desc_2);
+      field_desc_1 = field1->type;
+      field_desc_2 = field2->type;
       ret = gras_datadesc_type_cmp(field_desc_1,field_desc_2);
       if (ret) {
        DEBUG6("%s->field[%d]=%s != %s->field[%d]=%s",
@@ -250,8 +262,8 @@ int gras_datadesc_type_cmp(const gras_datadesc_type_t *d1,
     gras_dynar_foreach(d1->category.union_data.fields, cpt, field1) {
       
       gras_dynar_get(d2->category.union_data.fields, cpt, field2);
-      gras_datadesc_by_id(field1->code,&field_desc_1); /* FIXME: errcode ignored */
-      gras_datadesc_by_id(field2->code,&field_desc_2);
+      field_desc_1 = field1->type;
+      field_desc_2 = field2->type;
       ret = gras_datadesc_type_cmp(field_desc_1,field_desc_2);
       if (ret)
        return ret;
@@ -264,13 +276,13 @@ int gras_datadesc_type_cmp(const gras_datadesc_type_t *d1,
     if (d1->category.ref_data.selector != d2->category.ref_data.selector) 
       return d1->category.ref_data.selector > d2->category.ref_data.selector ? 1 : -1;
     
-    if (d1->category.ref_data.code != d2->category.ref_data.code) 
-      return d1->category.ref_data.code > d2->category.ref_data.code ? 1 : -1;
+    if (d1->category.ref_data.type != d2->category.ref_data.type) 
+      return d1->category.ref_data.type > d2->category.ref_data.type ? 1 : -1;
     break;
     
   case e_gras_datadesc_type_cat_array:
-    if (d1->category.array_data.code != d2->category.array_data.code) 
-      return d1->category.array_data.code > d2->category.array_data.code ? 1 : -1;
+    if (d1->category.array_data.type != d2->category.array_data.type) 
+      return d1->category.array_data.type > d2->category.array_data.type ? 1 : -1;
     
     if (d1->category.array_data.fixed_size != d2->category.array_data.fixed_size) 
       return d1->category.array_data.fixed_size > d2->category.array_data.fixed_size ? 1 : -1;
@@ -305,7 +317,7 @@ gras_error_t gras_datadesc_cpy(gras_datadesc_type_t *type,
 
 static gras_error_t 
 gras_datadesc_send_rec(gras_socket_t        *sock,
-                      gras_dd_cbps_t       *state,
+                      gras_cbps_t       *state,
                       gras_dict_t          *refs,
                       gras_datadesc_type_t *type, 
                       char                 *data) {
@@ -317,8 +329,8 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
   VERB2("Send a %s (%s)", 
        type->name, gras_datadesc_cat_names[type->category_code]);
 
-  if (type->pre) {
-    type->pre(state,type,data);
+  if (type->send) {
+    type->send(state,data);
   }
 
   switch (type->category_code) {
@@ -330,30 +342,26 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
     gras_dd_cat_struct_t  struct_data;
     gras_dd_cat_field_t  *field;
     char                 *field_data;
-
-   gras_assert1(type->category.struct_data.closed,
-               "Please call gras_datadesc_declare_array_close on %s before sending it",
-               type->name);
+    
     struct_data = type->category.struct_data;
+    gras_assert1(struct_data.closed,
+      "Please call gras_datadesc_declare_struct_close on %s before sending it",
+               type->name);
     VERB1(">> Send all fields of the structure %s",type->name);
     gras_dynar_foreach(struct_data.fields, cpt, field) {
       field_data = data;
       field_data += field->offset[GRAS_THISARCH];
       
-      errcode=gras_datadesc_by_id(field->code, &sub_type);
-      if (errcode != no_error) 
-       RAISE4(errcode,
-              "Got %s while searching for the sub type %d, #%d of %s",
-              gras_error_name(errcode),field->code,cpt,type->name);
+      sub_type = field->type;
       
       if (field->pre)
-       field->pre(state,sub_type,field_data);
+       field->pre(state,field_data);
       
       VERB1("Send field %s",field->name);
       TRY(gras_datadesc_send_rec(sock,state,refs,sub_type, field_data));
       
       if (field->post)
-       field->post(state,sub_type,field_data);
+       field->post(state,field_data);
     }
     VERB1("<< Sent all fields of the structure %s", type->name);
     
@@ -362,7 +370,7 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
 
   case e_gras_datadesc_type_cat_union: {
     gras_dd_cat_union_t    union_data;
-    gras_dd_cat_field_t   *field;
+    gras_dd_cat_field_t   *field=NULL;
     int                    field_num;
     
     union_data = type->category.union_data;
@@ -371,7 +379,7 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
                "Please call gras_datadesc_declare_union_close on %s before sending it",
                type->name);
     /* retrieve the field number */
-    field_num = union_data.selector(state, type, data);
+    field_num = union_data.selector(state, data);
     
     gras_assert1(field_num > 0,
                 "union field selector of %s gave a negative value", 
@@ -386,33 +394,34 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
     
     /* Send the content */
     gras_dynar_get(union_data.fields, field_num, field);
-    TRY(gras_datadesc_by_id(field->code, &sub_type));
+    sub_type = field->type;
     
     if (field->pre)
-      field->pre(state,sub_type,data);
+      field->pre(state,data);
     
     TRY(gras_datadesc_send_rec(sock,state,refs, sub_type, data));
       
     if (field->post)
-      field->post(state,sub_type,data);
+      field->post(state,data);
     
     break;
   }
     
   case e_gras_datadesc_type_cat_ref: {
     gras_dd_cat_ref_t      ref_data;
-    int                    ref_code;
 
     void                 **ref=(void**)data;
+#ifdef DETECT_CYCLE
     void *dummy;
+#endif
     
     ref_data = type->category.ref_data;
     
     /* Detect the referenced type and send it to peer if needed */
-    ref_code = ref_data.code;
-    if (ref_code < 0) {
-      ref_code = ref_data.selector(state,type,data);
-      TRY(gras_dd_send_int(sock, ref_code));
+    sub_type = ref_data.type;
+    if (sub_type == NULL) {
+      sub_type = (*ref_data.selector)(state,data);
+      TRY(gras_dd_send_int(sock, sub_type->code));
     }
     
     /* Send the actual value of the pointer for cycle handling */
@@ -433,7 +442,6 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
     if (errcode == mismatch_error) {
       VERB1("Sending data referenced at %p", *ref);
       TRY(gras_dict_set_ext(refs, (char*)ref, sizeof(void*), ref, NULL));
-      TRY(gras_datadesc_by_id(ref_code, &sub_type));
       TRY(gras_datadesc_send_rec(sock,state,refs, sub_type, *ref));
       
     } else if (errcode == no_error) {
@@ -456,17 +464,17 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
     /* determine and send the element count */
     count = array_data.fixed_size;
     if (count <= 0) {
-      count = array_data.dynamic_size(state,type,data);
+      count = array_data.dynamic_size(state,data);
       gras_assert1(count >=0,
                   "Invalid (negative) array size for type %s",type->name);
       TRY(gras_dd_send_int(sock, count));
     }
     
     /* send the content */
-    TRY(gras_datadesc_by_id(array_data.code, &sub_type));
+    sub_type = array_data.type;
     elm_size = sub_type->aligned_size[GRAS_THISARCH];
     if (sub_type->category_code == e_gras_datadesc_type_cat_scalar) {
-      VERB1("Array of %d scalars, send it in one shoot",count);
+      VERB1("Array of %ld scalars, send it in one shot",count);
       TRY(gras_trp_chunk_send(sock, data, 
                              sub_type->aligned_size[GRAS_THISARCH] * count));
     } else {
@@ -482,10 +490,6 @@ gras_datadesc_send_rec(gras_socket_t        *sock,
     gras_assert0(0, "Invalid type");
   }
 
-  if (type->post) {
-    type->post(state,type,data);
-  }
-
   return no_error;
 }
 
@@ -500,16 +504,16 @@ gras_error_t gras_datadesc_send(gras_socket_t *sock,
                                void *src) {
 
   gras_error_t errcode;
-  gras_dd_cbps_t *state = NULL;
+  gras_cbps_t *state = NULL;
   gras_dict_t    *refs; /* all references already sent */
  
   TRY(gras_dict_new(&refs));
-  TRY(gras_dd_cbps_new(&state));
+  TRY(gras_cbps_new(&state));
 
   errcode = gras_datadesc_send_rec(sock,state,refs,type,(char*)src);
 
   gras_dict_free(&refs);
-  gras_dd_cbps_free(&state);
+  gras_cbps_free(&state);
 
   return errcode;
 }
@@ -528,7 +532,7 @@ gras_error_t gras_datadesc_send(gras_socket_t *sock,
  */
 gras_error_t
 gras_datadesc_recv_rec(gras_socket_t        *sock, 
-                      gras_dd_cbps_t       *state,
+                      gras_cbps_t          *state,
                       gras_dict_t          *refs,
                       gras_datadesc_type_t *type,
                       int                   r_arch,
@@ -548,12 +552,14 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
   case e_gras_datadesc_type_cat_scalar:
     if (type->size[GRAS_THISARCH] == type->size[r_arch]) {
       TRY(gras_trp_chunk_recv(sock, (char*)l_data, type->size[r_arch]));
-      TRY(gras_dd_convert_elm(type,1,r_arch, l_data,l_data));
+      if (r_arch != GRAS_THISARCH)
+       TRY(gras_dd_convert_elm(type,1,r_arch, l_data,l_data));
     } else {
       void *ptr = NULL;
       ptr = malloc((size_t)type->size[r_arch]);
       TRY(gras_trp_chunk_recv(sock, (char*)ptr, type->size[r_arch]));
-      TRY(gras_dd_convert_elm(type,1,r_arch, ptr,l_data));
+      if (r_arch != GRAS_THISARCH)
+       TRY(gras_dd_convert_elm(type,1,r_arch, ptr,l_data));
       free(ptr);
     }
     break;
@@ -571,7 +577,7 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
     gras_dynar_foreach(struct_data.fields, cpt, field) {
       char                 *field_data = l_data + field->offset[GRAS_THISARCH];
 
-      TRY(gras_datadesc_by_id(field->code, &sub_type));
+      sub_type = field->type;
 
       TRY(gras_datadesc_recv_rec(sock,state,refs, sub_type,
                                 r_arch,NULL,0,
@@ -584,7 +590,7 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
 
   case e_gras_datadesc_type_cat_union: {
     gras_dd_cat_union_t    union_data;
-    gras_dd_cat_field_t   *field;
+    gras_dd_cat_field_t   *field=NULL;
     int                    field_num;
 
     union_data = type->category.union_data;
@@ -604,7 +610,7 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
     
     /* Recv the content */
     gras_dynar_get(union_data.fields, field_num, field);
-    TRY(gras_datadesc_by_id(field->code, &sub_type));
+    sub_type = field->type;
     
     TRY(gras_datadesc_recv_rec(sock,state,refs, sub_type,
                               r_arch,NULL,0,
@@ -616,14 +622,16 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
     char             **r_ref = NULL;
     char             **l_ref = NULL;
     gras_dd_cat_ref_t  ref_data;
-    int                ref_code;
     
     ref_data = type->category.ref_data;
 
     /* Get the referenced type locally or from peer */
-    ref_code = ref_data.code;
-    if (ref_code < 0) 
+    sub_type = ref_data.type;
+    if (sub_type == NULL) {
+      int ref_code;
       TRY(gras_dd_recv_int(sock, r_arch, &ref_code));
+      TRY(gras_datadesc_by_id(ref_code, &sub_type));
+    }
 
     /* Get the actual value of the pointer for cycle handling */
     if (!pointer_type) {
@@ -653,8 +661,6 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
       int subsubcount = -1;
       void *l_referenced=NULL;
 
-      TRY(gras_datadesc_by_id(ref_code, &sub_type));
-      
       VERB2("Receiving a ref to '%s', remotely @%p",
            sub_type->name, *(void**)r_ref);
       if (sub_type->category_code == e_gras_datadesc_type_cat_array) {
@@ -667,7 +673,7 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
        if (subsubcount < 0)
          TRY(gras_dd_recv_int(sock, r_arch, &subsubcount));
 
-       TRY(gras_datadesc_by_id(array_data.code, &subsub_type));
+       subsub_type = array_data.type;
 
 
        TRY(gras_dd_alloc_ref(refs,
@@ -718,19 +724,21 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
             "Invalid (negative) array size for type %s",type->name);
 
     /* receive the content */
-    TRY(gras_datadesc_by_id(array_data.code, &sub_type));
+    sub_type = array_data.type;
     if (sub_type->category_code == e_gras_datadesc_type_cat_scalar) {
       VERB1("Array of %d scalars, get it in one shoot", count);
       if (sub_type->aligned_size[GRAS_THISARCH] >= 
          sub_type->aligned_size[r_arch]) {
        TRY(gras_trp_chunk_recv(sock, (char*)l_data, 
                                sub_type->aligned_size[r_arch] * count));
-       TRY(gras_dd_convert_elm(sub_type,count,r_arch, l_data,l_data));
+       if (r_arch != GRAS_THISARCH)
+         TRY(gras_dd_convert_elm(sub_type,count,r_arch, l_data,l_data));
       } else {
        ptr = malloc((size_t)sub_type->aligned_size[r_arch] * count);
        TRY(gras_trp_chunk_recv(sock, (char*)ptr, 
                                sub_type->size[r_arch] * count));
-       TRY(gras_dd_convert_elm(sub_type,count,r_arch, ptr,l_data));
+       if (r_arch != GRAS_THISARCH)
+         TRY(gras_dd_convert_elm(sub_type,count,r_arch, ptr,l_data));
        free(ptr);
       }
     } else {
@@ -752,6 +760,9 @@ gras_datadesc_recv_rec(gras_socket_t        *sock,
     gras_assert0(0, "Invalid type");
   }
   
+  if (type->recv)
+    type->recv(state,l_data);
+
   return no_error;
 }
 
@@ -769,18 +780,18 @@ gras_datadesc_recv(gras_socket_t *sock,
                   void *dst) {
 
   gras_error_t errcode;
-  gras_dd_cbps_t *state = NULL; /* callback persistent state */
+  gras_cbps_t *state = NULL; /* callback persistent state */
   gras_dict_t    *refs;         /* all references already sent */
 
   TRY(gras_dict_new(&refs));
-  TRY(gras_dd_cbps_new(&state));
+  TRY(gras_cbps_new(&state));
 
   errcode = gras_datadesc_recv_rec(sock, state, refs, type, 
                                   r_arch, NULL, 0,
                                   (char *) dst,-1);
 
   gras_dict_free(&refs);
-  gras_dd_cbps_free(&state);
+  gras_cbps_free(&state);
 
   return errcode;
 }