--- stack/stack.c	2002/03/10 06:34:01	1.95
+++ stack/stack.c	2002/03/10 08:30:43	1.97
@@ -48,7 +48,7 @@
 {
   int i;
 
-  env->gc_limit= 20;
+  env->gc_limit= 200;
   env->gc_count= 0;
   env->gc_ref= NULL;
   env->gc_protect= NULL;
@@ -83,7 +83,6 @@
   free(temp);			/* Free the old top stack item */
 
   env->gc_limit--;
-  gc_init(env);
 }
 
 /* Returns a pointer to a pointer to an element in the hash table. */
@@ -128,21 +127,18 @@
   env->gc_ref= nitem;
 
   env->gc_count++;
-
-  protect(env, nval);
-  gc_init(env);
-  unprotect(env);
+  nval->gc_garb= 1;
 
   return nval;
 }
 
 /* Mark values recursively. 
    Marked values are not collected by the GC. */
-void gc_mark(value *val)
+inline void gc_mark(value *val)
 {
   stackitem *iterator;
 
-  if(val==NULL || val->gc_garb==0)
+  if(val->gc_garb==0)
     return;
 
   val->gc_garb= 0;
@@ -157,6 +153,14 @@
   }
 }
 
+inline void gc_maybe(environment *env)
+{
+  if(env->gc_count < env->gc_limit)
+    return;
+  else
+    return gc_init(env);
+}
+
 /* Start GC */
 extern void gc_init(environment *env)
 {
@@ -164,16 +168,6 @@
   symbol *tsymb;
   int i;
 
-  if(env->gc_count < env->gc_limit)
-    return; 
-
-  /* Garb by default */
-  iterator= env->gc_ref;
-  while(iterator!=NULL) {
-    iterator->item->gc_garb= 1;
-    iterator= iterator->next;
-  }
-
   /* Mark protected values */
   iterator= env->gc_protect;
   while(iterator!=NULL) {
@@ -192,7 +186,8 @@
   for(i= 0; i<HASHTBLSIZE; i++) {
     tsymb= env->symbols[i];
     while(tsymb!=NULL) {
-      gc_mark(tsymb->val);
+      if (tsymb->val != NULL)
+	gc_mark(tsymb->val);
       tsymb= tsymb->next;
     }
   }
@@ -213,21 +208,22 @@
 	  env->gc_ref->item->content.ptr= titem->next;
 	  free(titem);
 	}
-	break;
       default:
-	break;
       }
       free(env->gc_ref->item);	/* Remove from gc_ref */
       titem= env->gc_ref->next;
       free(env->gc_ref);	/* Remove value */
       env->gc_ref= titem;
-    } else {			/* Keep values */
-      titem= env->gc_ref->next;
-      env->gc_ref->next= new_head;
-      new_head= env->gc_ref;
-      env->gc_ref= titem;
-      env->gc_count++;
+      continue;
     }
+    
+    /* Keep values */
+    titem= env->gc_ref->next;
+    env->gc_ref->next= new_head;
+    new_head= env->gc_ref;
+    new_head->item->gc_garb= 1;
+    env->gc_ref= titem;
+    env->gc_count++;
   }
 
   env->gc_limit= env->gc_count*2;
@@ -613,6 +609,8 @@
 
  eval_start:
 
+  gc_maybe(env);
+
   if(env->head==NULL) {
     printerr("Too Few Arguments");
     env->err=1;
@@ -869,7 +867,7 @@
   }
 
   env->gc_limit= 0;
-  gc_init(env);
+  gc_maybe(env);
 
   if(env->free_string!=NULL)
     free(env->free_string);
@@ -980,7 +978,7 @@
   }
 
   if(myenv.interactive) {
-    printf("Stack version $Revision: 1.95 $\n\
+    printf("Stack version $Revision: 1.97 $\n\
 Copyright (C) 2002  Mats Alritzson and Teddy Hogeborn\n\
 Stack comes with ABSOLUTELY NO WARRANTY; for details type `warranty;'.\n\
 This is free software, and you are welcome to redistribute it\n\
@@ -1008,7 +1006,7 @@
       toss(&myenv); 		/* No error check in main */
       eval(&myenv);
     }
-    gc_init(&myenv);
+    gc_maybe(&myenv);
   }
   quit(&myenv);
   return EXIT_FAILURE;