bugfixes mainly...

[mir.git] / source / mircoders / storage / DatabaseBreaking.java

diff --git a/source/mircoders/storage/DatabaseBreaking.java b/source/mircoders/storage/DatabaseBreaking.java
index 888bc4e..50af618 100755 (executable)
--- a/source/mircoders/storage/DatabaseBreaking.java
+++ b/source/mircoders/storage/DatabaseBreaking.java
@@ -19,29 +19,30 @@ import mir.misc.*;
 
 public class DatabaseBreaking extends Database implements StorageObject{
 
-       private static DatabaseBreaking instance;
-
-       public static DatabaseBreaking getInstance() throws StorageObjectException {
-               if (instance == null) {
-                       instance = new DatabaseBreaking();
-                       instance.myselfDatabase = instance;
-               }
-               return instance;
-       }
-
-       private DatabaseBreaking() throws StorageObjectException
-       {
-               super();
-    theLog.printDebugInfo("-- outside database");
-               this.cache = new HashMap();
-               this.theTable="breaking";
-               try {
-                       this.theEntityClass = Class.forName("mir.entity.EntityBreaking");
-               }
-               catch (Exception e) { throw new StorageObjectException(e.toString());   }
-
-    theLog.printDebugInfo("-- leaving dbb");
-       }
+  private static DatabaseBreaking instance;
+
+  // the following *has* to be sychronized cause this static method
+  // could get preemted and we could end up with 2 instances of DatabaseFoo..
+  // see the "Singletons with needles and thread" article at JavaWorld -mh
+  public synchronized static DatabaseBreaking getInstance() 
+    throws StorageObjectException {
+    if (instance == null) {
+      instance = new DatabaseBreaking();
+      instance.myselfDatabase = instance;
+    }
+    return instance;
+  }
+
+  private DatabaseBreaking() throws StorageObjectException
+  {
+    super();
+    //this.cache = new DatabaseCache(4);
+    this.theTable="breaking";
+    try {
+      this.theEntityClass = Class.forName("mircoders.entity.EntityBreaking");
+    }
+    catch (Exception e) { throw new StorageObjectException(e.toString()); }
+  }
 
 
 }