harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Oliver Deakin <oliver.dea...@googlemail.com>
Subject 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
Date Tue, 14 Jul 2009 15:50:42 GMT
Yes, that's right actually - Ive been looking at the grammar files but 
can't figure out the right place to make the change. Does anyone have a 
good enough knowledge of the javacc tool to figure out the correct change?

Regards,
Oliver

Nathan Beyer wrote:
> 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, odeakin@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);
>>
>>
>

-- 
Oliver Deakin
Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU


Mime
View raw message