On May 19, 2011, at 3:07 AM, Richard Shann <[email protected]> wrote:

Jeremiah
@@ -506,6 +516,7 @@ static SCM scheme_activate_menu_item(SCM menupath) {
    item = scm_to_locale_string(menupath);
    if(item) {
      gboolean ret = activate_action(item) ? TRUE:FALSE;
+      g_free(item);
      return SCM_BOOL(ret);
    }
  }

Is this one right? scm_to_locale_string() cannot be returning something allocated with g_malloc can it (since scheme lib knows nothing of glib)?
So it must be malloc, in which case free(item) not g_free()???

Ok. I see i was cinfused because some of these were declared as gchar. Maybe it would be clearer if i have the variables be char instead because the scm_to_locale_string is returning char * and not gchar *.

other examples of the same are further down in this patch...

I don't see any patch.

Jeremiah

Richard



_______________________________________________
Denemo-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/denemo-devel

_______________________________________________
Denemo-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/denemo-devel

Reply via email to