From 0c124ba807b8a16f4119df64cb11734ed8e4ac1b Mon Sep 17 00:00:00 2001
From: "David A. Madore" <david+git@madore.org>
Date: Fri, 16 Apr 2010 17:38:31 +0200
Subject: Use static initializers correctly (I had forgotten the "static"
 keyword!).

---
 org/madore/damlengine/TodoAttr.java | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

(limited to 'org/madore/damlengine/TodoAttr.java')

diff --git a/org/madore/damlengine/TodoAttr.java b/org/madore/damlengine/TodoAttr.java
index b9cb800..2b3df23 100644
--- a/org/madore/damlengine/TodoAttr.java
+++ b/org/madore/damlengine/TodoAttr.java
@@ -11,8 +11,7 @@ public abstract class TodoAttr extends TodoItem {
 
     protected static Map<String,Constructor<? extends TodoAttr>> damlAttrConstructors;
 
-    protected static void initializeDamlAttrConstructors() {
-	// FIXME: this should be a static initializer, but for some reason does not work...
+    static {
 	damlAttrConstructors = new HashMap<String,Constructor<? extends TodoAttr>>();
 	Class[] argTypes = new Class[]{ Attr.class, Element.class, Map.class, Map.class };
 	try {
@@ -40,8 +39,6 @@ public abstract class TodoAttr extends TodoItem {
 				       Map<String,Object> options) {
 	Constructor<? extends TodoAttr> constructor = null;
 	String nsuri = attr.getNamespaceURI();
-	if ( damlAttrConstructors == null )
-	    initializeDamlAttrConstructors(); // FIXME: see above
 	if ( nsuri != null && nsuri.equals(DamlEngine.DAML_NS) )
 	    constructor = damlAttrConstructors.get(attr.getLocalName());
 	if ( constructor != null )
-- 
cgit v1.2.3