[Gnucash-changes] CashUtil Sync: business backend

Derek Atkins warlord at MIT.EDU
Wed Aug 17 08:51:34 EDT 2005


Neil Williams <codehelp at cvs.gnucash.org> writes:

>      xmlAddChild(ret, gnc_owner_to_dom_tree (job_owner_string,
> -					    gncJobGetOwner (job)));
> +					    (GncOwner*)gncJobGetOwner (job)));

Why this cast?  gncJobGetOwner() returns a GncOwner* so you
shouldn't need to cast it.    (See what I said before about
casts being bad)...

-derek

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available


More information about the gnucash-devel mailing list