[Gnucash-changes] r13390 - gnucash/trunk/lib/libqof/qof - Implement QOF_COMMIT_EDIT_PART2 as a function instead of a macro.

[Derek Atkins]

Chris Shoemaker <c.shoemaker at cox.net> writes:

> :) I'm betting you've never been caught be a bug involving
> control-flow hidden behind a macro.  In my experience, it only takes
> once until someone decides that they will never, ever write one.

Nope, I've not.

> <aside>
> BTW, this is only slightly less horrible than macros that use local
> variables from their calling context.  The general rule is, "If it
> looks like a function, it sure better behave like a function." (*)
> </aside>

See, I never make the assumption that Macro == Function, even
if it looks like one.

> -chris

-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