[svn:parrot] r43354 - branches/pmc_freeze_cleanup/src

plobsing at svn.parrot.org plobsing at svn.parrot.org
Fri Jan 1 17:07:11 UTC 2010


Author: plobsing
Date: Fri Jan  1 17:07:09 2010
New Revision: 43354
URL: https://trac.parrot.org/parrot/changeset/43354

Log:
remove more thaw constant code and document lack of thaw constant in docs

Modified:
   branches/pmc_freeze_cleanup/src/pmc_freeze.c

Modified: branches/pmc_freeze_cleanup/src/pmc_freeze.c
==============================================================================
--- branches/pmc_freeze_cleanup/src/pmc_freeze.c	Fri Jan  1 10:08:03 2010	(r43353)
+++ branches/pmc_freeze_cleanup/src/pmc_freeze.c	Fri Jan  1 17:07:09 2010	(r43354)
@@ -631,8 +631,6 @@
             /* BEGIN create appropriate pmc */
             if (info->what == VISIT_THAW_NORMAL)
                 pmc = pmc_new_noinit(interp, type);
-            else if (info->what == VISIT_THAW_CONSTANTS)
-                pmc = constant_pmc_new_noinit(interp, type);
             else
                 Parrot_ex_throw_from_c_args(interp, NULL, 1, "Illegal info->what type");
             /* END create appropriate pmc */
@@ -731,8 +729,7 @@
     ASSERT_ARGS(visit_loop_todo_list)
     PMC        **list_item;
     List * const todo           = (List *)PMC_data(info->todo);
-    const int    thawing        = info->what == VISIT_THAW_CONSTANTS
-                               || info->what == VISIT_THAW_NORMAL;
+    const int    thawing        = info->what == VISIT_THAW_NORMAL;
 
     /* can't cache upper limit, visit may append items */
     do {
@@ -746,11 +743,6 @@
 
             PARROT_ASSERT(current->vtable);
 
-            /* Workaround for thawing constants. Clear constant flag */
-            /* See src/packfile.c:3999 */
-            if (thawing)
-                PObj_constant_CLEAR(current);
-
             VTABLE_visit(interp, current, info);
         }
         current = NULL;
@@ -918,6 +910,7 @@
 =item C<PMC* Parrot_thaw_constants(PARROT_INTERP, STRING *image)>
 
 Thaws constants, used by PackFile for unpacking PMC constants.
+This is a lie. It does nothing different from Parrot_thaw at the moment.
 
 =cut
 
@@ -930,7 +923,7 @@
 Parrot_thaw_constants(PARROT_INTERP, ARGIN(STRING *image))
 {
     ASSERT_ARGS(Parrot_thaw_constants)
-    return run_thaw(interp, image, VISIT_THAW_CONSTANTS);
+    return Parrot_thaw(interp, image);
 }
 
 


More information about the parrot-commits mailing list