Re: svn commit: r793587 - in /harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/apache/harmony/jndi/provider/ldap/parser: FilterParserTokenManager.java LdapUrlParserTokenManager.java

[Nathan Beyer]

Is this class generated code? Should the change be made to a BNF file  
instead?

Sent from my iPhone
On Jul 13, 2009, at 8:54 AM, odea...@apache.org wrote:

Author: odeakin
Date: Mon Jul 13 13:54:57 2009
New Revision: 793587

URL: http://svn.apache.org/viewvc?rev=793587&view=rev
Log:
Handle the z/OS NEL character correctly when parsing tokens.

Modified:
   harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/ 
apache/harmony/jndi/provider/ldap/parser/FilterParserTokenManager.java
   harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/ 
apache/harmony/jndi/provider/ldap/parser/ 
LdapUrlParserTokenManager.java

Modified: harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/ 
org/apache/harmony/jndi/provider/ldap/parser/ 
FilterParserTokenManager.java
URL: 
http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/apache/harmony/jndi/provider/ldap/parser/FilterParserTokenManager.java?rev=793587&r1=793586&r2=793587&view=diff
=== 
=== 
=== 
=====================================================================
--- harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/ 
apache/harmony/jndi/provider/ldap/parser/ 
FilterParserTokenManager.java (original)
+++ harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/ 
apache/harmony/jndi/provider/ldap/parser/ 
FilterParserTokenManager.java Mon Jul 13 13:54:57 2009
@@ -63,6 +63,7 @@
   switch(curChar)
   {
      case 10:
+      case 133: // NEL character for z/OS new lines
         return jjStopAtPos(0, 24);
      case 33:
         return jjStopAtPos(0, 10);

Modified: harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/ 
org/apache/harmony/jndi/provider/ldap/parser/ 
LdapUrlParserTokenManager.java
URL: 
http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/apache/harmony/jndi/provider/ldap/parser/LdapUrlParserTokenManager.java?rev=793587&r1=793586&r2=793587&view=diff
=== 
=== 
=== 
=====================================================================
--- harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/ 
apache/harmony/jndi/provider/ldap/parser/ 
LdapUrlParserTokenManager.java (original)
+++ harmony/enhanced/classlib/trunk/modules/jndi/src/main/java/org/ 
apache/harmony/jndi/provider/ldap/parser/ 
LdapUrlParserTokenManager.java Mon Jul 13 13:54:57 2009
@@ -61,6 +61,7 @@
   switch(curChar)
   {
      case 10:
+      case 133: // NEL character for z/OS new lines
         return jjStopAtPos(0, 17);
      case 33:
         return jjStopAtPos(0, 10);