593747 - TM: 64-bit VC9: dist\include\jsval.h(636) : error C2143: syntax error : missing ';' before 'type'

Status: RESOLVED FIXED

Description

[Makoto Kato [:m_kato]]

This is tracemonkey tree.

jsval.h is included from C source files such as jsdebug.c in js/jsd.  So variable define should be before assertion code.


c:/mozilla-build/python25/python2.5.exe -O c:/Workspace/hg.mozilla.org/tracemonkey/build/cl.py cl -Fojsdebug.obj -c -DXPCOM_TRANSLATE_NSGM_ENTRY_POINT=1 -DMOZILLA_INTERNAL_API -D_IMPL_NS_COM -DEXPORT_XPT_API -DEXPORT_XPTC_API -D_IMPL_NS_GFX -D_IMPL_NS_WIDGET -DIMPL_XREAPI -DIMPL_NS_NET -DIMPL_THEBES  -DSTATIC_EXPORTABLE_JS_API -DZLIB_INTERNAL -DOSTYPE=\"WINNT6.0\" -DOSARCH=WINNT -DEXPORT_JSD_API  -I/c/Workspace/hg.mozilla.org/tracemonkey/js/jsd -I. -I../../dist/include -I../../dist/include/nsprpub  -Ic:/Workspace/hg.mozilla.org/objdir-tm-win64/dist/include/nspr -Ic:/Workspace/hg.mozilla.org/objdir-tm-win64/dist/include/nss         -TC -nologo -W3 -Gy -Fdgenerated.pdb  -DNDEBUG -DTRIMMED -Zi -Zi -UDEBUG -DNDEBUG -GL -O1 -MD            -FI ../../dist/include/mozilla-config.h -DMOZILLA_CLIENT /c/Workspace/hg.mozilla.org/tracemonkey/js/jsd/jsdebug.c
jsdebug.c
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(636) : error C2143: syntax error : missing ';' before 'type'
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(638) : error C2065: 'ptrBits' : undeclared identifier
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(674) : error C2143: syntax error : missing ';' before 'type'
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(676) : error C2065: 'objBits' : undeclared identifier
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(681) : error C2061: syntax error : identifier 'JSVAL_IS_NULL_IMPL'
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(681) : error C2059: syntax error : ';'
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(681) : error C2059: syntax error : 'type'
c:\workspace\hg.mozilla.org\objdir-tm-win64\dist\include\jsval.h(703) : warning C4013: 'JSVAL_IS_NULL_IMPL' undefined; assuming extern returning int

Comment 1

[Makoto Kato [:m_kato]]

Created attachment 472327 [details] [diff] [review]
fix

Comment 2

[Makoto Kato [:m_kato]]

http://hg.mozilla.org/tracemonkey/rev/a4da54094b4c

Comment 3

[Makoto Kato [:m_kato]]

already merged into m-c
http://hg.mozilla.org/mozilla-central/rev/a4da54094b4c