r14611 - gnucash/trunk/src - Various whitespace, line-wrap, and comment changes.

Derek Atkins warlord at MIT.EDU
Sun Aug 13 16:59:03 EDT 2006


Chris,

Chris Shoemaker <chris at cvs.gnucash.org> writes:

> +/* FIXME: These macros are not consistent with the type name */
>  #define GNC_IS_TRANS(obj)  (QOF_CHECK_TYPE((obj), GNC_ID_TRANS))
>  #define GNC_TRANS(obj)     (QOF_CHECK_CAST((obj), GNC_ID_TRANS, Transaction))

I'm a little confused about what you mean by this new comment.
How are these macros now consistent with the type name?
Are you complaining that "TRANS" != "Transaction"?

I'll point out that the TRANS shortcut is used many places already.
Did you feel that this was confusing to somebody?

-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