You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by sc...@apache.org on 2007/07/16 05:12:19 UTC

svn commit: r556499 - /incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/TypeSystemImpl.java

Author: schor
Date: Sun Jul 15 20:12:17 2007
New Revision: 556499

URL: http://svn.apache.org/viewvc?view=rev&rev=556499
Log:
[UIMA-500] remove 2 unneeded calls to ll_isValidTypeCode

Modified:
    incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/TypeSystemImpl.java

Modified: incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/TypeSystemImpl.java
URL: http://svn.apache.org/viewvc/incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/TypeSystemImpl.java?view=diff&rev=556499&r1=556498&r2=556499
==============================================================================
--- incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/TypeSystemImpl.java (original)
+++ incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/TypeSystemImpl.java Sun Jul 15 20:12:17 2007
@@ -1353,6 +1353,7 @@
     if (!isCommitted())
       this.numCommittedTypes = this.types.size();
     this.arrayCodeToTypeMap.put(arrayTypeCode, arrayType);
+//    System.out.println("*** adding to arrayCodeToTypeMap: " + arrayType.getName() + ", committed=" + isCommitted());
     // For built-in arrays, we need to add the abstract base array as parent
     // to the inheritance tree. This sucks. Assumptions about the base
     // array are all over the place. Would be nice to just remove it.
@@ -1372,9 +1373,9 @@
   }
 
   public boolean ll_isArrayType(int typeCode) {
-    if (!ll_isValidTypeCode(typeCode)) {
-      return false;
-    }
+//    if (!ll_isValidTypeCode(typeCode)) {
+//      return false;
+//    }
     return this.arrayCodeToTypeMap.containsKey(typeCode);
   }
 
@@ -1391,9 +1392,9 @@
   }
 
   public String[] ll_getStringSet(int typeCode) {
-    if (!ll_isValidTypeCode(typeCode)) {
-      return null;
-    }
+//    if (!ll_isValidTypeCode(typeCode)) {
+//      return null;
+//    }
     if (!ll_isStringSubtype(typeCode)) {
       return null;
     }