322110 - Build fails in nsWebBrowserPersist.cpp due to calling an inline(d) function that has not yet been declared

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Mitch]

User-Agent:       Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.9a1) Gecko/20051207 SeaMonkey/1.5a
Build Identifier: Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.9a1) Gecko/20051207 SeaMonkey/1.5a

Compiling with Sun Forte 

/share/lang/solaris/sparc/7.0/SUNWspro/bin/CC -V
CC: Forte Developer 7 C++ 5.4 2002/03/09

I get the following build error

Building deps for nsWebBrowserPersist.cpp
/share/lang/solaris/sparc/7.0/SUNWspro/bin/CC -I/home/mitch/bld/support/include -o nsWebBrowserPersist.o -c  -DMOZILLA_INTERNAL_API -DOSTYPE=\"SunOS5\" -DOSARCH=\"SunOS\" -DBUILD_ID=2006010209  -I../../../../dist/include/xpcom -I../../../../dist/include/string -I../../../../dist/include/dom -I../../../../dist/include/content -I../../../../dist/include/layout -I../../../../dist/include/necko -I../../../../dist/include/widget -I../../../../dist/include/exthandler -I../../../../dist/include/uriloader -I../../../../dist/include/mimetype -I../../../../dist/include/intl -I../../../../dist/include/unicharutil -I../../../../dist/include/docshell -I../../../../dist/include/shistory -I../../../../dist/include/imglib2 -I../../../../dist/include/gfx -I../../../../dist/include/webbrowserpersist -I../../../../dist/include -I../../../../dist/include/nspr    -I../../../../dist/sdk/include    -KPIC   -I/home/mitch/bld/support/include -xbuiltin=%all -features=tmplife -norunpath -mt  -DNDEBUG -DTRIMMED -O   -DMOZILLA_VERSION=\"1.9a1\" -DMOZILLA_VERSION_U=1.9a1 -DSOLARIS=1 -DNSCAP_DISABLE_DEBUG_PTR_TYPES=1 -DD_INO=d_ino -DSTDC_HEADERS=1 -DHAVE_ST_BLKSIZE=1 -DHAVE_SIGINFO_T=1 -DHAVE_INT16_T=1 -DHAVE_INT32_T=1 -DHAVE_INT64_T=1 -DHAVE_UINT=1 -DHAVE_UINT_T=1 -DHAVE_UINT16_T=1 -DHAVE_DIRENT_H=1 -DHAVE_SYS_BYTEORDER_H=1 -DHAVE_MEMORY_H=1 -DHAVE_UNISTD_H=1 -DHAVE_NL_TYPES_H=1 -DHAVE_MALLOC_H=1 -DHAVE_X11_XKBLIB_H=1 -DHAVE_SYS_STATVFS_H=1 -DHAVE_SYS_STATFS_H=1 -DHAVE_LIBM=1 -DHAVE_LIBDL=1 -DHAVE_LIBSOCKET=1 -DFUNCPROTO=15 -DHAVE_XSHM=1 -D_REENTRANT=1 -DHAVE_RANDOM=1 -DHAVE_STRERROR=1 -DHAVE_LCHOWN=1 -DHAVE_FCHMOD=1 -DHAVE_SNPRINTF=1 -DHAVE_MEMMOVE=1 -DHAVE_RINT=1 -DHAVE_STAT64=1 -DHAVE_LSTAT64=1 -DHAVE_FLOCKFILE=1 -DHAVE_LOCALTIME_R=1 -DHAVE_STRTOK_R=1 -DHAVE_LANGINFO_CODESET=1 -DVA_COPY=va_copy -DHAVE_VA_COPY=1 -DHAVE_I18N_LC_MESSAGES=1 -DMOZ_DEFAULT_TOOLKIT=\"gtk\" -DMOZ_WIDGET_GTK=1 -DMOZ_ENABLE_XREMOTE=1 -DMOZ_X11=1 -DMOZ_SUITE=1 -DMOZ_BUILD_APP=suite -DMOZ_DISTRIBUTION_ID=\"org.mozilla\" -DMOZ_ENABLE_COREXFONTS=1 -DMOZ_EXTRA_X11CONVERTERS=1 -DOJI=1 -DIBMBIDI=1 -DMOZ_VIEW_SOURCE=1 -DACCESSIBILITY=1 -DMOZ_XPINSTALL=1 -DMOZ_JSLOADER=1 -DNS_PRINTING=1 -DNS_PRINT_PREVIEW=1 -DMOZ_XTF=1 -DMOZ_MATHML=1 -DMOZ_UPDATE_CHANNEL=default -DMOZ_STORAGE=1 -DMOZ_LOGGING=1 -DMOZ_USER_DIR=\".mozilla\" -DMOZ_XUL=1 -DMOZ_PROFILELOCKING=1 -DMOZ_DLL_SUFFIX=\".so\" -DXP_UNIX=1 -DUNIX_ASYNC_DNS=1 -DJS_THREADSAFE=1 -DMOZILLA_LOCALE_VERSION=\"1.9a1\" -DMOZILLA_REGION_VERSION=\"1.9a1\" -DMOZILLA_SKIN_VERSION=\"1.8\"  -D_MOZILLA_CONFIG_H_ -DMOZILLA_CLIENT nsWebBrowserPersist.cpp
"nsWebBrowserPersist.cpp", line 3284: Error: "nsWebBrowserPersist::StoreURIAttribute(nsIDOMNode*, const char*, int, URIData**)" has already been called and cannot be defined inline.
"nsWebBrowserPersist.cpp", line 3401: Error: "nsWebBrowserPersist::FixupNodeAttribute(nsIDOMNode*, const char*)" has already been called and cannot be defined inline.
2 Error(s) detected.
gmake[7]: *** [nsWebBrowserPersist.o] Error 2


Reproducible: Always

Steps to Reproduce:
1. Attempt a build from cvs
2.
3.

Actual Results:  
Build failure

Comment 1

[Mitch]

Looks like a similar problem to what i logged a while back with bad inlining ! See bug 224968 for the options to fixing this.

Comment 2

[Jonathan Watt [:jwatt]]

Attachment: bustage fix

Sorry! I've checked in so it should cycle green shortly.

Comment 3

[Jonathan Watt [:jwatt]]

Oh, I thought there was a tinderbox that had gone red. I guess not. Anyway, marking fixed. Appologies once again for the broken inlining.