377751 - Switching JSClass.mark in XPConnect to the tracing semantics

Status: RESOLVED FIXED

(Core :: XPCOM, enhancement)

Description

[Igor Bukanov]

This is a continuation of work from bug 375270 comment 38 to switch all JSMarkOp functions to the new JSTraceOp semantics.

Comment 1

[Igor Bukanov]

Attachment: Implementation v1

The implementation is a mostly rename exercise. But also pints to a deficiency of the GC and new trace infrastructure as the code would not trace auto roots during non-GC tracing. Effectively any calls to JS_GetGCMarkingTracer means that potentially something is not exposed to the generic tracer and the patch has to add the call the API in 3 places.

I will fix that in bug 340212.

Comment 2

[Igor Bukanov]

Comment on attachment 261787 [details] [diff] [review]
Implementation v1

Who should I ask for sr= ?

Comment 3

[Igor Bukanov]

Recording patch dependency of the fix for bug 340212 on this bug.

Comment 4

[Igor Bukanov]

Attachment: Implementation v2

The new version consistently uses TraceJS for all methods that should trace JS things in a paerticular class/structure. The tracing code from XPCWrappedNativeScope::FinishedMarkPhaseOfGC and JSBool XPCJSRuntime::GCCallback is also moved to separated TraceJS methods to simplify a fix for bug 340212.

Comment 5

[Brendan Eich [:brendan]]

Comment on attachment 261945 [details] [diff] [review]
Implementation v2

>+ * Is called to trace things that the object helds.

s/helds/holds/

>+void XPCDispIDArray::TraceJS(JSTracer* trc)
> {
>     // If already marked nothing to do
>-    if(IsMarked())
>-        return;
>-    mMarked = JS_TRUE;
>-    XPCCallContext ccx(NATIVE_CALLER);
>-    // Bail if our call context is bad
>-    if(!ccx.IsValid())
>-        return;
>+    if(JS_IsGCMarkingTracer(trc)) {

Nit: brace on new line as elsewhere in xpconnect/src.

r=me, as peer (am I listed? I'll check) of xpconnect.

/be

Comment 6

[Igor Bukanov]

Attachment: Implementation v2b

Here is a new version of the patch to address the couple of nits from the previous comment.

Comment 7

[Igor Bukanov]

Ping for sr=something

Comment 8

[Igor Bukanov]

Comment on attachment 262659 [details] [diff] [review]
Implementation v2b

The patch is not complete, I missed the msIXPCScriptable.idl and friends.

Comment 9

[Igor Bukanov]

Attachment: Implementation v3

This is the previous patch + changes to nsIXPCScriptable.idl and dependent files   to replace mark by trace.

There are also changes to XPCDispIDArray where I removed duplicated TraceJS declaration. But since XPCDispTypeInfo.cpp is not part of the build, I do not now how to verify the changes there.

Comment 10

[Johnny Stenback (:jst)]

Comment on attachment 262893 [details] [diff] [review]
Implementation v3

- In js/src/xpconnect/idl/nsIXPCScriptable.idl:

-    PRUint32 mark(in nsIXPConnectWrappedNative wrapper,
-                  in JSContextPtr cx, in JSObjectPtr obj, in voidPtr arg);
+    void trace(in nsIXPConnectWrappedNative wrapper,
+               in JSTracerPtr cx, in JSObjectPtr obj);

The JSTracerPtr argument should be named something other than "cx" here too.

And since you're changing the signature of this interface you'll need to generate a new IID.

sr=jst with that.

Comment 11

[Brendan Eich [:brendan]]

Comment on attachment 262893 [details] [diff] [review]
Implementation v3

What jst said -- I'll check over a final patch when you attach.

/be

Comment 12

[Igor Bukanov]

Attachment: Implementation v4

The new version to address comment 10.

Comment 13

[Igor Bukanov]

Attachment: Implementation v4 for real

I forgot to regenerate CVS diff the last time, now this should address comment 10.

Comment 14

[Brendan Eich [:brendan]]

Comment on attachment 262999 [details] [diff] [review]
Implementation v4 for real


>+++ js/src/xpconnect/src/XPCDispPrivate.h	27 Apr 2007 07:25:08 -0000
>@@ -302,30 +302,25 @@ public:
>      * @param cx a JS context
>      * @param index index into the array
>      * @return the ID as a jsval
>      */
>     jsval Item(JSContext* cx, PRUint32 index) const;
> 
>     /**
>-     * Called to mark the ID's during GC
>+     * Called to trace jsval associated with the ID's
>      */
>-    void Mark();
>+    void TraceJS(JSTracer* trc);
>     /**
>-     * Called to unmark the ID's after GC has been done
>+     * Called to unmark the ID's marked during GC marking trace
>      */
>     void Unmark();
>     /**
>      * Tests whether the ID is marked
>      */
>     JSBool IsMarked() const;

While you are here, how about a blank line before each doc-comment?

>+void XPCDispIDArray::TraceJS(JSTracer* trc)
> {
>     // If already marked nothing to do
>-    if(IsMarked())
>-        return;
>-    mMarked = JS_TRUE;
>-    XPCCallContext ccx(NATIVE_CALLER);
>-    // Bail if our call context is bad
>-    if(!ccx.IsValid())
>-        return;
>+    if(JS_IsGCMarkingTracer(trc))
>+    {
>+        if (IsMarked())
>+            return;
>+        mMarked = JS_TRUE;
>+    }
> 
>     PRInt32 count = Length();
>     jsval val;
>-    JSContext* cx = ccx;
>+
>     // Iterate each of the ID's and mark them
>     for(PRInt32 index = 0; index < count; ++index)
>     {
>         if(JS_IdToValue(cx,

This won't compile -- use trc->context or restore the cx local (can you test the IDispatch stuff at least to see that it compiles?).

> mozStorageStatementRow::Equality(nsIXPConnectWrappedNative *wrapper,
>@@ -997,18 +996,18 @@ mozStorageStatementParams::Construct(nsI
> NS_IMETHODIMP
> mozStorageStatementParams::HasInstance(nsIXPConnectWrappedNative *wrapper, JSContext * cx,
>                                     JSObject * obj, jsval val, PRBool *bp, PRBool *_retval)
> {
>     return NS_ERROR_NOT_IMPLEMENTED;
> }
> 
>-/* PRUint32 mark (in nsIXPConnectWrappedNative wrapper, in JSContextPtr cx, in JSObjectPtr obj, in voidPtr arg); */
>+/* void trace (in nsIXPConnectWrappedNative wrapper, in JSTracerPtr trc, in JSObjectPtr obj); */
> NS_IMETHODIMP
>-mozStorageStatementParams::Mark(nsIXPConnectWrappedNative *wrapper, JSContext * cx,
>-                             JSObject * obj, void * arg, PRUint32 *_retval)
>+mozStorageStatementParams::Trace(nsIXPConnectWrappedNative *wrapper,
>+                                JSTracer *trc, JSObject * obj)

Nit: could line up overflow params to underhang as usual (even for pre-existing nit above this one).

r=me with fixes, sorry I didn't catch them earlier.

/be

Comment 15

[Igor Bukanov]

(In reply to comment #14)
> >+void XPCDispIDArray::TraceJS(JSTracer* trc)
...
> This won't compile -- use trc->context or restore the cx local (can you test
> the IDispatch stuff at least to see that it compiles?).

I am fixing the bugs in the body of TraceJS, but that does not make XPCDispTypeInfo.cpp to compile (I used the same command that make uses to compile other files in js/src/xpconnect/src):
 
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: variable or field 'FillOutElemDesc' declared void
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: 'FillOutElemDesc' declared as an 'inline' variable
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: 'VARTYPE' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: expected primary-expression before 'paramFlags'
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: 'ELEMDESC' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: 'elemDesc' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:59: error: initializer expression list treated as compound expression
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:60: error: expected ',' or ';' before '{' token
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:69: error: expected constructor, destructor, or type conversion before '::' token
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:74: error: 'XPCDispJSPropertyInfo' has not been declared
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:74: error: 'ELEMDESC' has not been declared
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp: In function 'void GetReturnType(XPCCallContext&, int&)':
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:76: error: 'VARTYPE' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:76: error: expected `;' before 'vt'
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:77: error: 'IsSetter' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:79: error: 'vt' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:79: error: 'VT_EMPTY' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:81: error: 'IsProperty' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:83: error: 'vt' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:83: error: 'XPCDispConvert' has not been declared
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:83: error: 'mProperty' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:87: error: 'vt' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:87: error: 'VT_VARIANT' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:89: error: 'vt' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:89: error: 'PARAMFLAG_FRETVAL' was not declared in this scope
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:89: error: 'FillOutElemDesc' cannot be used as a function
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp: At global scope:
/home/igor/m/trunk/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp:92: error: expected constructor, destructor, or type conversion before '*'

Any particular reason for keeping XPCDispTypeInfo.cpp together with other sources?

Comment 16

[Igor Bukanov]

Attachment: Implementation v5

This is the previous patch + changes to address nits from comment 14. 

In storage/src/mozStorageStatementWrapper.cpp the bad indentation exists throughout the whole file so I fixed only indentation only for Trace that replaces Mark keeping the rest of the source unchanged.

Comment 17

[Igor Bukanov]

I committed the patch from comment 16 to the trunk:

Waiting for Emacs...Done
Checking in dom/src/base/nsDOMClassInfo.cpp;
/cvsroot/mozilla/dom/src/base/nsDOMClassInfo.cpp,v  <--  nsDOMClassInfo.cpp
new revision: 1.437; previous revision: 1.436
done
Checking in js/src/xpconnect/idl/nsIXPCScriptable.idl;
/cvsroot/mozilla/js/src/xpconnect/idl/nsIXPCScriptable.idl,v  <--  nsIXPCScriptable.idl
new revision: 1.10; previous revision: 1.9
done
Checking in js/src/xpconnect/idl/nsIXPConnect.idl;
/cvsroot/mozilla/js/src/xpconnect/idl/nsIXPConnect.idl,v  <--  nsIXPConnect.idl
new revision: 1.48; previous revision: 1.47
done
Checking in js/src/xpconnect/public/xpc_map_end.h;
/cvsroot/mozilla/js/src/xpconnect/public/xpc_map_end.h,v  <--  xpc_map_end.h
new revision: 1.9; previous revision: 1.8
done
Checking in js/src/xpconnect/src/XPCDispInlines.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispInlines.h,v  <--  XPCDispInlines.h
new revision: 1.13; previous revision: 1.12
done
Checking in js/src/xpconnect/src/XPCDispParamPropJSClass.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispParamPropJSClass.cpp,v  <--  XPCDispParamPropJSClass.cpp
new revision: 1.6; previous revision: 1.5
done
Checking in js/src/xpconnect/src/XPCDispPrivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispPrivate.h,v  <--  XPCDispPrivate.h
new revision: 1.18; previous revision: 1.17
done
Checking in js/src/xpconnect/src/XPCDispTypeInfo.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp,v  <--  XPCDispTypeInfo.cpp
new revision: 1.10; previous revision: 1.9
done
Checking in js/src/xpconnect/src/XPCNativeWrapper.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCNativeWrapper.cpp,v  <--  XPCNativeWrapper.cpp
new revision: 1.48; previous revision: 1.47
done
Checking in js/src/xpconnect/src/xpcjsruntime.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcjsruntime.cpp,v  <--  xpcjsruntime.cpp
new revision: 1.48; previous revision: 1.47
done
Checking in js/src/xpconnect/src/xpcprivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/xpcprivate.h,v  <--  xpcprivate.h
new revision: 1.208; previous revision: 1.207
done
Checking in js/src/xpconnect/src/xpcthreadcontext.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcthreadcontext.cpp,v  <--  xpcthreadcontext.cpp
new revision: 1.45; previous revision: 1.44
done
Checking in js/src/xpconnect/src/xpcwrappednative.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp,v  <--  xpcwrappednative.cpp
new revision: 1.134; previous revision: 1.133
done
Checking in js/src/xpconnect/src/xpcwrappednativejsops.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp,v  <--  xpcwrappednativejsops.cpp
new revision: 1.67; previous revision: 1.66
done
Checking in js/src/xpconnect/src/xpcwrappednativescope.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativescope.cpp,v  <--  xpcwrappednativescope.cpp
new revision: 1.34; previous revision: 1.33
done
Checking in storage/src/mozStorageStatementWrapper.cpp;
/cvsroot/mozilla/storage/src/mozStorageStatementWrapper.cpp,v  <--  mozStorageStatementWrapper.cpp
new revision: 1.11; previous revision: 1.10
done

Comment 18

[Igor Bukanov]

Attachment: XPCDispprivate.h build fix

Extra patch to fix windows compilation problem. I wonder how I could make it compile on Linux?

Comment 19

[Igor Bukanov]

I committed the patch from comment 18 to the trunk to fix win compilation:

Waiting for Emacs...Done
Checking in XPCDispInlines.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispInlines.h,v  <--  XPCDispInlines.h
new revision: 1.14; previous revision: 1.13
done
Checking in XPCDispPrivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispPrivate.h,v  <--  XPCDispPrivate.h
new revision: 1.19; previous revision: 1.18
done

Comment 20

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

This broke all the Windows tinderboxes:

e:\builds\tinderbox\fx-trunk\winnt_5.2_depend\mozilla\js\src\xpconnect\src\XPCDispInlines.h(467) : error C2511: 'void XPCDispIDArray::Mark(void)' : overloaded member function not found in 'XPCDispIDArray'
        e:\builds\tinderbox\fx-trunk\winnt_5.2_depend\mozilla\js\src\xpconnect\src\XPCDispPrivate.h(287) : see declaration of 'XPCDispIDArray'

Comment 21

[Igor Bukanov]

Attachment: another XPCDispprivate.h build fix

Another fix I just committed to the trunk:

Waiting for Emacs...Done
Checking in XPCDispPrivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispPrivate.h,v  <--  XPCDispPrivate.h
new revision: 1.20; previous revision: 1.19
done

If this still would lead to compilation probnlems, I will remove the whole patch.

Comment 22

[Igor Bukanov]

Attachment: another XPCDispprivate.h build fix for real

This the committed fix for real:

Checking in XPCDispPrivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispPrivate.h,v  <--  XPCDispPrivate.h
new revision: 1.20; previous revision: 1.19
done

Comment 23

[Igor Bukanov]

I removed the commited patch:

Waiting for Emacs...Done
Checking in dom/src/base/nsDOMClassInfo.cpp;
/cvsroot/mozilla/dom/src/base/nsDOMClassInfo.cpp,v  <--  nsDOMClassInfo.cpp
new revision: 1.438; previous revision: 1.437
done
Checking in js/src/xpconnect/idl/nsIXPCScriptable.idl;
/cvsroot/mozilla/js/src/xpconnect/idl/nsIXPCScriptable.idl,v  <--  nsIXPCScriptable.idl
new revision: 1.11; previous revision: 1.10
done
Checking in js/src/xpconnect/idl/nsIXPConnect.idl;
/cvsroot/mozilla/js/src/xpconnect/idl/nsIXPConnect.idl,v  <--  nsIXPConnect.idl
new revision: 1.49; previous revision: 1.48
done
Checking in js/src/xpconnect/public/xpc_map_end.h;
/cvsroot/mozilla/js/src/xpconnect/public/xpc_map_end.h,v  <--  xpc_map_end.h
new revision: 1.10; previous revision: 1.9
done
Checking in js/src/xpconnect/src/XPCDispInlines.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispInlines.h,v  <--  XPCDispInlines.h
new revision: 1.15; previous revision: 1.14
done
Checking in js/src/xpconnect/src/XPCDispParamPropJSClass.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispParamPropJSClass.cpp,v  <--  XPCDispParamPropJSClass.cpp
new revision: 1.7; previous revision: 1.6
done
Checking in js/src/xpconnect/src/XPCDispPrivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispPrivate.h,v  <--  XPCDispPrivate.h
new revision: 1.21; previous revision: 1.20
done
Checking in js/src/xpconnect/src/XPCDispTypeInfo.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp,v  <--  XPCDispTypeInfo.cpp
new revision: 1.11; previous revision: 1.10
done
Checking in js/src/xpconnect/src/XPCNativeWrapper.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCNativeWrapper.cpp,v  <--  XPCNativeWrapper.cpp
new revision: 1.49; previous revision: 1.48
done
Checking in js/src/xpconnect/src/xpcjsruntime.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcjsruntime.cpp,v  <--  xpcjsruntime.cpp
new revision: 1.49; previous revision: 1.48
done
Checking in js/src/xpconnect/src/xpcprivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/xpcprivate.h,v  <--  xpcprivate.h
new revision: 1.209; previous revision: 1.208
done
Checking in js/src/xpconnect/src/xpcthreadcontext.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcthreadcontext.cpp,v  <--  xpcthreadcontext.cpp
new revision: 1.46; previous revision: 1.45
done
Checking in js/src/xpconnect/src/xpcwrappednative.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp,v  <--  xpcwrappednative.cpp
new revision: 1.135; previous revision: 1.134
done
Checking in js/src/xpconnect/src/xpcwrappednativejsops.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp,v  <--  xpcwrappednativejsops.cpp
new revision: 1.68; previous revision: 1.67
done
Checking in js/src/xpconnect/src/xpcwrappednativescope.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativescope.cpp,v  <--  xpcwrappednativescope.cpp
new revision: 1.35; previous revision: 1.34
done
Checking in storage/src/mozStorageStatementWrapper.cpp;
/cvsroot/mozilla/storage/src/mozStorageStatementWrapper.cpp,v  <--  mozStorageStatementWrapper.cpp
new revision: 1.12; previous revision: 1.11
done

Comment 24

[Igor Bukanov]

Attachment: Implementation v6

PCDispTypeInfo.cpp is a part of the build when XPC_IDISPATCH_SUPPORT is set. According to the configure script this is only enabled when:

case "$target_os" in
12492     msvc*|mks*|cygwin*|mingw*)
12493         if test -z "$GNU_CC"; then 
12494             XPC_IDISPATCH_SUPPORT=1

So AFAICS I can not verify that part of the patch on a Linux box and has to rely on just manually checking the code :(

In any case here is an updated patch that should fix the problems
time here is the updated fix that should fix the problems.

The patch removes the following lines in XPCDispPrivate.h:
-    /**
-     * NOP. This is just here to make the AutoMarkingPtr code compile.
-     */
-    inline void MarkBeforeJSFinalize(JSContext*);

The comment is wrong as XPCDispIDArray can not be used with AutoMarkingPtr as it does not define AutoMark which is called from AutoMarkingPtr. Moreover, AFAICS neither MarkBeforeJSFinalize no Mark are never in fact invoked so I do not know what purpose they have. So the patch tries to preserve the logic in the implementation providing only TraceJS method as a replacement for both MarkBeforeJSFinalize and Mark.

Comment 25

[Brendan Eich [:brendan]]

Comment on attachment 263400 [details] [diff] [review]
Implementation v6

sr=me, letting jst r= this one.

/be

Comment 26

[Johnny Stenback (:jst)]

Comment on attachment 263400 [details] [diff] [review]
Implementation v6

r=jst

Comment 27

[Igor Bukanov]

I committed the patch from comment 24 to the trunk:

Checking in dom/src/base/nsDOMClassInfo.cpp;
/cvsroot/mozilla/dom/src/base/nsDOMClassInfo.cpp,v  <--  nsDOMClassInfo.cpp
new revision: 1.440; previous revision: 1.439
done
Checking in js/src/xpconnect/idl/nsIXPCScriptable.idl;
/cvsroot/mozilla/js/src/xpconnect/idl/nsIXPCScriptable.idl,v  <--  nsIXPCScriptable.idl
new revision: 1.12; previous revision: 1.11
done
Checking in js/src/xpconnect/idl/nsIXPConnect.idl;
/cvsroot/mozilla/js/src/xpconnect/idl/nsIXPConnect.idl,v  <--  nsIXPConnect.idl
new revision: 1.50; previous revision: 1.49
done
Checking in js/src/xpconnect/public/xpc_map_end.h;
/cvsroot/mozilla/js/src/xpconnect/public/xpc_map_end.h,v  <--  xpc_map_end.h
new revision: 1.11; previous revision: 1.10
done
Checking in js/src/xpconnect/src/XPCDispInlines.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispInlines.h,v  <--  XPCDispInlines.h
new revision: 1.16; previous revision: 1.15
done
Checking in js/src/xpconnect/src/XPCDispParamPropJSClass.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispParamPropJSClass.cpp,v  <--  XPCDispParamPropJSClass.cpp
new revision: 1.8; previous revision: 1.7
done
Checking in js/src/xpconnect/src/XPCDispPrivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispPrivate.h,v  <--  XPCDispPrivate.h
new revision: 1.22; previous revision: 1.21
done
Checking in js/src/xpconnect/src/XPCDispTypeInfo.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCDispTypeInfo.cpp,v  <--  XPCDispTypeInfo.cpp
new revision: 1.12; previous revision: 1.11
done
Checking in js/src/xpconnect/src/XPCNativeWrapper.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCNativeWrapper.cpp,v  <--  XPCNativeWrapper.cpp
new revision: 1.50; previous revision: 1.49
done
Checking in js/src/xpconnect/src/xpcjsruntime.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcjsruntime.cpp,v  <--  xpcjsruntime.cpp
new revision: 1.50; previous revision: 1.49
done
Checking in js/src/xpconnect/src/xpcprivate.h;
/cvsroot/mozilla/js/src/xpconnect/src/xpcprivate.h,v  <--  xpcprivate.h
new revision: 1.210; previous revision: 1.209
done
Checking in js/src/xpconnect/src/xpcthreadcontext.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcthreadcontext.cpp,v  <--  xpcthreadcontext.cpp
new revision: 1.47; previous revision: 1.46
done
Checking in js/src/xpconnect/src/xpcwrappednative.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp,v  <--  xpcwrappednative.cpp
new revision: 1.136; previous revision: 1.135
done
Checking in js/src/xpconnect/src/xpcwrappednativejsops.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp,v  <--  xpcwrappednativejsops.cpp
new revision: 1.69; previous revision: 1.68
done
Checking in js/src/xpconnect/src/xpcwrappednativescope.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/xpcwrappednativescope.cpp,v  <--  xpcwrappednativescope.cpp
new revision: 1.36; previous revision: 1.35
done
Checking in storage/src/mozStorageStatementWrapper.cpp;
/cvsroot/mozilla/storage/src/mozStorageStatementWrapper.cpp,v  <--  mozStorageStatementWrapper.cpp
new revision: 1.13; previous revision: 1.12
done

Comment 28

[Stefan Sitter [:ssitter]]

It seems this checkin has busted all the Sunbird and Lightning Tinderboxes [http://tinderbox.mozilla.org/Sunbird]>:

mozilla/calendar/base/src/calDateTime.cpp(943) : 
error C2039: 'Mark' : is not a member of 'calDateTime'

Comment 29

[Igor Bukanov]

(In reply to comment #28)
> It seems this checkin has busted all the Sunbird and Lightning Tinderboxes
> [http://tinderbox.mozilla.org/Sunbird]>:
> 
> mozilla/calendar/base/src/calDateTime.cpp(943) : 
> error C2039: 'Mark' : is not a member of 'calDateTime'

I will fix it ASAP.

Comment 30

[Igor Bukanov]

Attachment: fix for mozilla/calendar/base/src/calDateTime.cpp

An extra patch to fix calendar compilation problem.

Comment 31

[Igor Bukanov]

I committed the patch from comment 30 fix the calendar:

Checking in calendar/base/src/calDateTime.cpp;
/cvsroot/mozilla/calendar/base/src/calDateTime.cpp,v  <--  calDateTime.cpp
new revision: 1.64; previous revision: 1.63
done