[classlib] Requirement: Regex support for \p{javaDigit}

2006-06-28 Thread Richard Liang

Hello All,

I'm trying to implement java.util.Scanner.nextInt, which requires a 
regular expression pattern to represent  Integer. I find it's hard to 
express /NonASCIIDigit /because now Harmony regex does not support 
/NonASCIIDigit/.


As our regex has supported \p{javaLowerCase} Equivalent to 
java.lang.Character.isLowerCase(), to facilitate the development of 
j.u.Scanner, shall we provide support for \p{javaDigit}, which is 
equivalent to java.lang.Character.isDigit()? I could provide patch for 
this feature if there is no objection. Thanks lot.

*
*You may want to refer to the number syntax in java.util.Scanner Spec:*
*/QUOTE
NonASCIIDigit  :: = A non-ASCII character c for which 
Character.isDigit(c) returns true


Non0Digit  ::= [1-Rmax] | NonASCIIDigit

Digit  ::= [0-Rmax] | NonASCIIDigit

GroupedNumeral  ::  = ( Non0Digit Digit? Digit? ( LocalGroupSeparator 
Digit Digit Digit )+ )


Numeral  ::= ( ( Digit+ )  |  GroupedNumeral )

Integer  ::= ( [-+]? ( Numeral ) )  |  LocalPositivePrefix Numeral 
LocalPositiveSuffix  |  LocalNegativePrefix Numeral LocalNegativeSuffix/

//QUOTE/


--
Richard Liang
China Software Development Lab, IBM 



Re: [classlib] Requirement: Regex support for \p{javaDigit}

2006-06-28 Thread Nikolay Kuznetsov

Hello Richard,
Thanks for pointing this out. While implementing java character
classes I missed the fact that _all_ Character.isXXX methods except
deprecated should have corresponding \\p{javaXXX} character class
analogues. So the the answer to your question whether we should
implement javaDigit char class is YES.

I've opened an issue to support the rest of character classes:
http://issues.apache.org/jira/browse/HARMONY-696

Also, thanks a lot for your willing to add javaDigit class, but if you
don't mind I would implement full range of {javaXXX} classes by myself
just because it would be easier for me, as for an author :-).
Hopefully, I'll be ready with this tomorrow.

On 6/28/06, Richard Liang [EMAIL PROTECTED] wrote:

Hello All,

I'm trying to implement java.util.Scanner.nextInt, which requires a
regular expression pattern to represent  Integer. I find it's hard to
express /NonASCIIDigit /because now Harmony regex does not support
/NonASCIIDigit/.

As our regex has supported \p{javaLowerCase} Equivalent to
java.lang.Character.isLowerCase(), to facilitate the development of
j.u.Scanner, shall we provide support for \p{javaDigit}, which is
equivalent to java.lang.Character.isDigit()? I could provide patch for
this feature if there is no objection. Thanks lot.
*
*You may want to refer to the number syntax in java.util.Scanner Spec:*
*/QUOTE
NonASCIIDigit  :: = A non-ASCII character c for which
Character.isDigit(c) returns true

Non0Digit  ::= [1-Rmax] | NonASCIIDigit

Digit  ::= [0-Rmax] | NonASCIIDigit

GroupedNumeral  ::  = ( Non0Digit Digit? Digit? ( LocalGroupSeparator
Digit Digit Digit )+ )

Numeral  ::= ( ( Digit+ )  |  GroupedNumeral )

Integer  ::= ( [-+]? ( Numeral ) )  |  LocalPositivePrefix Numeral
LocalPositiveSuffix  |  LocalNegativePrefix Numeral LocalNegativeSuffix/
//QUOTE/


--
Richard Liang
China Software Development Lab, IBM





-
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [classlib] Requirement: Regex support for \p{javaDigit}

2006-06-28 Thread Richard Liang

That's great! Nik. You're so helpful. Thanks a lot.  ;-)

Nikolay Kuznetsov wrote:

Hello Richard,
Thanks for pointing this out. While implementing java character
classes I missed the fact that _all_ Character.isXXX methods except
deprecated should have corresponding \\p{javaXXX} character class
analogues. So the the answer to your question whether we should
implement javaDigit char class is YES.

I've opened an issue to support the rest of character classes:
http://issues.apache.org/jira/browse/HARMONY-696

Also, thanks a lot for your willing to add javaDigit class, but if you
don't mind I would implement full range of {javaXXX} classes by myself
just because it would be easier for me, as for an author :-).
Hopefully, I'll be ready with this tomorrow.

On 6/28/06, Richard Liang [EMAIL PROTECTED] wrote:

Hello All,

I'm trying to implement java.util.Scanner.nextInt, which requires a
regular expression pattern to represent  Integer. I find it's hard to
express /NonASCIIDigit /because now Harmony regex does not support
/NonASCIIDigit/.

As our regex has supported \p{javaLowerCase} Equivalent to
java.lang.Character.isLowerCase(), to facilitate the development of
j.u.Scanner, shall we provide support for \p{javaDigit}, which is
equivalent to java.lang.Character.isDigit()? I could provide patch for
this feature if there is no objection. Thanks lot.
*
*You may want to refer to the number syntax in java.util.Scanner Spec:*
*/QUOTE
NonASCIIDigit  :: = A non-ASCII character c for which
Character.isDigit(c) returns true

Non0Digit  ::= [1-Rmax] | NonASCIIDigit

Digit  ::= [0-Rmax] | NonASCIIDigit

GroupedNumeral  ::  = ( Non0Digit Digit? Digit? ( LocalGroupSeparator
Digit Digit Digit )+ )

Numeral  ::= ( ( Digit+ )  |  GroupedNumeral )

Integer  ::= ( [-+]? ( Numeral ) )  |  LocalPositivePrefix Numeral
LocalPositiveSuffix  |  LocalNegativePrefix Numeral LocalNegativeSuffix/
//QUOTE/


--
Richard Liang
China Software Development Lab, IBM





-
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




--
Richard Liang
China Software Development Lab, IBM 




-
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]