Refactor DistributionPointName

Author: peterdettman

core/src/main/java/org/bouncycastle/asn1/x509/DistributionPointName.java

@@ -6,6 +6,7 @@
 import org.bouncycastle.asn1.ASN1Primitive;
 import org.bouncycastle.asn1.ASN1Set;
 import org.bouncycastle.asn1.ASN1TaggedObject;
+import org.bouncycastle.asn1.ASN1Util;
 import org.bouncycastle.asn1.DERTaggedObject;
 import org.bouncycastle.util.Strings;
 
@@ -22,21 +23,10 @@ public class DistributionPointName
     extends ASN1Object
     implements ASN1Choice
 {
-    ASN1Encodable        name;
-    int                 type;
-
     public static final int FULL_NAME = 0;
     public static final int NAME_RELATIVE_TO_CRL_ISSUER = 1;
 
-    public static DistributionPointName getInstance(
-        ASN1TaggedObject obj,
-        boolean          explicit)
-    {
-        return getInstance(ASN1TaggedObject.getInstance(obj, true));
-    }
-
-    public static DistributionPointName getInstance(
-        Object  obj)
+    public static DistributionPointName getInstance(Object obj)
     {
         if (obj == null || obj instanceof DistributionPointName)
         {
@@ -50,6 +40,19 @@ else if (obj instanceof ASN1TaggedObject)
         throw new IllegalArgumentException("unknown object in factory: " + obj.getClass().getName());
     }
 
+    public static DistributionPointName getInstance(ASN1TaggedObject taggedObject, boolean declaredExplicit)
+    {
+        return getInstance(ASN1Util.getInstanceChoiceBaseObject(taggedObject, declaredExplicit, "DistributionPointName"));
+    }
+
+    public static DistributionPointName getTagged(ASN1TaggedObject taggedObject, boolean declaredExplicit)
+    {
+        return getInstance(ASN1Util.getTaggedChoiceBaseObject(taggedObject, declaredExplicit, "DistributionPointName"));
+    }
+
+    private final ASN1Encodable name;
+    private final int type;
+
     public DistributionPointName(
         int             type,
         ASN1Encodable   name)
@@ -83,22 +86,25 @@ public ASN1Encodable getName()
     {
         return (ASN1Encodable)name;
     }
-    
-    public DistributionPointName(
-        ASN1TaggedObject    obj)
+
+    public DistributionPointName(ASN1TaggedObject obj)
     {
         this.type = obj.getTagNo();
-        
-        if (type == 0)
+
+        if (obj.hasContextTag(FULL_NAME))
         {
             this.name = GeneralNames.getInstance(obj, false);
         }
-        else
+        else if (obj.hasContextTag(NAME_RELATIVE_TO_CRL_ISSUER))
         {
             this.name = ASN1Set.getInstance(obj, false);
         }
+        else
+        {
+            throw new IllegalArgumentException("unknown tag: " + ASN1Util.getTagText(obj));
+        }
     }
-    
+
     public ASN1Primitive toASN1Primitive()
     {
         return new DERTaggedObject(false, type, name);