r18303 - gnucash/trunk/src - Use SWIG properly to wrap functions to free strings which need to be freed by the caller.
Phil Longstaff
plongstaff at rogers.com
Wed Sep 9 14:48:21 EDT 2009
Yes, I do need to revert that change. Thanks. In fact, gnc_account_get_full_name() just calls xaccAccountGetFullName(). Any reason I shouldn't go through all of the code and change xaccAccountXXX to gnc_account_XXX (and Trans and SchedXaction and Split and ...) and give us standardized naming for 2.4?
Phil
________________________________
From: Christian Stimming <stimming at tuhh.de>
To: gnucash-devel at gnucash.org
Sent: Wednesday, September 9, 2009 2:36:53 PM
Subject: Re: r18303 - gnucash/trunk/src - Use SWIG properly to wrap functions to free strings which need to be freed by the caller.
Dear Phil,
thanks a lot for finding this out and implementing it.
Doesn't this also mean r18295, the change in gnc_account_get_full_name () in
src/app-utils/gnc-ui-util.c, needs to be reverted as well? Thanks!
Christian
Am Mittwoch, 9. September 2009 01:48 schrieb Phil Longstaff:
> Author: plongstaff
> Date: 2009-09-08 19:48:42 -0400 (Tue, 08 Sep 2009)
> New Revision: 18303
> Trac: http://svn.gnucash.org/trac/changeset/18303
>
> Modified:
> gnucash/trunk/src/engine/Account.c
> gnucash/trunk/src/engine/Account.h
> gnucash/trunk/src/engine/engine.i
> gnucash/trunk/src/html/gnc-html.c
> gnucash/trunk/src/html/gnc-html.h
> gnucash/trunk/src/html/gnc-html.i
> gnucash/trunk/src/report/report-gnome/gnc-plugin-page-report.c
> gnucash/trunk/src/report/report-gnome/window-report.c
> Log:
> Use SWIG properly to wrap functions to free strings which need to be freed
> by the caller.
>
>
> _______________________________________________
> gnucash-patches mailing list
> gnucash-patches at gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-patches
_______________________________________________
gnucash-devel mailing list
gnucash-devel at gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel
More information about the gnucash-devel
mailing list