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 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 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