build failure in src/calculation/test

David Jafferian - Sun Microsystems david.jafferian at east.sun.com
Sun Mar 13 19:36:38 EST 2005


Hi Derek -

The problem persisted after changing "static inline" to G_INLINE_FUNC.
Only changing "inline" to "__inline__" removed the problem.
Yours was the only reply I got.

- David Jafferian

Derek Atkins writes:
 > Interesting.  What happens if you change "static inline" to G_INLINE_FUNC?
 > Does it still work then?
 > 
 > Thanks,
 > 
 > -derek
 > 
 > PS: Unified diff is usually preferred to context diff.
 > 
 > David Jafferian - Sun Microsystems <david.jafferian at east.sun.com> writes:
 > 
 > > Hi -
 > >
 > > I wasn't able to find this problem in mailing list archives.
 > >
 > [snip]
 > >  =>diff -c gnc-numeric.h.orig gnc-numeric.h.new
 > > *** gnc-numeric.h.orig  Sun Oct 31 20:37:09 2004
 > > --- gnc-numeric.h.new   Tue Feb  8 16:19:04 2005
 > > ***************
 > > *** 417,423 ****
 > >    * Shortcut for common case: gnc_numeric_add(a, b, GNC_DENOM_AUTO,
 > >    *                        GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > >    */
 > > ! static inline
 > >   gnc_numeric gnc_numeric_add_fixed(gnc_numeric a, gnc_numeric b) {
 > >      return gnc_numeric_add(a, b, GNC_DENOM_AUTO,
 > >                            GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > > --- 417,423 ----
 > >    * Shortcut for common case: gnc_numeric_add(a, b, GNC_DENOM_AUTO,
 > >    *                        GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > >    */
 > > ! static __inline__
 > >   gnc_numeric gnc_numeric_add_fixed(gnc_numeric a, gnc_numeric b) {
 > >      return gnc_numeric_add(a, b, GNC_DENOM_AUTO,
 > >                            GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > > ***************
 > > *** 427,433 ****
 > >    * Shortcut for most common case: gnc_numeric_sub(a, b, GNC_DENOM_AUTO,
 > >    *                        GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > >    */
 > > ! static inline 
 > >   gnc_numeric gnc_numeric_sub_fixed(gnc_numeric a, gnc_numeric b) {
 > >     return gnc_numeric_sub(a, b, GNC_DENOM_AUTO,
 > >                            GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > > --- 427,433 ----
 > >    * Shortcut for most common case: gnc_numeric_sub(a, b, GNC_DENOM_AUTO,
 > >    *                        GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > >    */
 > > ! static __inline__ 
 > >   gnc_numeric gnc_numeric_sub_fixed(gnc_numeric a, gnc_numeric b) {
 > >     return gnc_numeric_sub(a, b, GNC_DENOM_AUTO,
 > >                            GNC_HOW_DENOM_FIXED | GNC_HOW_RND_NEVER);
 > >
 > > Apparently there is a reason why __inline__ is recommended for header
 > > files.  Can anyone offer an explanation ?
 > >
 > > -- 
 > > David N. Jafferian
 > > Solaris Kernel and Drivers
 > > Product Technical Support
 > > Sun Microsystems, Inc.
 > >
 > > _______________________________________________
 > > gnucash-devel mailing list
 > > gnucash-devel at gnucash.org
 > > https://lists.gnucash.org/mailman/listinfo/gnucash-devel
 > >
 > >
 > 
 > -- 
 >        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
 > _______________________________________________
 > 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