[Sheets-cvs] sheets Sheets.sheets,1.60,1.61

[<ox-...@us...>]

Update of /cvsroot/sheets/sheets
In directory sc8-pr-cvs1:/tmp/cvs-serv10367

Modified Files:
	Sheets.sheets 
Log Message:
1. Fixed a timing bug where you could cause a history manager error by closing sheets immediately after importing a lot of code.  The synchronization checks added should be good for stability all around.
2. New version of OX with the following changes:
a. Made the garbage collection algorithm more aggressive about cleaning up outdated references (in anticipation of implementing compaction soon)
b. Performance tweaking to make sure the aforementioned change does not reduce performance.


Index: Sheets.sheets
===================================================================
RCS file: /cvsroot/sheets/sheets/Sheets.sheets,v
retrieving revision 1.60
retrieving revision 1.61
diff -C2 -d -r1.60 -r1.61
*** Sheets.sheets	12 Jun 2003 08:14:29 -0000	1.60
--- Sheets.sheets	15 Jun 2003 10:57:33 -0000	1.61
***************
*** 37200,37205 ****
   type=java
  section text ram:479
!  public static void HistoryManager.endChangeCommand
!    (String cmd, boolean lethalIntent) {
     if (!initialized) return;
     
--- 37200,37204 ----
   type=java
  section text ram:479
!  public synchronized static void HistoryManager.endChangeCommand(String cmd, boolean lethalIntent) {
     if (!initialized) return;
     
***************
*** 38611,38616 ****
   // has any effect.  We allow this nesting so that commands can call other
   // commands, etc.
!  public static void HistoryManager.beginChangeCommand
!      (String cmd) {
     if (!initialized) return;
     if (changeCommandStack.empty()) {
--- 38610,38614 ----
   // has any effect.  We allow this nesting so that commands can call other
   // commands, etc.
!  public synchronized static void HistoryManager.beginChangeCommand(String cmd) {
     if (!initialized) return;
     if (changeCommandStack.empty()) {
***************
*** 54657,54661 ****
   // by passing the return value to "restartChangeCommand" or cancelled by simply
   // taking no action.
!  public static Stack HistoryManager.deferChangeCommand () { 
     Stack retval = changeCommandStack;
     changeCommandStack = new Stack();
--- 54655,54659 ----
   // by passing the return value to "restartChangeCommand" or cancelled by simply
   // taking no action.
!  public synchronized static Stack HistoryManager.deferChangeCommand () { 
     Stack retval = changeCommandStack;
     changeCommandStack = new Stack();
***************
*** 54667,54672 ****
  section text rgs:8735
   // See "deferChangeCommand" for details.
!  public static void HistoryManager.restartChangeCommand (Stack old) 
!  { changeCommandStack = old; }
  object rgs:8761
   pkg=org.browsecode.sheets
--- 54665,54671 ----
  section text rgs:8735
   // See "deferChangeCommand" for details.
!  public synchronized static void HistoryManager.restartChangeCommand (Stack old) {
!    changeCommandStack = old;
!  }
  object rgs:8761
   pkg=org.browsecode.sheets