Changeset 284

Timestamp:

04/14/10 15:17:12 (2 years ago)

Author:

sean

Message:

A post of Steve's in the NG reminded me that delegates for opApply need to be "scope" or they'll allocte a dynamic stack frame. Now they are.

Files:

• trunk/import/object.di (modified) (4 diffs)
• trunk/src/core/runtime.d (modified) (1 diff)
• trunk/src/core/thread.d (modified) (2 diffs)
• trunk/src/object_.d (modified) (4 diffs)

trunk/import/object.di

@@ -245 +245 @@
-    @property void function() unitTest();
-
-
+scope 
-}
-
@@ -258 +258 @@
-    interface TraceInfo
-    {
-
+scope 
-        string toString();
-    }
@@ -300 +300 @@
-    void*  _aaRehash(void** pp, TypeInfo keyti);
-
-
+scope 
-    int _aaApply(void* aa, size_t keysize, _dg_t dg);
-
-
+scope 
-    int _aaApply2(void* aa, size_t keysize, _dg2_t dg);
-
@@ -333 +333 @@
-    }
-
-
+scope 
-    {
-        return _aaApply2(p, Key.sizeof, cast(_dg2_t)dg);
-    }
-
-
+scope 
-    {
-        return _aaApply(p, Key.sizeof, cast(_dg_t)dg);

trunk/src/core/runtime.d

@@ -261 +261 @@
-            }
-            
-
+scope 
-            {
-                // NOTE: The first 5 frames with the current implementation are

trunk/src/core/thread.d

@@ -1158 +1158 @@
-     *  Zero if all elemented are visited, nonzero if not.
-     */
-
+scope 
-    {
-        synchronized( slock )
@@ -2422 +2422 @@
-     * Operates on all threads currently tracked by this object.
-     */
-
+scope 
-    {
-        synchronized( this )

trunk/src/object_.d

@@ -1122 +1122 @@
-    interface TraceInfo
-    {
-
+scope 
-    }
-
@@ -1565 +1565 @@
-
-
-
+scope 
-    {
-        int ret = 0;
@@ -2079 +2079 @@
-    void* _aaRehash(void** pp, TypeInfo keyti);
-
-
+scope 
-    int _aaApply(void* aa, size_t keysize, _dg_t dg);
-
-
+scope 
-    int _aaApply2(void* aa, size_t keysize, _dg2_t dg);
-
@@ -2112 +2112 @@
-    }
-
-
+scope 
-    {
-        return _aaApply2(p, Key.sizeof, cast(_dg2_t)dg);
-    }
-
-
+scope 
-    {
-        return _aaApply(p, Key.sizeof, cast(_dg_t)dg);