Skip to content

Commit

Permalink
g_list_free swigged GList* arguments
Browse files Browse the repository at this point in the history
used by xaccQueryAddAccountMatch and gnc_price_list_equal. SplitList
TransList LotList and CommodityList are not currently being passed as
arguments.
  • Loading branch information
christopherlam committed Oct 16, 2021
1 parent df2d9bc commit 16184da
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion bindings/engine.i
Expand Up @@ -67,7 +67,6 @@ GLIST_HELPER_INOUT(TransList, SWIGTYPE_p_Transaction);
GLIST_HELPER_INOUT(LotList, SWIGTYPE_p_GNCLot);
GLIST_HELPER_INOUT(AccountList, SWIGTYPE_p_Account);
GLIST_HELPER_INOUT(PriceList, SWIGTYPE_p_GNCPrice);
// TODO: free PriceList?
GLIST_HELPER_INOUT(CommodityList, SWIGTYPE_p_gnc_commodity);

%typemap(newfree) gchar * "g_free($1);"
Expand Down Expand Up @@ -103,6 +102,13 @@ static const GncGUID * gncBudgetGetGUID(GncBudget *x)
{ return qof_instance_get_guid(QOF_INSTANCE(x)); }
%}

%typemap (freearg) AccountList * "g_list_free ($1);"
%typemap (freearg) PriceList * "g_list_free ($1);"
%typemap (freearg) SplitList * "g_list_free ($1);"
%typemap (freearg) TransList * "g_list_free ($1);"
%typemap (freearg) LotList * "g_list_free ($1);"
%typemap (freearg) CommodityList * "g_list_free ($1);"

/* NB: The object ownership annotations should already cover all the
functions currently used in guile, but not all the functions that are
wrapped. So, we should contract the interface to wrap only the used
Expand Down

0 comments on commit 16184da

Please sign in to comment.