You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2005/06/21 09:02:09 UTC

svn commit: r191636 - /directory/sandbox/trunk/asn1-new-codec/src/java/org/apache/asn1/ldap/codec/grammars/AddResponseGrammar.java

Author: elecharny
Date: Tue Jun 21 00:02:06 2005
New Revision: 191636

URL: http://svn.apache.org/viewcvs?rev=191636&view=rev
Log:
Deleted the useless LENGTH transitions

Modified:
    directory/sandbox/trunk/asn1-new-codec/src/java/org/apache/asn1/ldap/codec/grammars/AddResponseGrammar.java

Modified: directory/sandbox/trunk/asn1-new-codec/src/java/org/apache/asn1/ldap/codec/grammars/AddResponseGrammar.java
URL: http://svn.apache.org/viewcvs/directory/sandbox/trunk/asn1-new-codec/src/java/org/apache/asn1/ldap/codec/grammars/AddResponseGrammar.java?rev=191636&r1=191635&r2=191636&view=diff
==============================================================================
--- directory/sandbox/trunk/asn1-new-codec/src/java/org/apache/asn1/ldap/codec/grammars/AddResponseGrammar.java (original)
+++ directory/sandbox/trunk/asn1-new-codec/src/java/org/apache/asn1/ldap/codec/grammars/AddResponseGrammar.java Tue Jun 21 00:02:06 2005
@@ -69,13 +69,14 @@
         // AddResponse ::= [APPLICATION 9] LDAPResult (Tag)
         // Nothing to do.
         super.transitions[LdapStatesEnum.ADD_RESPONSE_TAG][0x69] = new GrammarTransition(
-                LdapStatesEnum.ADD_RESPONSE_TAG, LdapStatesEnum.ADD_RESPONSE_LENGTH, null );
+                LdapStatesEnum.ADD_RESPONSE_TAG, LdapStatesEnum.ADD_RESPONSE_VALUE, null );
 
-        // We have to allocate a AddResponse
         // LdapMessage ::= ... AddResponse ...
-        // AddResponse ::= [APPLICATION 9] LDAPResult (Length)
-        super.transitions[LdapStatesEnum.ADD_RESPONSE_LENGTH][0x69] = new GrammarTransition(
-                LdapStatesEnum.ADD_RESPONSE_LENGTH, LdapStatesEnum.ADD_RESPONSE_VALUE,
+        // AddResponse ::= [APPLICATION 9] LDAPResult (Value)
+        // The next Tag will be the LDAPResult Tag (0x0A).
+        // We will switch the grammar then.
+        super.transitions[LdapStatesEnum.ADD_RESPONSE_VALUE][0x69] = new GrammarTransition(
+                LdapStatesEnum.ADD_RESPONSE_VALUE, LdapStatesEnum.ADD_RESPONSE_LDAP_RESULT, 
                 new GrammarAction( "Init AddResponse" )
                 {
                     public void action( IAsn1Container container ) throws DecoderException
@@ -86,10 +87,7 @@
                         LdapMessage      ldapMessage          =
                             ldapMessageContainer.getLdapMessage();
 
-                        checkLength( ldapMessageContainer.getLdapMessage(),
-                            ldapMessageContainer.getCurrentTLV() );
-
-                        // Now, we can allocate the AdddRequest Object
+                        // Now, we can allocate the AddRequest Object
                         AddResponse addResponse = new AddResponse();
 
                         // As this is a new Constructed object, we have to init its length
@@ -103,13 +101,6 @@
                         ldapMessage.setProtocolOP( addResponse );
                     }
                 } );
-
-        // LdapMessage ::= ... AddResponse ...
-        // AddResponse ::= [APPLICATION 9] LDAPResult (Value)
-        // The next Tag will be the LDAPResult Tag (0x0A).
-        // We will switch the grammar then.
-        super.transitions[LdapStatesEnum.ADD_RESPONSE_VALUE][0x69] = new GrammarTransition(
-                LdapStatesEnum.ADD_RESPONSE_VALUE, LdapStatesEnum.ADD_RESPONSE_LDAP_RESULT, null );
 
         // LdapMessage ::= ... AddResponse ...
         // AddResponse ::= [APPLICATION 9] LDAPResult (Value)